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
Parser interprets empty read as end-of-file #55
Comments
Indeed, that is not good. I'll fix it ASAP. |
Can you try the develop branch? The issue should be fixed. |
That was fast! I'll test it out as soon as I get into work. |
Here I betray my strong CMake bias — what magical invocations must I make to generate a proper |
The magic call is:
|
ahhh, see I was missing the |
I will do that. I was hoping to get a feed back from you that it indeed fixed the problem before generating a new version and upload it to the releases. |
I can confirm that this does, indeed, fix the problem (using Rory's test data). The parser now returns (and RapMap now reports) having observed 5 reads, instead of 0, which is correct. |
OK, the new distribution tarball is ready in the releases(https://github.com/gmarcais/Jellyfish/releases). Thanks for the bug report. |
Fantastic. Thanks, Guillaume! |
Thanks! |
Is it known how far back this bug has existed? |
Hi Guillaume,
@roryk noticed this issue in RapMap (COMBINE-lab/RapMap#19), and has a nice sample dataset to reproduce it. Basically, when a read is empty, but not the actual end of the file, the parser thinks it's done. Is there an easy way to change the parser logic to account for this situation?
--Rob
The text was updated successfully, but these errors were encountered: