biothings / pending.api

Set of standalone APIs built with the BioThings SDK for the Translator Project
https://biothings.ncats.io
Apache License 2.0
5 stars 13 forks source link

API Multiomics EHR Risk KP Update #113

Closed GitHubbit closed 8 months ago

GitHubbit commented 1 year ago
GitHubbit commented 1 year ago

Hello! Sorry for confusion. We are actually in the process of updating our TSVs for TCDC's MVP2, so I would request that we temporarily pause the API update until that is resolved.

erikyao commented 1 year ago

@GitHubbit thank you for the information!

GitHubbit commented 1 year ago

@erikyao Hello Yao! Apologies for the confusion. Earlier, I had requested that we pause this update because I thought that we were going to provide new TSVs. We have decided that we need to do a significant amount of work before we are able to generate new TSVs. Given the urgency of TRAPI 1.4 migration requirements, may I ask that you deploy this as before? The TSVs are thus what we previously used, and the only major change is the addition of the source provenance. Please let me know!

erikyao commented 1 year ago

@GitHubbit @colleenXu API updated!

colleenXu commented 1 year ago

@GitHubbit

Heads-up:

(1) BTE probably isn't hooked up to this API properly due to the changes. To get BTE hooked up properly, the currently registered SmartAPI yaml needs updates. I can meet with you if needed to go through some examples and get you started.

The updates that I can think of off the top of my head are...

(2) Once (1) is done, you'll probably also want the TRAPI 1.4 source data stuff working correctly. To support this, similar to the ClinicalTrials KP API, a separate yaml will need to be created (in a branch or fork) with the trapi_sources x-bte-response-mapping...

erikyao commented 1 year ago

@GitHubbit @colleenXu API updated again

colleenXu commented 8 months ago

It looks like the basics of this issue were addressed: