Conversation
|
TMKP = text mining knowledge provider, I think |
|
Thanks @saramsey. If TMKP = text mining knowledge provider, it seems like the only "text mining" kp is |
|
Hi @dkoslicki and @saramsey, I think I have resolved the (issue2634)[https://github.com//issues/2634]. These "elevate to treats" rules (please see below) now work for both the old ARAX system (TRAPI 1.5.0) and the new Shepherd system. For CTKP edges,
For DAKP/FAERS edges,
For TMKP edges,
Other |
|
Here is an example: Edge Key ID: It was elevated to Edge Key ID: |
|
Note: Matt relays that:
|
|
Thanks for the update @dkoslicki. I didn't find |
|
@mbrush can you confirm the above? We like to hold these kinds of discussions in the GitHub issues themselves instead of slack DMs, so the record is searchable in the future |
|
I took a look at the EPC edge attributes in the example provided, and they look reasonable. |
|
@chunyuma , three updates to this:
Then we should be good to go! |
|
Thanks for the updates @dkoslicki. As for the new rule to elevate SemMedDB However, according to the new rule, @mbrush or @dkoslicki, could you help clarify which the following one you want:
|
|
For these creative mode queries, the only results we wish to show are those that have been elevated to |
|
Thanks @dkoslicki. I have already completed the three new rules above. |
|
Thanks @chunyuma ! A question about how different KPs are being identified: since you are pulling this information from the edge keys, are we confident that in the new Shepherd framework, these edge keys will remain stable/formatted in this fashion. I.e. do we construct the edge keys at run time from the various edge properties, or are they created elsewhere? I just want to be sure that the logic is tied to something that might change unbeknownst to us |
Hi @dkoslicki, my code is based on the edge keys from "Dev Info" page of https://arax.ncats.io/shepherd/?r=434221. It seems like the new Shepherd framework uses a similar edge id format. Is that the correct place I should refer to? |
|
@chunyuma , we will need to base this logic off the edge attributes, not the edge keys. When looking for a particular |
|
For posterity, this request comes from a bunch of different committees. From Matt:
|
Updated the code for this already @dkoslicki. |
dkoslicki
left a comment
There was a problem hiding this comment.
Awesome, this looks good to go for me!
|
This is good to merge. @edeutsch , can you deploy this to the ARAX Shepherd endpoint? |
|
deployed! |

Hi @dkoslicki,
I have added the conditional features to CTKP and DAKP edges based on the requirements from #2634.
For TMKP edges, since I don't know what is TMKP, I don't know how to process this one. If "TM"KP represents
infores:text-mining-provider-cooccurrence, this kp doesn't have "treats" predicate but only have theoccurs_together_in_literature_withpredicate.