populationgenomics / images

Reusable single-software docker images
MIT License
1 stars 1 forks source link

update saige-qtl image with new step3 #127

Closed annacuomo closed 9 months ago

annacuomo commented 9 months ago

linked to the other PR on the association pipeline (https://github.com/populationgenomics/saige-tenk10k/pull/29).

Hopefully this is now ready to approve + build?

annacuomo commented 9 months ago

LGTM - so for testing purposes your script will use /usr/local/bin/X as default values?

yes i was going to come to that in the other PR! I was hoping that would work, yes

annacuomo commented 9 months ago

LGTM - so for testing purposes your script will use /usr/local/bin/X as default values?

yes i was going to come to that in the other PR! I was hoping that would work, yes

@MattWellie will this work? Just checking before I merge :) https://github.com/populationgenomics/saige-tenk10k/pull/29/commits/2a6ad6bf2c141eba9428dc4e7154e7373acfa317