-
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
add max research phase
to treatsChembl
edges from mychem.info
#813
Comments
The changes should be live on Dev/CI within 10 min of the linked commit. BTE/Service Provider now create a "biolink:max_research_phase" edge-attribute from the MyChem However, the values returned right now don't match the biolink-model spec for "max research phase"...
If we needed to return values in the biolink-model spec, I imagine we'd need to figure out all current possible values and what they mean, map them to the biolink-model's values, and do a BTE JQ/post-processing step with those mappings. |
|
Mapping:
References
|
I don't think "0.0" actually exists in the data, so let's remove that handling/mapping. I've edited my post above. There's been discussions happening in Slack w/ Chunlei and Dylan. They discovered an issue with the API that they're addressing, but it won't affect the actual values that we're working with (lab Slack links). |
In the above post, I wrote mappings between Chembl's "max phase for indication" values (specific to each drug-indication pair) and the biolink-model's MaxResearchPhaseEnum values. However, I'm not sure on "-1.0", "0.5", and "4.0" - because the actual definition in Chembl doesn't seem to quite match the options. I'm wondering if you have advice/opinions on this. |
I've tested the linked PR biothings/bte_trapi_query_graph_handler#192 and it works as-intended! Example query
Here's the before-after for some edges from chembl-treats operations: max phase for ind = -1.0
Before:
After:
max phase for ind = 0.5
Before:
After:
max phase for ind = 1.0
Before:
After:
max phase for ind = 2.0
Before:
After:
max phase for ind = 3.0 and 4.0 (disease meddra IDs treated as same entity)
Before:
After:
|
Related PRs deployed to Prod. @colleenXu good to close? |
Yeah, let's close this. We'll open a new issue if Matt Brush/others tell us to adjust the mapping. |
Per a request from @mbrush, I'm creating this issue to add a
max research phase
attribute to edges based ontreatsChembl
andtreatsChembl-rev
in the mychem.info openAPI annotation file. This info will be used in this CQS query template. Note that the application of the attribute constraint will occur within the CQS, but we need to make sure that BTE returns the attribute in the response.test query:
example edge returned from https://bte.ci.transltr.io/v1/query:
The fix to this issue would add a new entry under
attributes
formax research phase
. As noted in https://github.com/NCATS-Tangerine/translator-api-registry/blob/biolink-4-update/mychem.info/openapi_full.yml#L1069, this info is available from mychem.info underchembl.drug_indications.max_phase_for_ind
.The text was updated successfully, but these errors were encountered: