Update to v3 monarch api #140
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
includes two sets of x-bte operations, one for disease - pheno and one for gene - disease (causal)
see https://github.com/biothings/biothings_explorer/issues/774\#issuecomment-1923328949 for reasons to use association endpoint now covering all the operations written for old api (that are still available)
BTE does post-processing on the primary_knowledge_source and aggregator_knowledge_source fields of the response and creates the items.sources field, where the info is TRAPI-format. We can then ingest this with the same trapi_sources keyword we use for text-mining/multiomics apis also changed to use monarchinitiative rather than biolink-api as top-level infores minor adjustments to comments
on recheck, these are now available
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
for biothings/biothings_explorer#774 (comment)
don't merge this until PR to post-processing code is deployed to all BTE instances. biothings/api-respone-transform.js#63
And don't remove this branch until override is removed from all BTE instances. biothings/bte-server#16