-
Notifications
You must be signed in to change notification settings - Fork 17
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
pxcomp deal with missing data #136
Comments
Indeed, the ratio that should be getting smaller is instead getting larger. Hmmmmmmmm.... |
Ugh. |
I believe this has to do with missing data, which are not supported. When we count the (valid) characters, the row totals get messed up, so we are not even doing a valid chi-square test anymore. This should be fixed with 7457e45. |
Changing this issue to reflect that missing data are not supported. Can either: 1) through out columns with missing data or 2) something smarter (divide each amongst code states)? Check that p4/PAUP do in this case. |
Not sure why, but for a certain data set
pxcomp
is looping forever. The stopping criterion expects a small number, but this value becomes-nan
and so never completes. This alignment is longish (30K sites) and contains a lot of missing data (7%), so I don't yet know what is responsible...The text was updated successfully, but these errors were encountered: