-
Notifications
You must be signed in to change notification settings - Fork 34
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
Merge intermine-bio package into intermine-ws-python package #12
Comments
I've added the hacktoberfest label to this issue. Note it's probably better to get started by trying some of the other hacktoberfest issues on this repo before you try this one. |
@yochannah |
@mbasil09 Amazing - please go ahead! 😻 |
@yochannah I'll still try to pinpoint those errors and try to fix them. |
that sounds possible, and also like a really good reason to get it merged back into the main package. Thank you so much for tackling this! 😍 |
@yochannah |
@mbasil09 I can probably find them - where in interminebio are the broken links? |
@yochannah it's in this file in the interminebio repository: |
It happens on running this code (taken from the docs of interminebio): from intermine.webservice import Service |
right, I see! Try taking a look at iodocs (click on flymine) - I suspect it's looking for the gff and fasta endpoints |
@mbasil09 I also have a confession to make - I'm not sure how to install intermine bio to test it myself! It doesn't have install docs I can see (sigh) and best guess was to clone it then run |
I couldn't get it to install either, the only workaround I found till now was to use it from the cloned directory. We are able to import all the methods from the interminebio directory. |
I think that's a good idea - today's literally the day that python 2 retires 🤣 |
We have an
intermine-bio package
. Why?https://github.com/intermine/intermine-ws-bio-python
I don't see a utility in keeping these separate. Just making another dependency. (@justinccdev may disagree?).
Let's merge the functionality into the main intermine package and delete this repo.
What does the bio package do? Here are the docs:
Here are the end points it uses:
TODO
The text was updated successfully, but these errors were encountered: