-
-
Notifications
You must be signed in to change notification settings - Fork 26
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
Pondering psex calculation #101
Comments
Stacy says to definitely remove that sum parameter and change it to the latter. |
zkamvar
added a commit
that referenced
this issue
Mar 20, 2017
zkamvar
added a commit
that referenced
this issue
Mar 28, 2017
zkamvar
added a commit
that referenced
this issue
Mar 29, 2017
This further addresses #101. When adding tests for the new implementation of psex, I found out that the reduction presented in Parks and Werth was not apparently the same. I'm not quite sure where to go from here, but something's weird.
Merged
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
psex is currently calculated as
After looking at RClone, it's apparent that it's calculated as
Which is basically saying that the probability of each ADDITIONAL clone being reproduced by sexual reproduction. I'm still not sure which is right, but it's clear that I need to take away that
sum()
and replacen_samples_in_mlg
(second argument) withn_samples
orn_mlg
(or perhaps justG
to let the user control it).The text was updated successfully, but these errors were encountered: