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

Adding a parameter for "check_prior" with susie_rss #469

Open
floutt opened this issue Nov 17, 2022 · 2 comments
Open

Adding a parameter for "check_prior" with susie_rss #469

floutt opened this issue Nov 17, 2022 · 2 comments

Comments

@floutt
Copy link
Contributor

floutt commented Nov 17, 2022

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 the susie_rss function to F 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 this

@hsun3163
Copy link
Collaborator

Hmmm I wonder is the LD and the sumstat are matched correctly though? i.e. is the error msg was mistaken.

@hsun3163
Copy link
Collaborator

hsun3163 commented Dec 7, 2022

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
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants