-
Notifications
You must be signed in to change notification settings - Fork 42
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
Adding a parameter for "check_prior" with susie_rss #469
Comments
Hmmm I wonder is the LD and the sumstat are matched correctly though? i.e. is the error msg was mistaken. |
Let us keep this check mandatory for the time being until it turns out that it will be too resource intensive for the 1700 LD-Block |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
I noticed that I was getting errors when using the ROSMAP-derived LD matrix on the Bellenguez summary statistics. Whenever I would run it I would get the following error
The estimated prior variance is unreasonably large. This is usually caused by mismatch between the summary statistics and the LD matrix. Please check the input.
In order to get around this I had to edit the Jupyter Notebook and manually change the
check_prior
parameter of thesusie_rss
function toF
to get around this. I think that it would be useful to add a new parameter--check-prior
to the susie_rss fine-mapping pipeline to deal with situations like thisThe text was updated successfully, but these errors were encountered: