-
Notifications
You must be signed in to change notification settings - Fork 13
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
text_mining_targeted_association data and parser update #181
Comments
Hello @edgargaticaCU, The Let me know if I can proceed and release the data to the |
@everaldorodrigo This is a special situation where we don't want to deploy right away to Test/Production. We'll want to follow the schedule set by Translator for the "treats refactor" feature. Right now, they only want this on CI. |
@edgargaticaCU and I noticed an issue where we expected the association.edge_label value to update, aka for there to be records for https://biothings.ci.transltr.io/text_mining_targeted_association/query?q=association.edge_label:%22treats_or_applied_or_studied_to_treat%22. But there aren't any. Instead the records seem to still be using the old label: https://biothings.ci.transltr.io/text_mining_targeted_association/query?q=association.edge_label:%22treats%22 It's not clear if there's an issue with the parser or with the upload/deployment/update of the API |
Update: this update is only deployed on the CI instance (https://biothings.ci.transltr.io/text_mining_targeted_association). When the "treats refactor"/biolink-model-4-update moves to Test, @edgargaticaCU should request that this be deployed on Test. |
API URL: https://biothings.ncats.io/text_mining_targeted_association
Parser GitHub URL: https://github.com/UCDenver-ccp/text_mining_targeted_association
Git branch/commit: Master 969da7c
No. Documents: 1,476,108
Structure of documents is unchanged, but the treats predicate has been replaced with treats_or_applied_or_studied_to_treat per the treats refactor.
The text was updated successfully, but these errors were encountered: