-
Notifications
You must be signed in to change notification settings - Fork 83
DSL2 - Genotyping Input Updates #1126
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
… name coll. + not correct wet-lab-wise)
… same file, but still)
…ol and consistent saving of files.
Current issue, Read Group assignment is done at least in some cases to be consistent across all libraries of a given sample -- but this breaks mpiluep calling which expects each input file to have a unique Read Group Sample ID. will check if it breaks everything to swap to libID read groups (possibly only in the case we are running genotyping on unmerged libs?) 🗡️ RG assigned:
RG not assigned (?):
|
|
Got a working implementation for redeclaring the Read Group ID's. Needs test with multiple ref genomes/multiple libs from same sample. Then on to ANGSD |
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.
Various small-ish changes.
The PR seems contains various changes from the nf-test conversion branch. Merging this in its current state would likely break testing for everything. Please either rebase/revert commits causing merging that history, OR we wait till #1063 is merged.
@TCLamnidis Please take another look -- I have merged in all the changes from dev. There were a few merge conflicts in the modules.config file which were due to naming changes when I generalized the sample merging post mapping to be able to be run separately for damage manipulation. The CI nf-core test failed on MALT due to insufficient memory. Not sure how this has behaved for others. |
Main bulk of this adjustment relates to #1128.
Otherwise, this is a QOL improvement/update for post mapping inputs into genotyping and its interactions with damage manipulation. Ascurrently implemented, this is a bit inflexible and requires the user to have specific parameter combinations for saving damage_manipulation outputs.
goal for improvement:
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).