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

epicR? #71

Open
Bohdan-Khomtchouk opened this issue Mar 5, 2018 · 5 comments
Open

epicR? #71

Bohdan-Khomtchouk opened this issue Mar 5, 2018 · 5 comments

Comments

@Bohdan-Khomtchouk
Copy link

Any plans to create an R/Bioconductor package (e.g., epicR)?

@endrebak
Copy link
Member

endrebak commented Mar 5, 2018

Thanks for the suggestion.

I almost never use R. What would the advantages be?

Since I do not do much R work, I do not think I would be the best person to do it anyways, but I'm open to the idea.

@Bohdan-Khomtchouk
Copy link
Author

Bohdan-Khomtchouk commented Mar 6, 2018

Mostly exposure to a very large and vibrant community called Bioconductor, which I believe would significantly boost epic's usage and popularity. epicR wouldn't be as much work as it may initially seem, due to R's fantastic reticulate package, which is used to great acclaim in R's API to TensorFlow (https://github.com/rstudio/tensorflow), amongst other places.

In my colleagues' experience, SICER has been historically difficult to setup/install and not particularly intuitive to run. Also, I've heard that the SICER algorithm is notoriously bad at overcalling false positives, at least relative to other peak callers like MACS2. I wonder how extensively epic has looked into these known issues.

@endrebak
Copy link
Member

endrebak commented Mar 6, 2018

I did not implement the SICER algorithm that does not use input because in my experience it gives way too many false positives. Of course, as ChIP-Seq is 95% noise, it is hard to avoid this.

For H3K27me3 which I study, SICER seems to be the best one (I have tried thus far).

@endrebak
Copy link
Member

endrebak commented Mar 6, 2018

The fact that SICER was hard to install and run was my main reason for rewriting it :) And also, the fact that it seems to give the best results on our data.

@endrebak
Copy link
Member

endrebak commented Mar 8, 2018

I actually think this is a good idea from a "spreading the word" perspective, but it is not first on my todo-list.

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

No branches or pull requests

2 participants