-
Notifications
You must be signed in to change notification settings - Fork 49
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
Python Api #25
Comments
It's not broken. You are using SRAdb which requires SRAmetadb.sqlite to be downloaded locally on your machine using A few things that can be done:
|
Thanks @Chokerino for the bug report. I agree its a bug, but the API is not broken. @DaasDaham's findings are indeed correct. We are trying to move away from the @DaasDaham your suggestions look good! Feel free to send a PR and I will be happy to take a look. |
@saketkc Sure, I'll do it |
@saketkc why is there no mention or use case scenario of SRAweb() mode in the docs as python API (in the python API usage section). Is it because web mode isn't complete yet? |
Partially, yes. And partially because I have been slow at updating the documentation. Most part
The rationale for this design was to return the user some results rather than just failing with an error if |
Thanks for the clarification! |
Closed via #31 |
What I Did
The link to Python API is broken. Could you please fix it or list how I can use this library from inside my code. I tried the above code and got this output.
The text was updated successfully, but these errors were encountered: