New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add skani round 2 #5539
base: master
Are you sure you want to change the base?
Add skani round 2 #5539
Conversation
@jfy133 back in action part2. I'm still experimenting with the best way to add the |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM (just some small comments)
It's also advised to only add one module per PR, I know it's easier to add multiple in the same PR (been there, done that), but no one really wants to review a large PR 😁
description: | | ||
FASTA/skani sketch file to be used as query. | ||
pattern: "*.{fasta,fasta.gz,fa,fa.gz,fna,fna.gz,sketch}" | ||
- reference: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you add meta2
here too?
description: | | ||
Input FASTA/skani sketch to search with. | ||
pattern: "*.{fasta,fasta.gz,fa,fa.gz,fna,fna.gz}" | ||
- sketch_dir: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same here
'biocontainers/skani:0.2.1--h4ac6f70_0' }" | ||
|
||
input: | ||
tuple val(meta) , path(queries) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
tuple val(meta) , path(queries) | |
tuple val(meta), path(queries) |
@CarsonJM , can you make the adjustments suggested here? |
Yes I can! I am in the middle of a wet-lab onslaught at the moment and have fallen behind on nf-core stuff, but am hoping to be back at it soon! |
PR checklist
Re-opens #5502
versions.yml
file.label
nf-core modules test <MODULE> --profile docker
nf-core modules test <MODULE> --profile singularity
nf-core modules test <MODULE> --profile conda
nf-core subworkflows test <SUBWORKFLOW> --profile docker
nf-core subworkflows test <SUBWORKFLOW> --profile singularity
nf-core subworkflows test <SUBWORKFLOW> --profile conda