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
Update nextflow_schema.json and fix mzTab output #210
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.
Looks good to me!
withName: 'OPENMS_DECOYDATABASE|OPENMS_THERMORAWFILEPARSER|OPENMS_PEAKPICKERHIRES|OPENMS_PEPTIDEINDEXER|OPENMS_FALSEDISCOVERYRATE|OPENMS_IDMERGER|OPENMS_MAPRTTRANSFORMER.*ML|OPENMS_FEATURE.*|OPENMS_IDCONFLICTRESOLVER' { | ||
publishDir = [ enabled: false ] | ||
} | ||
|
||
withName: 'OPENMS_MZTABEXPORTER_QUANT' { |
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.
afaik definitions of OPENMS_MZTABEXPORTER
are inherited to OPENMS_MZTABEXPORTER_QUANT
because include { OPENMS_MZTABEXPORTER as OPENMS_MZTABEXPORTER_QUANT }
. I typically try to avoid specifying the base name (i.e. here OPENMS_MZTABEXPORTER
) but rather rename all processes (i.e. use include as
) so that those confusions do not happen. Just a comment. In this case here all should go fine.
|
||
### `Fixed` | ||
|
||
Resolved the problem that there was no output from `OPENMS_MZTABEXPORTER_QUANT` |
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'd typically add the link to the PR, i.e.
Resolved the problem that there was no output from `OPENMS_MZTABEXPORTER_QUANT` | |
- [#210](https://github.com/nf-core/mhcquant/pull/210) - Resolved the problem that there was no output from `OPENMS_MZTABEXPORTER_QUANT` |
Co-authored-by: Daniel Straub <42973691+d4straub@users.noreply.github.com>
This PR includes a better definition of saving the mzTab files in the
Intermediate
folder and has an updatednextflow_schema.json
to includePR checklist
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).