-
Notifications
You must be signed in to change notification settings - Fork 59
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
Remove deprecated Perl syntax and parse Fasta/q comment #7
base: master
Are you sure you want to change the base?
Conversation
By definition, FASTA/FASTQ names should never contain spaces. In the header line, anything following a space/tab is comment. It is not part of the sequence name. |
You are correct, I was referring to parsing the comment, which contains the pair information for Illumina 1.8+ data. For example, in the record below,
the pair information is lost by keeping just the name. There are other ways of handling this, such as returning the comment if it exists, but it seems like discarding this information without warning may lead to problems with downstream analyses. |
The name and comment parsing should follow the specifications now (sorry for the multiple commits). After some thought, I agreed with you on the original pull request (that is, the name should not include the comment, and extra parsing should not be required). An extra variable for the comment is now returned. The usage is,
|
Note that the syntax
Version info:
|
I have updated the title (removed the comment about warnings) because the syntax that was generating warnings will no longer compile.