opentargets / issues

Issue tracker for Open Targets Platform and Open Targets Genetics Portal
https://platform.opentargets.org https://genetics.opentargets.org
Apache License 2.0
12 stars 2 forks source link

Use major ancestry in PICS fine-mapping #3514

Closed addramir closed 1 month ago

addramir commented 2 months ago

As previously discussed several times the logic/math behind combining multiple ancestries in PICS fine-mapping is incorrect. PICS will benefit from using the same approach as used now in susie - use only major ancestry. In the future we can think about refining the logic, but for now it is better to be as correct as possible.

DSuveges commented 2 months ago

Essentially we need to do LD annotation for the lead variants, as they were coming from a study with 100% Europeans instead of 70%. There's already a method to get the major ancestry from the ld popultation field in the study index. The steps that need to be touched apply the ld_annotate method.