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

Seg fault on the following file #8

Closed
earonesty opened this issue Aug 13, 2014 · 6 comments
Closed

Seg fault on the following file #8

earonesty opened this issue Aug 13, 2014 · 6 comments

Comments

@earonesty
Copy link

Current version seg-faults on this read.. not sure why

@hwi:1:X:1:2211:11960:84344 3:N:0:
NNATNNTA
+

In order to get the seg fault, you have to compress and then uncompress the file.

@earonesty
Copy link
Author

Not just on it's own... in a combination with other reads... for example this (poor performing) barcode set:

@hwi:1:X:1:2211:11960:84344 3:N:0:
NNATNNTA
+
########
@hwi:1:X:1:2211:11865:84347 3:N:0:
NNAGNNGC
+
########
@hwi:1:X:1:2211:11830:84349 3:N:0:
NNATNNTA
+
########
@hwi:1:X:1:2211:11979:84355 3:N:0:
NNTANNCT
+
########
@hwi:1:X:1:2211:11822:84366 3:N:0:
NNAANNTT
+
########
@hwi:1:X:1:2211:11949:84370 3:N:0:
NNTANNCT
+
########
@hwi:1:X:1:2211:11969:84371 3:N:0:
NNTANNCT
+
########
@hwi:1:X:1:2211:11845:84380 3:N:0:
NNTANNCT
+
########
@hwi:1:X:1:2211:11989:84381 3:N:0:
NNAGNNGC
+
########
@hwi:1:X:1:2211:11790:84386 3:N:0:
NNTANNCT
+
########

@earonesty
Copy link
Author

I think the issue is the ##### quality scores. if you set even a single reads quality higher, it works.

@earonesty
Copy link
Author

In order to get the seg fault, you have to compress and then uncompress the file!

@lrog
Copy link
Collaborator

lrog commented Aug 19, 2014

Thanks for detailed bug report!

The issue was in the special case of the qualities encoding, having only '#' symbols in quality stream and some 'N' symbols in DNA stream.

I committed the fix in 464d5f3 to the quality-fix branch.

The fix passes the case you presented. I run the tests on the selected datasets from Illumina, 454 and ColorSpace and it also passed.

Could you please check if the fix also works in your full dataset? Then I'll merge the fix to master.

@earonesty
Copy link
Author

It does. Yes.

@lrog
Copy link
Collaborator

lrog commented Sep 15, 2014

Fix merged to master. Thanks!

@lrog lrog closed this as completed Sep 15, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants