-
Notifications
You must be signed in to change notification settings - Fork 82
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
DSL2: Starting subworkflow for circularmapper #1076
Conversation
|
This reverts commit 3fac93d.
Add elongated fasta and fai input for circularmapper
TODO:
|
@nf-core-bot fix linting |
❎ Adding the Also: Due to how the |
I attempted to tweak UPDATE: Found the problem. RealignSAMFile implicitly requires an additional input file that is a list of the circularised chromosomes. This is output by CircularGenerator, but was not picked up by the module. Need to update both modules to account for this extra file. In the presence of this file, RealignSAMFile will remove all |
Updated to the new CM modules, and added passing of the elongated chromosome list to RealignSAMFile. New problem: When the extended reference is provided, CircularGenerator is not run, so no elongated chromosome list exists to go into mapping... |
random thought: Check that Circularmapper does what it's supposed to in the absence of the elongated chromosome file |
Added some channel magic to create a file with the circular target info required by RealignSAMFile if an elongated reference is required. |
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 good to me. Other option would be to edit CircularMapper, but I've given up on the development as no time/different work these days.
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.
I don't see anything major
}, | ||
"fasta_circularmapper_elongatedfasta": { | ||
"type": "string", | ||
"description": "Specify an elongated reference FASTA to be used for circularmapper.", |
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.
Is this in addition to or instead of --fasta
?
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.
Ok this is just docs, can be fixed later if it becomes an issue
Closes #1033
Linked to #921
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).