Skip to content
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.

Sign up for GitHub

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

Jump to bottom

screed is too tolerant of spaces in FASTQ files #108

Closed
ctb opened this issue Aug 20, 2013 · 3 comments
Closed

screed is too tolerant of spaces in FASTQ files #108

ctb opened this issue Aug 20, 2013 · 3 comments
Labels
screed
Milestone
unscheduled

Comments

@ctb
Copy link
Member

ctb commented Aug 20, 2013

Screed allows blank lines between records in FASTQ, which is apparently against spec.
@mr-c
Copy link
Contributor

mr-c commented Aug 20, 2013

What's wrong with "conservative with what you produce and liberal with what you accept?"

@ctb
Copy link
Member Author

ctb commented Aug 20, 2013

Fine if there's a way to say "be strict." In this case a test succeeded improperly because our screed parser (used in the test) was accepting something that no other downstream program would.

@ctb ctb mentioned this issue Feb 20, 2015
Closed
7 tasks
@ahaerpfer ahaerpfer mentioned this issue Apr 16, 2015
Closed
@ctb
Copy link
Member Author

ctb commented Jun 12, 2015

Closing in favor of discussion on #760.

@ctb ctb closed this as completed Jun 12, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
screed
Projects
None yet
Milestone
unscheduled
Development

No branches or pull requests
3 participants
@ctb @mr-c @bocajnotnef