-
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
SmartAPI yaml with x-bte annotation for BioThings FoodData Central #683
Comments
example record https://biothings.ncats.io/fooddata/association/333008-1267
|
Related infores stuff is ready but not deployed yet:
|
Here's the SmartAPI yaml w/ x-bte annotation for BioThings FoodData Central. This yaml is registered in SmartAPI Registry. I haven't made a PR to add this to BTE's regular use (for the config file, Example query
send a POST request to the api-specific endpoint, BioThings FoodData Central only. Like Put this in the request body: It's querying with Beta-sitosterol (aka
You should get a response with this edge (from this data in the BioThings API, based on this operation's example:
Next steps: DONEEDIT: this change has been deployed to all instances of pending BioThings APIs (our instance biothings.ncats.io up to ITRB prod biothings.transltr.io; we are currently using ITRB prod). We will need to make a parser change / use post-processing on the FOODON IDs. Biolink-model folks have told us to use the BioRegistry regex strings for ID-format, and BioRegistry shows that FOODON IDs should be 8-digit numeric. Right now our IDs all start with "FOODON_". We hadn't heard this policy clarification when we were setting up this API and wondering about the ID format. And some small notes:
|
After discussion with Andrew yesterday, I've opened an issue for the next steps. However, it should be fine if these next steps aren't done by the time we add this API to BTE's regular use - we can still go forward with deploying. |
Edited the post above: the FOODON ID edit is done. So this is ready to deploy |
Added to the PR linked above to add this to BTE's regular use (for the config file, API_LIST variable). We'll try to get it into Translator's Lobster release (dev/CI -> Test this Friday) |
Deployed to Prod. |
Information on the creation and deployment of this API is here: biothings/pending.api#66
The text was updated successfully, but these errors were encountered: