-
Notifications
You must be signed in to change notification settings - Fork 721
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 qualimap to pipeline #205
Add qualimap to pipeline #205
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.
Minor comments
I forgot to add the qualimap version, and I realized that it doesn't have a way to do it without output with an error. |
Typically, you just add each tool that is in the pipeline to the You can redirect the output of https://www.brianstorti.com/understanding-shell-script-idiom-redirect/ |
Thanks. The |
Always in |
ok! that was helpful. Now is part of the version file. |
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.
Looking good now 👍 Thanks a bunch!
Added qualimap to the main pipeline, with option to skip it, labeled as low-memory step. Updated the docs and readme mentioning the tools.
Related issue: #202
PR checklist
nextflow run . -profile test,docker
).nf-core lint .
).docs
is updatedCHANGELOG.md
is updatedREADME.md
is updatedLearn more about contributing: https://github.com/nf-core/rnaseq/tree/master/.github/CONTRIBUTING.md