-
Notifications
You must be signed in to change notification settings - Fork 46
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
Adding New Databases #181
Comments
Hi Elif, Thanks, sounds really great! I see you have write access to this repo. Feel free to merge directly to master. New modules in Alternatively, you can open pull requests, in case you want me to review the code first. About DrugBank: I would suggest to check the legal notes first, the license, we should first know if it's okay to redistribute the data. Otherwise, the https://github.com/saezlab/pypath/blob/master/pypath/inputs/cell.py
https://github.com/saezlab/pypath/blob/master/pypath/inputs/cosmic.py https://github.com/saezlab/pypath/blob/master/pypath/inputs/innatedb.py https://github.com/saezlab/pypath/blob/master/pypath/inputs/protmapper.py A very useful tool is the Inspector of your browser, where on the Network tab you can inspect the request and response headers of each request and by right click copy them as I hope this helps. If you experience any difficulties, just let me know. Best, Denes |
Hi Denes,
We are working on adding new databases in pypath. We have completed one of them, Drugcentral. How would you like to proceed about merging new scripts or updates into repo? Can we open a pull request directly or would you prefer discussing about them in issues at first, like we did before?
For Drugbank, it requires user and password. We tried to implement it by checking similar databases in pypath. But we are missing some points I think, could you help us about this issue? Here is the programmatical access of Drugbank: https://go.drugbank.com/releases/help I guess -L option is about an authentication procedure. We are not sure how to implement these options to the current curl script in pypath.
The text was updated successfully, but these errors were encountered: