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.

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

Suggestion: Improved error messages for bad fastqs that cause problems #265

Open
morganx opened this issue Apr 6, 2020 · 1 comment
Open
Assignees

Comments

@morganx
Copy link

morganx commented Apr 6, 2020

Hi Torsten,
Thanks for making Nullarbor - we really like it. A small suggestion - we spent a while this week trying to figure out why Nullarbor was quietly dying during assembly without generating any helpful messages in the log files, and eventually figured out it was because one of the hundreds of isolates had a corrupt fastq input. Adding a couple of lines of code to check whether read1 and read2 have the same number of lines and generate a helpful warning if they don't may save users many hours of troubleshooting.

@tseemann tseemann self-assigned this Apr 6, 2020
@tseemann
Copy link
Owner

tseemann commented Apr 6, 2020

Yes some more sanity checking could be added, but i did assume basic QC was done at some point before Nullarbor, but yes things can still go wrong.

egrep 'warn|error' nullarbor.log is a good starting point but I will add more stuff.

Thanks for the feedback!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants