-
Notifications
You must be signed in to change notification settings - Fork 15
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
Lassosum data input requirements #173
Comments
Thanks @lambdr . @danielnachun see above -- it is not straightforward to include lassosum in a way compatible to our current interface of summary stats and LD matrices. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
In the original paper describing lassosum, the package was developed for the scenario that you have the summary statistics and a set of PLINK files. It does not take an LD matrix as input, and the PLINK reading is built into the C++ component of the pipeline.
The method never calculates the whole LD matrix explicitly but considers each variant iteratively in the original C++ code. The approach used is a coordinate descent algorithm to estimate the beta coefficients for LASSO. Here they update beta for variant i in iteration t.
Our current understanding is if you can expand equation 11, the X-tilde terms will reduce to an element/row of the LD matrix. And therefore, it is possible to apply this algorithm to perform LASSO on summary statistics, but that would require development of a new set of C++ functions.
The text was updated successfully, but these errors were encountered: