-
Notifications
You must be signed in to change notification settings - Fork 12
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
Missing data in ldNe #36
Comments
First, is there a typo in above? Do you get the warning message and NULL returned when you have 'drop.missing = FALSE'? Your text says that it works when you have it set to TRUE, but then you say when you set it to TRUE you get the message. If it works when 'drop.missing = TRUE', but you get the warning message when 'drop.missing = FALSE', there is nothing being done wrong. Do you get the same values as NeEstimator when you use 'drop.missing = TRUE'? I would hope so as I believe NeEstimator is dropping missing values. I do not think they impute them somehow. The option in the ldNe() function is there to be explicit that missing data is being dropped. |
Hi Eric, Yes, it's a typo, I'm so sorry. The correct is: When I used "drop.missing = TRUE" the Ne is estimated correctly, same values than NeEstimator v2.1 without missing data. When I used "drop.missing = FALSE", NULL is returned. Do you get the same values as NeEstimator when you use 'drop.missing = TRUE'? "Based on findings from a simulation study (Peel et al. 2013), the software So, I excluded all missing data and reran ldNe (with drop.missing = TRUE) and NeEstimator, and this time, WITHOUT missing, both results were identical. I would hope so as I believe NeEstimator is dropping missing values. I do not think they impute them somehow. The option in the ldNe() function is there to be explicit that missing data is being dropped. The reference is: Thank you again, |
You're right that strataG::ldNe() does not currently implement this correction from Peel et al 2013. When I have some time, I could look into it further and see if it makes sense to do so. |
Thank you so much for your reply! |
Hello Eric,
My name is Jeronymo and I'd like to estimate Ne using strataG similarly to NeEstimator. My dataset shows 6.3% of missing, represent one panmictic population, and when I used the function ldNe:
Ne = ldNe(snps_gtypes, maf.threshold = 0, by.strata = TRUE, ci = 0.95, drop.missing = TRUE, num.cores = 4)
Ne is estimated excluding missing values, same values than NeEstimator v2.1. When I used "drop.missing = TRUE", NULL is returned:
Warning message:
Can't compute ldNe in '1' because loci are missing genotypes and 'drop.missing = FALSE'. NULL returned.
I would like to know how I should proceed in your opinion:
Thank you so much for your time,
Best regards,
Jeronymo Dalapicolla
The text was updated successfully, but these errors were encountered: