-
Notifications
You must be signed in to change notification settings - Fork 32
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
Adds filtering out of FASTA for tools that don't support it #60
Conversation
|
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.
I think we could benefit from some logging/error formatting facilities but otherwise this looks good.
subworkflows/local/profiling.nf
Outdated
|
||
ch_input_for_metaphlan3 = ch_input_for_profiling.metaphlan3 | ||
.filter{ | ||
if (it[0].is_fasta) log.warn "[nf-core/taxprofiler] MetaPhlAn3 currently does not accept FASTA files as input. Skipping MetaPhlAn3 for sample " + it[0].id |
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.
Similar to above
if (it[0].is_fasta) log.warn "[nf-core/taxprofiler] MetaPhlAn3 currently does not accept FASTA files as input. Skipping MetaPhlAn3 for sample " + it[0].id | |
if (it[0].is_fasta) log.warn "[nf-core/taxprofiler] MetaPhlAn3 currently does not accept FASTA files as input. Skipping MetaPhlAn3 for sample ${it[0].id}." |
Co-authored-by: Moritz E. Beber <midnighter@posteo.net>
This reverts commit b7d0557.
Ok actually, as this will require more work - I will revert the commit with the logging thing and make an issue - I agree the nf-core logging stuff would be much better here but I've not looked into it (nor Groovy stuff) |
PR checklist
Closes #56
Basically if a file is marked as a 'fasta' file, will only select those which are NOT FASTA for tools such as MetaPhlAn3, and also spits out a warning that sample is being ignored.
Note a couple of things:
.filter()
operator closure does NOT seem to supportmeta, reads ->
syntax, gives some wierd error aboutcall()
Also standardises warning messages
nf-core lint
).nextflow run . -profile test,docker --outdir <OUTDIR>
).docs/usage.md
is updated.docs/output.md
is updated.CHANGELOG.md
is updated.README.md
is updated (including new tool citations and authors/contributors).