nf-core / nascent

Nascent Transcription Processing Pipeline
https://nf-co.re/nascent
MIT License
18 stars 10 forks source link

Add in nf-validation for samplesheet #114

Open edmundmiller opened 1 year ago

edmundmiller commented 1 year ago

Description of feature

Follow up to #113 to address the possible foot gun.

edmundmiller commented 1 year ago

Need to validate that #113 is prevented

Caffeinated-Code commented 1 year ago

Hi @Emiller88!

Ran into the same issue.

The nf-core also discusses this issue with the solution here for now: https://github.com/nf-core/chipseq/issues/313

Turns out the naming convention in the sample sheet is throwing issues midway in the pipeline at samtools sort after UMI deduplication. Can we please try to catch this at the check_samplesheet.py level in this fix?