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

parsers: blast #16

Closed
wants to merge 1 commit into from
Closed

parsers: blast #16

wants to merge 1 commit into from

Conversation

@wilzbach
Copy link
Member

@wilzbach wilzbach commented Nov 1, 2014

Hi,

I hacked a BLAST parser together.

It currently reads the whole BLAST file into memory (shouldn't be a problem as they are generally < 1MB).
I would be happy to improve this parser if needed for a better connectivity with other modules :)

Seb
Hi,

I hacked a BLAST parser together.

It currently reads the whole BLAST file into memory (shouldn't be a problem as they are generally < 1MB).
I would be happy to improve this parser if needed for a better connectivity with other modules :)
@wilzbach wilzbach closed this Dec 8, 2014
@doomedramen

This comment has been minimized.

Copy link
Member

@doomedramen doomedramen commented on 684b4f2 Jan 16, 2015

I am in the middle of writing a blast wrapper (and a few other bio libs) if its of use. https://github.com/wookoouk/GeeFuTu/tree/master/lib

This comment has been minimized.

Copy link
Member

@bmpvieira bmpvieira replied Jan 19, 2015

@wookoouk that's great! Would you like to make them streamable/functional and add to bionode? BTW, there's a server/client side streamable FASTA parser with tests at bionode-fasta (I noticed you also started a fasta parser).

This comment has been minimized.

Copy link
Member

@doomedramen doomedramen replied Jan 19, 2015

This comment has been minimized.

Copy link
Member

@bmpvieira bmpvieira replied Jan 19, 2015

Something like this?

This comment has been minimized.

Copy link
Member

@doomedramen doomedramen replied Jan 19, 2015

This comment has been minimized.

Copy link
Member

@bmpvieira bmpvieira replied Jan 20, 2015

Awesome :-D

@bmpvieira bmpvieira added the community label Apr 6, 2017
@bmpvieira bmpvieira self-requested a review Apr 6, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants