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

Add Blast API #39

Closed
Fohlen opened this issue Oct 28, 2017 · 2 comments
Closed

Add Blast API #39

Fohlen opened this issue Oct 28, 2017 · 2 comments

Comments

@Fohlen
Copy link

Fohlen commented Oct 28, 2017

Hey there. I've ported the Blast API as a rudimentary example here, ncbi-lib.
I've seen it is missing in this great NCBI library so I'd be willing to provide some help and source code to add a more feature-rich version to bionode

What do you guys think?

@thejmazz
Copy link
Member

Hi @Fohlen, have you seen bionode-blast?

Looks like you already encountered some of the inconveniences of the Blast API yourself :)

Perhaps take a look at bionode-blast and see if there are any differences between your approach and ours? (I actually uncompressed JSON instead of parsing XML - forget whether or not there was a real reason for doing that). Perhaps eventually your Stdlib function would just declare params and pass them through to a bionode module?

Cheers!

@Fohlen
Copy link
Author

Fohlen commented Oct 31, 2017

Hi @thejmazz
I probably didn't see the module, because it is not included in the docs yet!
I used XML there because the JSON2 module will create an archive to download, whereas the XML API makes it available directly.

Probably I could enhance the current bionode-blast utility 👍

@Fohlen Fohlen closed this as completed Oct 31, 2017
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