Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update to latest version of ah_var_store gatk override jar #7793

Merged
merged 1 commit into from
Apr 20, 2022

Conversation

gbggrant
Copy link
Collaborator

No description provided.

@gbggrant gbggrant requested review from mcovarr and kcibul April 19, 2022 13:40
@gatk-bot
Copy link

Travis reported job failures from build 38949
Failures in the following jobs:

Test Type JDK Job ID Logs
conda openjdk8 38949.5 logs

@@ -14,7 +14,7 @@ workflow GvsCreateFilterSet {
Array[String] snp_recalibration_annotation_values = ["AS_QD", "AS_MQRankSum", "AS_ReadPosRankSum", "AS_FS", "AS_MQ", "AS_SOR"]

File interval_list = "gs://gcp-public-data--broad-references/hg38/v0/wgs_calling_regions.hg38.noCentromeres.noTelomeres.interval_list"
File gatk_override = "gs://broad-dsp-spec-ops/scratch/bigquery-jointcalling/jars/ah_var_store_20220406/gatk-package-4.2.0.0-480-gb62026a-SNAPSHOT-local.jar"
File gatk_override = "gs://broad-dsp-spec-ops/scratch/bigquery-jointcalling/jars/ah_var_store_20220415/gatk-package-4.2.0.0-492-g1387d47-SNAPSHOT-local.jar"
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this is at least a couple of revisions back now so it wouldn't be picking up the zero-padding improvements from #7783

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

True - I'll rebuild it - but the only gatk jar improvements in 7783 were really just cleaning up some logging.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

oh yeah you're right... wow did I misremember that code 😅

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

in an ideal world, I think we would also create these ahead of our releases and have one named for the release---so we'd have a Charlie jar (though we've re-made it a few times over the last few weeks....)

Copy link

@rsasch rsasch left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks for this 👍🏻

@gbggrant gbggrant force-pushed the gg_RestoreRscriptConditionally branch from b5ef520 to 3dd04f3 Compare April 20, 2022 19:05
@gatk-bot
Copy link

Travis reported job failures from build 38970
Failures in the following jobs:

Test Type JDK Job ID Logs
conda openjdk8 38970.5 logs

@gatk-bot
Copy link

Travis reported job failures from build 38972
Failures in the following jobs:

Test Type JDK Job ID Logs
conda openjdk8 38972.5 logs

@gbggrant gbggrant merged commit 8d08a6e into ah_var_store Apr 20, 2022
@gbggrant gbggrant deleted the gg_RestoreRscriptConditionally branch April 20, 2022 21:11
This was referenced Mar 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants