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

mosaic simulate: allow small values of '-c' / '-p' #1

Open
tobiasmarschall opened this issue Jan 12, 2018 · 1 comment
Open

mosaic simulate: allow small values of '-c' / '-p' #1

tobiasmarschall opened this issue Jan 12, 2018 · 1 comment

Comments

@tobiasmarschall
Copy link
Collaborator

This currently does not work due to limitations in the boost implementation of the negative binomial distribution. But we need this to test smaller window sizes.

@meiers
Copy link
Member

meiers commented May 22, 2018

Reminder to myself: This does not work because the NB distribution in boost requires integer numbers as dispersion parameter r, hence it cannot be <1 and loses resolution already in the range <5.
Interestingly this is only the case for sampling from an NB, not for calculating the probability density (basically boost has two different NB implementations).
Maybe I can find a proper NB implementation somewhere else.

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