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

replacement C++ FAST[AQ] parser #643

Closed
mr-c opened this issue Oct 30, 2014 · 2 comments
Closed

replacement C++ FAST[AQ] parser #643

mr-c opened this issue Oct 30, 2014 · 2 comments

Comments

@mr-c
Copy link
Contributor

mr-c commented Oct 30, 2014

To address #641, #355, #77, and #249 I feel that it is time to look for a new C++ FAST[AQ] parser.

A trial of Seqan is ongoing in PR #642

This discussion was initially offline. I did the mockup to see if the performance was relatively okay and went from there.

Other libraries are either license incompatible (GATB is Affero GPL); not up to the task (Heng Li's code was raw + zlib only, no bzip2); or not oriented for use as a library (SeqDB as suggested by @luizirber)

See also: https://www.biostars.org/p/486/

Seqan already supports SAM/BAM, which will be useful for #523

@ctb
Copy link
Member

ctb commented Oct 30, 2014

Good stuff - thanks :).

@ctb ctb closed this as completed Oct 30, 2014
@ctb
Copy link
Member

ctb commented Oct 30, 2014

(Well, maybe it doesn't need to be closed until the seqan implementation is merged. But I concur with the reasoning and decisions.)

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