-
Notifications
You must be signed in to change notification settings - Fork 0
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
Homogenise tags and better define tag usage #108
Comments
I think for the upcoming technologies makes sense to do |
I would argue that they while their purpose are different, both the samplesheet of the flow cells as well as the workflows are samplesheets. It is just a file that contains samples and sample metadata that is consumed in one way or another. |
Points
Decision
|
If we use StrEnum and go for many tags we can do: class Prerequisite (StrEnum)
CONFIG = auto()
SAMPLESHEET = auto() which is more MERRy (Maintainable, Extendable, Readable and Robust) and pythonic. |
Description
A question on slack brought up a discussion on usage of tags in housekeeper.
The question was whether to add
samplesheet
as a tag for samplesheets used in workflows. However, curretnylsamplesheet
is "reserved" or "limited to"flow-cell
samplesheets. Currently the solution is to add a new tag using:nextflow-samplesheet
.Basically we are creating a new tag consisting of two tags, which to me seems counterintuitive. Ideally these should be two tags.
nextflow
andsamplesheet
. Furthermore, this pattern oftag-tag
seems to exist for files likenextflow-config
.To do
Discuss the design patterns of tags, decide what to do, document and implement decision.
Some other points brought up
Especially with upcoming new technologies in production that possibly also require
samplesheet
´s (pacbio
,ONT
,Sephyr
) and other files we likely need to introduce new tags and retrospectively alterillumina
tags.Example of inefficient tag usage / construction
The
VariantTags
in hermes - althought here might be some additional step involved that I do not fully understand, yet.The text was updated successfully, but these errors were encountered: