nf-core / modules

Repository to host tool-specific module files for the Nextflow DSL2 community!
https://nf-co.re/modules
MIT License
263 stars 661 forks source link

Update a few container declarations #3799

Open edmundmiller opened 11 months ago

edmundmiller commented 11 months ago

While working on #3797 I found a few containers sections that need updating(because I was matching against them 😆):

lukbut commented 9 months ago

What needs to be updated @Emiller88?

edmundmiller commented 9 months ago

Great question! 😆

The containers have the full registry URL.

container "nvcr.io/nvidia/clara/clara-parabricks:4.0.1-1"

as compared to

biocontainers/samtools:1.17--h00cdaf9_0

The issue is that https://github.com/emiller88/modules/blob/57455d80bf19140829c9c0cab037692a9681441a/tests/config/nf-test.config#L37 put it in front of nvcr.io

So I'm not sure if we can use quay.io with the nvidia images... But I'm also not sure if they work currently?