Skip to content
This repository was archived by the owner on Jan 27, 2020. It is now read-only.

new Dockerfile for new container gatk4#603

Merged
szilvajuhos merged 2 commits intoSciLifeLab:masterfrom
maxulysse:GATK4
Jun 20, 2018
Merged

new Dockerfile for new container gatk4#603
szilvajuhos merged 2 commits intoSciLifeLab:masterfrom
maxulysse:GATK4

Conversation

@maxulysse
Copy link
Copy Markdown
Member

Docker container is accessible at maxulysse/gatk4.0:latest or maxulysse/gatk4.0:2.0
https://hub.docker.com/r/maxulysse/gatk4/tags/

@maxulysse maxulysse self-assigned this Jun 20, 2018
@maxulysse maxulysse requested a review from szilvajuhos June 20, 2018 08:21
Copy link
Copy Markdown
Collaborator

@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.

Why don't you use a conda recipe for this ? Would make your image much "cleaner". There are reipes for GATK4+ available (even more than for GATK3.X).

@maxulysse
Copy link
Copy Markdown
Member Author

Very good point
Will change that ;-)

@szilvajuhos
Copy link
Copy Markdown
Collaborator

Cool, I was testing the container before accepting the PR, but had to delete 40G docker rubbish first on my machine. So, will wait for the new version ...

@maxulysse
Copy link
Copy Markdown
Member Author

It's slightly bigger 647 MB vs 506 MB, but the installation and the update will indeed be easier, cleaner...

@szilvajuhos szilvajuhos merged commit 72e01ba into SciLifeLab:master Jun 20, 2018
@maxulysse maxulysse deleted the GATK4 branch June 20, 2018 11:34
jherrero referenced this pull request in UCL-BLIC/Sarek_v2.2.1 Apr 11, 2019
new Dockerfile for new container gatk4
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants