-
Notifications
You must be signed in to change notification settings - Fork 10
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
Reads in Peaks Statistic is probably wrong #96
Comments
can you create a test data set where you know the percentage, and see how it performs? |
Has this been fixed? I would need to know for the discussion of the experiment with our collaborators. |
Hi @messersc , Sorry, but this is not fixed yet. I can have a closer look at this next week, but I need to create a controlled test set first. How does your distribution look like, do you have get some bars? For the case your discussion is soon, please use this code to get some rough values:
Best, |
Hi Alex, wow, you're super responsive 👍 I just needed to know if we can rely on these numbers or not. I will try to run your code, maybe I can contribute a bit to find the bug. Thanks for your help and hope you have a nice weekend. |
@messersc I figured out the bug that caused this issue. It had to do with some default settings in summarizeOverlaps that were messing with our counts. |
I have the feeling that there is something wrong with this.
The text was updated successfully, but these errors were encountered: