-
Notifications
You must be signed in to change notification settings - Fork 404
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
insert check for one line sample sheet #591
Conversation
|
Ok, I think this is ready for review now. Intended behaviour:
|
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.
Nice! Thanks for adding this. Just something small, I would propose renaming to "numberOfLinesInSampleSheet" or maybe at least num
.
Do you think it would be possible to add a check that each header field is one of (patient, sample, status, gender, lane, fastq1, fastq2, bam, bai, table, cram, crai, vcf)? (I think those are all but I would need to double check) |
I was thinking along the same lines. Could we do something like
|
No, I don't like that solution. Because if you then do So any solution has to be conditional on |
I am not sure I follow. The way I understand Anders proposal, it would mean that a header could just not be
. I agree it wouldn't check at all that the combination are proper, i.e. |
But either way we can save this for the next PR :) |
This fixes #589 by checking that sample sheet is at least two lines. If it's only 1 line, the subsequent
splitCsv(header: true)
call will read it as empty and skip all other samplesheet checks. Also - upgraded missingpatient
andsample
cases from log.warn to log.error.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).