statgen / pheweb

A tool to build a website to browse hundreds or thousands of GWAS.
MIT License
158 stars 65 forks source link

[request] Use new LDServer for LD requests #128

Closed abought closed 4 years ago

abought commented 4 years ago

Purpose

The PortalDev API team is trying to nudge users off the old, deprecated LD endpoint in favor of the new LDServer.

Phewebs are (collectively) a big user of the old endpoint. If you can confirm no technical barriers, I'd be happy to send a pull request updating all PheWeb pages to use LDLZ2. (including the switch LD button, which I'll be adding to LZ core in the next few days)

See comparison of old vs new code: https://github.com/statgen/pheweb/blob/532edd45d943a2aa6a78bcd1ff71f4b33865fbe6/pheweb/serve/static/region.js#L86 vs https://github.com/statgen/locuszoom/blob/3c8344130f06513b06b74acb66c93f8e2c2fcf1d/index.html#L212