-
Notifications
You must be signed in to change notification settings - Fork 13
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
update pending dgidb API #40
Comments
Before refreshing our pending API, @colleenXu, can you look at whether we can annotate their API for use within BTE directly? https://www.dgidb.org/api |
We could theoretically annotate the interactions endpoint with x-bte annotation, but there are some issues...
|
noting current parser issues:
|
Hi @colleenXu, could you please double check the mapping from DGIdb interaction types to Biolink predicates? Within the DGIdb Feb 2022 interactions.tsv, the Please correct me if I am wrong. |
Never mind. The scores were already parsed, the comments in parser are outdated. |
Hi @colleenXu, w.r.t.
I noticed that in the tsv file, the column
I'll query MyChem with drug names for chembl IDs for case 1 and case 2, to solve the Could you explain more on W.r.t.
I'll swap
|
On the wikidata vs chembl.compound IDsI don't quite remember what was going on, and I can't find an example in the current API. My guess is that every record in the current API has an object.CHEMBL_COMPOUND field, and I found some wikidata IDs in that field for some records... To clarify:
the column-to-field tableI'd like to keep the tsv column names in some cases....
|
Also, my understanding has been that:
The current parser keeps the CHEMBL.COMPOUND prefix on both fields of the object (id + CHEMBL_COMPOUND) right now... (note: I don't know where the convention above comes from. I get the sense that it's a biothings api / our lab thing. BTE used to need prefixes/no-prefixes on certain ID namespaces....but it seems to be doing okay right now...) |
Sorry it's a typo. I mean chembl ids actually. And it's MyChem.
yes, that's what I am going to do.
I can fix this issue as well. |
@colleenXu, and since we are going to remove the mapped predicates, the |
https://biothings.ncats.io/dgidb updated |
@erikyao also, most of the API records seem to not have an association.relation_name field....it would help to give these records some kind of value. Right now BTE cannot get these associations... I see "n/a" and "other/unknown" interaction types in https://dgidb.org/interaction_types so it would help to maybe keep these values. Maybe by having the values |
updated x-bte annotation for dgidb. still have the issues noted above NCATS-Tangerine/translator-api-registry@0307491 |
Revised as suggested |
|
These are the interaction_types in the raw data. I've used 12 unique interaction_type values for x-bte annotations (24 operations total for forward + reverse). NCATS-Tangerine/translator-api-registry@5ac52b7
For the other interaction_types in the data, I write x-bte annotation and commented it out. this is because there were only a few records for the operation...
|
BTE is using https://pending.biothings.io/dgidb quite a bit, and I believe it hasn't been updated since it was made (in 2018/2019?).
We are interested in updating BOTH the parser + data for this API. This API is using the biolink-model predicates in the association.edge-label field....so we may want to remove this or map the dgidb relations to the most recent version of the biolink-model.
The text was updated successfully, but these errors were encountered: