-
Notifications
You must be signed in to change notification settings - Fork 129
Hsmurali atac seq verified #383
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
base: dev
Are you sure you want to change the base?
Conversation
This PR is against the
|
|
@tahirdme Can I get a review? |
|
Hi @hsmurali and thanks for contributing! I don't have very clear what you are doing on this PR would you mind clarifying it. Also, seems that there was a leak on your PR as there are many logs that are included on it. Would you mean cleaning this PR for its review. |
Made changes to ATACSeqQC.R to handle hg38 and handle errors if it is a custom reference.
PR checklist
nf-core lint).nextflow run . -profile test,docker --outdir <OUTDIR>).docs/usage.mdis updated.docs/output.mdis updated.CHANGELOG.mdis updated.README.mdis updated (including new tool citations and authors/contributors).