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 GvsExtractCallset.example.inputs.json #7469

Merged
merged 4 commits into from
Sep 15, 2021
Merged

Conversation

kcibul
Copy link
Contributor

@kcibul kcibul commented Sep 14, 2021

No description provided.

@gatk-bot
Copy link

gatk-bot commented Sep 14, 2021

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

Test Type JDK Job ID Logs
unit openjdk11 36024.13 logs
unit openjdk8 36024.3 logs

@@ -19,7 +19,7 @@ workflow GvsImportGenomes {
Int batch_size = 1

Int? preemptible_tries
File? gatk_override
File? gatk_override = "gs://broad-dsp-spec-ops/scratch/bigquery-jointcalling/jars/ah_var_store_20210914/gatk-package-4.2.0.0-406-ga9206a2-SNAPSHOT-local.jar"
Copy link
Contributor

Choose a reason for hiding this comment

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

is there any benefit to making this a constant somewhere so we dont have to update it in multiple places? ( I guess it's only 2, so not really an issue though)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

yeah and I don't know of a good way to have a constant/shared value across two WDLs

@gatk-bot
Copy link

gatk-bot commented Sep 14, 2021

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

Test Type JDK Job ID Logs
unit openjdk11 36027.13 logs
unit openjdk8 36027.3 logs

@kcibul kcibul merged commit 7b3eb29 into ah_var_store Sep 15, 2021
@kcibul kcibul deleted the kcibul-patch-1 branch September 15, 2021 13:29
RoriCremer pushed a commit that referenced this pull request Sep 20, 2021
* updated default overrider jar
* fixed bugs in removed headers, processing Avro PLs and unit tests
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.

4 participants