-
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
update edge provenance info to comply with Translator Standard -- July 1 #208
update edge provenance info to comply with Translator Standard -- July 1 #208
Comments
[updated 7/21 to reflect the discussion in the 7/20 lab call; rearrange to put what we're doing first at the top] Situation B: BTE uses x-bte to get edge, API called counts as an "aggregator". This is the situation for most APIs BTE uses (see Situations A/C for the exceptions).Update notes:
What to do:BTE has to add all the source-related information to the edge attributes array:
A Current-and-Desired example:Current (the source-related attribute objects for an edge):
Desired (comments as //):
Here's some thoughts on how to update provenance. The situations below are based on what API BTE called to get that edge. Important notes to read first:
Situation A: BTE ingests edge from a TRAPI APIcurrently BTE ingests these TRAPI APIs:
What to do:
Situation C: BTE uses x-bte to get the edge, the API we call counts as "primary".This is the situation for APIs from multiomics and text mining provider, since they create knowledge from their analysis of data/publications...and perhaps some external APIs that we bring in. The APIs BTE ingests right now that fit this are:
Other APIs that fit this (but BTE doesn't ingest right now):
What to doBTE has to add all the source-related information to the edge attributes array:
An example:Ideally from clinical risk kp api (the source-related attribute objects for an edge) - doesn't exist right now:
Desired (comments as //):
Additional reference:
|
As a very quick recap of today's discussion, @ariutta will take the lead on modifying the structure of the JSON output in the edge attributes, and @colleenXu will take the lead on updating the SmartAPI records for where most of those values are drawn. There undoubtedly will be other details and edge cases to fix later, but let's start with that... |
as example for source provenance update for situation B/C here: biothings/biothings_explorer#208 (comment)
Quick note that the ARAX results viewer for Translator now has a nice visualization for the edge provenance info. For example, from https://arax.ncats.io/?source=ARS&id=a7af1e97-eae3-430d-b570-4da271ea56c7 |
…res curies also note: 3 apis don't have the hard-coded source field now: - text mining co-occurrence api - text mining targeted association api - drug response kp api also note: one api fits situation C of https://github.com/biothings/BioThings_Explorer_TRAPI/issues/208\#issuecomment-866620512: tcga mutational freq api
@ariutta All APIs with yamls in registry update here are updated to address this issue. Note that 3 APIs don't have the "hard-coded" source field anymore; this is fine - they just won't have the corresponding attribute object in their attributes array. once the 2 multiomics api yamls have their PRs merged / smartapi registry entries updated, they may also not have the "hard-coded" source field anymore |
Note that Provenance situation A may be dealt with, once this PR are merged. I notice that this PR seems to add the BTE provenance object mentioned above and included below:
|
I'm okay with closing this issue for now, and opening it again to deal with Provenance situation C related issues as that comes up... This is going to happen with text mining targeted association soon where the plan is to ingest the edge attributes field from records and preserve its structure... |
The parent ticket is here: NCATSTranslator/TranslatorArchitecture#48
This is an example edge with provenance:
These are the desired edge properties (copied from the parent ticket):
The text was updated successfully, but these errors were encountered: