Skip to content
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

Add miRTarBase processor #919

Closed
wants to merge 5 commits into from
Closed

Add miRTarBase processor #919

wants to merge 5 commits into from

Conversation

cthoyt
Copy link
Collaborator

@cthoyt cthoyt commented Jul 21, 2019

References #255

This doesn't update the BioPAX processor, but at least it gives a sure-fire set of miRNA-target interactions. Right now it's lacking in the species specificity or the propagation of the experimental conditions. The database even has confidence information associated with each, so this could be used to calculate BELIEF scores.

Question: how should this be tested?

cthoyt added 4 commits July 22, 2019 01:41
References sorgerlab#255 

This doesn't update the BioPAX processor, but at least it gives a sure-fire set of miRNA-target interactions. Right now it's lacking in the species specificity or the propagation of the experimental conditions. The database even has confidence information associated with each, so this could be used to calculate BELIEF scores.
@bgyori
Copy link
Member

bgyori commented Jul 21, 2019

Hey, this is interesting but we already get good interactions via BioPAX from this resource, so I'm not sure this is critical.

@cthoyt
Copy link
Collaborator Author

cthoyt commented Jul 22, 2019

I might have misunderstood but I thought somebody (you or @johnbachman) mentioned that there was room for improvement on the BioPAX import of miRTarBase. Perhaps it was just due to the lack of grounding.

@bgyori
Copy link
Member

bgyori commented Jul 22, 2019

No, they actually come out reliably grounded and name standardized, we haven't noticed any issues with it. Looking back, #255 is perhaps not totally clearly phrased. What I meant there is that INDRA doesn't (didn't) have a standardized way to deal with micro-RNA grounding, not that the BioPAX source produced ungrounded agents.

@cthoyt
Copy link
Collaborator Author

cthoyt commented Jul 22, 2019

I think I was actually remembering our discussion about the WikiPathways and KEGG pathways coming from BioPAX, then I mentioned WikiPathways has some good coverage of miRNA interactions. Sorry for the confusion - #255 is pretty clear (except for its completion conditions).

After we finish up the other two PRs I will check how many miRNAs we're able to get from PathMe with @ddomingof (but he'll be on vacation so I'll just do it and inform him after :p)

@bgyori
Copy link
Member

bgyori commented Jul 30, 2020

Given the new BioPAX implementation in INDRA, I'll take another look at this to see if processing miRTarBase directly has advantages - in many cases we found that going to the source directly is better so I suspect the same might be the case here. More updates coming soon.

@cthoyt cthoyt closed this Apr 18, 2021
@bgyori
Copy link
Member

bgyori commented Apr 18, 2021

I actually never gave up on this one and just a couple of months ago, did another test of using this processor (with some modifications I have on a local branch) and comparing to what we get from the latest state of the BioPAX processor via Pathway Commons. I actually found that the diff was non-trivial and it would have taken much more work to drill down to figure out what differs exactly and why, and which interpretation is better. I think if any specific use case comes up where we have to rely on microRNA target information, we can revisit this and see how it works.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants