Skip to content

GATK UnifiedGenotyper Fix due to Broad Website change#332

Merged
apeltzer merged 13 commits intonf-core:devfrom
jfy133:gatkug_fix
Jan 15, 2020
Merged

GATK UnifiedGenotyper Fix due to Broad Website change#332
apeltzer merged 13 commits intonf-core:devfrom
jfy133:gatkug_fix

Conversation

@jfy133
Copy link
Copy Markdown
Member

@jfy133 jfy133 commented Jan 10, 2020

Unfortunately, GATK have removed the ability to (easily) download the old jar with their new website. GATK 3.5 might actually be 'lost' now.

I have changed the system to require the user to supply their own JAR, if they want to use UnifiedGenotyper. Accordingly, I've removed the UG (and downstream tools requiring UG) github actions/travis test for now. Can consider replacing some downstream steps (e.g. vcf2genome with bcftools consensus - https://samtools.github.io/bcftools/howtos/consensus-sequence.html).

I may have a lead where could alternatively download the JAR (and add to the github actions step a download command), but waiting for feedback from Broad.

Important The failure to download UG is why the other PRs are currently failing, this will need to be merged first.

PR checklist

  • This comment contains a description of changes (with reason)
  • If you've fixed a bug or added code that should be tested, add tests!
  • Ensure the test suite passes (nextflow run . -profile test,docker).
  • Make sure your code lints (nf-core lint .).
  • Documentation in docs is updated

Learn more about contributing: https://github.com/nf-core/eager/tree/master/.github/CONTRIBUTING.md

@jfy133 jfy133 added the major label Jan 10, 2020
@jfy133 jfy133 added this to the V2.1 "Ulm" milestone Jan 10, 2020
@jfy133 jfy133 requested a review from apeltzer January 10, 2020 10:22
@jfy133 jfy133 self-assigned this Jan 10, 2020
@jfy133 jfy133 changed the title Gatkug fix GATK UnifiedGenotyper Fix due to Broad Website change Jan 10, 2020
@jfy133
Copy link
Copy Markdown
Member Author

jfy133 commented Jan 10, 2020

Alternatively could switch to the new (now they've fixed it), storage location: https://console.cloud.google.com/storage/browser/gatk-software/package-archive/gatk

But I don't trust them anymore ;)

@jfy133 jfy133 added the WIP Work in progress label Jan 15, 2020
@jfy133
Copy link
Copy Markdown
Member Author

jfy133 commented Jan 15, 2020

New download location avaliable for GATK 3.5 from here

I've kept new JAR system for EAGER, but re-enabled tests (hopefully) with a pre-run download step in github actions workflow.

Copy link
Copy Markdown
Member

@apeltzer apeltzer left a comment

Choose a reason for hiding this comment

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

I actually love that they moved things to a better (stable) URL :-D

@apeltzer apeltzer merged commit 91b5e38 into nf-core:dev Jan 15, 2020
@jfy133
Copy link
Copy Markdown
Member Author

jfy133 commented Jan 15, 2020

Unfortunately still couldn't get the tests to work, but will hopefully do it soon.

I agree!

@jfy133 jfy133 deleted the gatkug_fix branch February 10, 2020 14:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

major WIP Work in progress

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants