Skip to content

Updates#19

Open
sq-96 wants to merge 2 commits intocumc:masterfrom
sq-96:master
Open

Updates#19
sq-96 wants to merge 2 commits intocumc:masterfrom
sq-96:master

Conversation

@sq-96
Copy link

@sq-96 sq-96 commented Nov 17, 2020

Changes in summary_statistics_finemapping.ipynb:
s = susie_rss(z_scores, r, r_tol = 5e-3, L = L, min_abs_corr = 0.1, prior_weights=prior, track_fit=${"T" if track_fit else "F"})

Changes in finemapping_results_wrangler.ipynb
unit = dirname(f)
rds_file = dscrutils::dscread(unit, tools::file_path_sans_ext(basename(f)))

" library(susieR)\n",
" L = min(${L}, length(z_scores))\n",
" s = susie_z(z_scores, r, r_tol = 5e-3, L = L, min_abs_corr = 0.1,\n",
" s = susie_rss(z_scores, r, r_tol = 5e-3, L = L, min_abs_corr = 0.1,\n",
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@sq-96 z_ld_weight is a crucial parameter that hsould be included. You probably have to pass in an additional parameter "ref_panel_size" . Otherwise the result may not be reliable as shown in the warning message of your data. See also this section https://stephenslab.github.io/susieR/articles/finemapping_summary_statistics.html#using-ld-from-reference-panel for before and after using the parameter.

gaow pushed a commit that referenced this pull request Oct 25, 2021
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

Successfully merging this pull request may close these issues.

2 participants