-
Notifications
You must be signed in to change notification settings - Fork 11
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
annotate mychem.info fda_adverse_event mapping to "biolink:causes_adverse_event" #217
Comments
My understanding is that we try NOT to map to those higher biolink entity classes. I would suggest using PhenotypicFeature (or maybe Disease) rather than DiseaseOrPhenotypicFeature. With the example link above, most things look like PhenotypicFeatures (one exception is Hepatocellular carcinoma). For the operation above, the IDs are names (ChemicalSubstances) and MEDDRA (PhenotypicFeatures?). Perhaps we should use something else for ChemicalSubstances (other drugcentral-based operations are using CHEMBL.COMPOUND ids)? There are MEDDRA mappings for PhenotypicFeature in the biomedical id resolver config file currently....and I added Disease MEDDRA mappings in this PR: biothings/biomedical_id_resolver.js#61 |
note: mapping to PhenotypicFeature MEDDRA rather than Disease....is that okay? To address biothings/biothings_explorer#217
The commit to add |
TDLR: deciding to map MEDDRA IDs to Disease semantic type right now, since
Comparing the xrefs to MEDDRA that we have right now:
|
Making it Biolink Disease for now, since we have more support for mapping to other IDs through MyDisease (see [comment](biothings/biothings_explorer#217 (comment))).
How to find examples to test BTE with:
Some associations that we can expect to retrieve:
|
I didn't do a live test, but these changes look like they address the intent and desired behavior describe in this issue... |
[edit: Fix was to use drugcentral.xrefs.chembl_id (1994 / 2016 of the drugcentral adverse event records also have this field] The operation for Chemical -> Disease will sometimes encounter issues with ID resolving the chemical. There are 2016 records with drugcentral adverse event info, and 1768 of them also have chembl.molecule_chembl_id. This leaves 248 records that don't have the chembl field; however, their _id does seem to be CHEMBL.COMPOUND. For Disease -> Chemical, I can handle this by changing the scope of the mychem query (to have "_id,chembl.molecule_chembl_id"). But for Chemical -> Disease, I can only give one field for the output ID mapping...so I'll do chembl.molecule_chembl_id since more records will have this. |
note: mapping to PhenotypicFeature MEDDRA rather than Disease....is that okay? To address biothings/biothings_explorer#217
Making it Biolink Disease for now, since we have more support for mapping to other IDs through MyDisease (see [comment](biothings/biothings_explorer#217 (comment))).
Should be addressed, since both PRs are merged now. |
mychem reports adverse event data from FAERS (through drug central). For example: https://mychem.info/v1/query?q=drugbank.name:nitisinone&fields=drugcentral.fda_adverse_event
We should add an edge in the mychem openapi file (https://github.com/NCATS-Tangerine/translator-api-registry/blob/master/mychem.info/openapi_full.yml) that links
ChemicalSubstance
toDiseaseOrPhenotypicFeature
using the predicatebiolink:causes_adverse_event
.Prompted by this Translator standup query: NCATSTranslator/testing#82
The text was updated successfully, but these errors were encountered: