-
Notifications
You must be signed in to change notification settings - Fork 143
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
Pull request - trying to add BSSNPer #223
Conversation
Adding intermediate step - add readgroups to BAM files - before adding BisSNP
Added new process AFTER markDuplicates to call SNPs Added extra channel in markDuplicates output for new process
Change it eventually
This comment has been minimized.
This comment has been minimized.
hi @AliaMW could i ask about the status of this PR? |
hi @AliaMW the pipeline will have changed much to accommodate these changes. If you are still interested in contributing this process, I would recommend incorporating BSSNPer as an nf-core module and then importing it into the pipeline rather than a local process. I will close this PR for now. Kindly reach out if you need any clarification or help. Thanks |
PR checklist
scrape_software_versions.py
nf-core lint .
).nextflow run . -profile test,docker
).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).