-
Notifications
You must be signed in to change notification settings - Fork 34
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
Fix two instances of wrong process name passed to ch_versions.mix()
when running in sentieon mode
#454
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.
Nice catch @alkc and thanks for the PR!
Glad to be able to help! Speaking of sentieon, I've identified an issue in the nf-core sentieon dedup module that crashes the rd pipeline when trying to do a sentieon-mode stub run: nf-core/modules#4632 Should I document it as an issue in the raredisease issue tracker? |
It's always nice to get it documented as an issue and very nice to see that you have a PR in modules to solve it. Once that PR is merged we can update the dedup module in the pipeline. |
Done! #462 |
This small update fixes two similar bugs where the wrong process name is passed in a call to
ch_versions.mix()
when run in sentieon mode.PR checklist
nf-core lint
).nextflow run . -profile test,docker --outdir <OUTDIR>
).nextflow run . -profile test_one_sample,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).