-
Notifications
You must be signed in to change notification settings - Fork 4
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
get ancillary info #43
Comments
GRAS, SIDER, and Drugbank all use their own identifiers and I couldn't find a readily available mapping from Pubchem cids (what I currently have). SIDER does use Pubchem cids in their hyperlinks so I just need to check that a url exists. I can't seem to figure out where I first encountered SIDER (I though it was on the Pubchem record for metformin but it doesn't seem to be there). The most straightforward way that I found to get these data sources/links is from Pubchem. I've gone ahead and created a script to pull the JSON for the Pubchem records. |
Were 1230 unique cids with DrugBank links and 7 with GRAS notices
Have 465 unique cids with SIDER link
There are only 7 compounds from CMAP02/L1000 identified as GRAS. All of them have a BRH |
Factored add_linkout out of add_table_html to generalize adding link HTML
Factored add_linkout out of add_table_html to generalize adding link HTML
Several changes: 1) BRH and Pug-View annotations are merged on CID, then on pert_iname, then grouped/filled by both. This fixes issues where the same pert_iname has multiple CIDs and only one has BRH/Pug-View annotations. 2) Created a common setup_annot function to process both CMAP/L1000. This makes things easier to maintain. 3) Incorated Wikipedia into annotation for #43
Were 1230 unique cids with DrugBank links and 7 with GRAS notices Former-commit-id: 2e83344
Have 465 unique cids with SIDER link Former-commit-id: ebe40e4
Factored add_linkout out of add_table_html to generalize adding link HTML Former-commit-id: 9de631b
Several changes: 1) BRH and Pug-View annotations are merged on CID, then on pert_iname, then grouped/filled by both. This fixes issues where the same pert_iname has multiple CIDs and only one has BRH/Pug-View annotations. 2) Created a common setup_annot function to process both CMAP/L1000. This makes things easier to maintain. 3) Incorated Wikipedia into annotation for #43 Former-commit-id: 6eebdc3
Several changes: 1) BRH and Pug-View annotations are merged on CID, then on pert_iname, then grouped/filled by both. This fixes issues where the same pert_iname has multiple CIDs and only one has BRH/Pug-View annotations. 2) Created a common setup_annot function to process both CMAP/L1000. This makes things easier to maintain. 3) Incorated Wikipedia into annotation for #43 Former-commit-id: 6eebdc3 Former-commit-id: 3db3ac8 Former-commit-id: 40effdb
Go ahead with:
Also see #4 for further discussion
The text was updated successfully, but these errors were encountered: