-
Notifications
You must be signed in to change notification settings - Fork 2
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
Integrate Ribo-TISH #39
Conversation
|
assets/samplesheet.csv
Outdated
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 has became highly detailed
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'll strip the extraneous columns - thanks!
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.
Can you remove the quotes as well?
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.
Why? Perfectly valid CSV
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.
Looking good, just a bit confused with the aligner params
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
@@ -74,7 +74,7 @@ params { | |||
kallisto_quant_fraglen_sd = 200 | |||
save_merged_fastq = false | |||
save_unaligned = false | |||
save_align_intermeds = false | |||
save_align_intermeds = true |
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 there a need for this as default?
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.
Yeah, I think the alignment files are more of a necessary output for riboseq, while in rnaseq they're only a stop on a path to quantification.
I might reverse it later, but for now it was intentional.
Thanks for the review @maxulysse |
This PR wires in RiboTISH, and adds associated config, documentation etc. I also updated a bunch of modules, which is what makes the number of changed files a bit scary.
This is the first ribo-seq specific functionality, so there's also the wiring from the sample sheet to separate riboseq from rnaseq from TI-seq.
There's still some work to do on wider documentation etc, will do in a separate PR before release.
PR checklist
nf-core lint
).nextflow run . -profile test,docker --outdir <OUTDIR>
).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).