update test full profile#290
Conversation
|
Warning Newer version of the nf-core template is available. Your pipeline is using an old version of the nf-core template: 3.5.2. For more documentation on how to update your pipeline, please see the nf-core documentation and Synchronisation documentation. |
LouisLeNezet
left a comment
There was a problem hiding this comment.
I'm not sure about this modification.
Stitch and quilt are quite slow.
This will increase quite a lot the credit consumption on seqera.
Do we really need it ?
|
We can inspect results, and if they are slow, then we have an opportunity to improve how inputs are processed when reaching these tools. Sarek launches even different tests with different tools on cloud: https://github.com/nf-core/sarek/blob/master/.github/workflows/cloudtest.yml. I can ask core or the community how they handle this so we can be sure :) |
If we could have someone experienced with big test to help us this would be great. |
LouisLeNezet
left a comment
There was a problem hiding this comment.
If we can monitor it and stop it before it cost too much, it's all good for me.
I agree it would be nice to have a test with all the tools !
PR checklist
nf-core pipelines lint).nextflow run . -profile test,docker --outdir <OUTDIR>).nextflow run . -profile debug,test,docker --outdir <OUTDIR>).docs/usage.mdis updated.docs/output.mdis updated.CHANGELOG.mdis updated.README.mdis updated (including new tool citations and authors/contributors).