-
Notifications
You must be signed in to change notification settings - Fork 17
Update loadDrugMechDB.py to include treats edges for indication paths #264
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
base: master
Are you sure you want to change the base?
Conversation
|
Looks like this needs a refactor. |
|
Given the "treats"/"ameliorates" edge refactor, I hesitate to approve this PR. I suggest we look at it more before merging it into main. |
|
From the DrugMechDB paper: "DrugMechDB was constructed considering drug-disease indications from the DrugCentral So each drug-disease pair in each path in the source file could be considered an indication, at least as of 2020. |
|
@beasleyjonm, how about we use the biolink:treats_or_applied_or_studied_to_treat for now until we can dig in to this some more? |
Let's do that, since it's not clear that they update this regularly with latest indications on drug central and they state they used the 2020 version. It's possible some of the indications could have changed since then, so "treats_or_applied_or_studied_to_treat" seems the safest option. |
eKathleenCarter
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me!
Each indication path in drugmechdb describes the COP for a drug-disease indication pair, but the "treats" edge is not explicitly included. I edited the parser to introduce these as "biolink:treats" edges.