search

nf-core / demultiplex

Demultiplexing pipeline for sequencing data
https://nf-co.re/demultiplex
MIT License
41 stars 36 forks source link

Clean up samplesheet reading #102

Open edmundmiller opened 1 year ago

edmundmiller commented 1 year ago

Description of feature

Follow up to #99

grst commented 1 month ago

@edmundmiller can this be closed? If not can you be more specific what needs to be done?

edmundmiller commented 1 month ago

IDK if it can be closed, I can look more into it if you don't know the status.

More specifically, what needs to be done is using different input schemas for different demultiplexing tools. I think this is also related to #133. We just had a lot of different language for each tool (which is mainly the tool's fault).

grst commented 1 month ago

More specifically, what needs to be done is using different input schemas for different demultiplexing tools.

That has its separate issue though: https://github.com/nf-core/demultiplex/issues/138

I think this is also related to https://github.com/nf-core/demultiplex/pull/133. We just had a lot of different language for each tool (which is mainly the tool's fault).

We already updated the documentation a bit and @atrigila is currently revising the entire docs again, if there's anything you'd like to have considered there, please let her know.