-
Notifications
You must be signed in to change notification settings - Fork 13
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
added new flag --reads_in_memory #33
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 great thank you very much for the addition! I think the if
statement is obsolete, no?
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.
This looks great! Thank you for adding it. While you are at it, could you maybe also add a link to the samtools collate fast documentation to --samtools_collate_fast
in usage docs? We are not really explaining what it actually does in our docs, and since it might have consequences for the output, it wouldn't hurt I think
Feel free to merge this yourself once you are ready :) |
PR checklist
nf-core lint .
).docs
is updatedCHANGELOG.md
is updated