statgen / locuszoom

A Javascript/d3 embeddable plugin for interactively visualizing statistical genetic data from customizable sources.
https://statgen.github.io/locuszoom/
MIT License
156 stars 29 forks source link

Recombination Rate Line #55

Closed Frencil closed 8 years ago

Frencil commented 8 years ago

What

This branch utilizes the new recomb data source to implement a basic recombination rate line.

While the line data layer already existed the tool tip positioning logic needed some iteration to behave a bit more intuitively for all lines in general.

Changes

MrFlick commented 8 years ago

I noticed this only works for demo_responsive.html, not demo.html. Should we still keep the two separate should we merge down to one demo?

For the recombination rate label, we should probably include the units "cM/Mb" aka centimorgans per megabase. Otherwise it's less clear what those values actually mean. If that looks too bad, we may have to thing of other ways to label that access. But it's so nice to see that friendly blue line there again.

One other minor thing, the Download SVG button seems to be getting cut off. Screenshot attached.

image

Other than that, things look great!

Frencil commented 8 years ago

Issues addressed:

As for merging the demos, I was thinking that as LocusZoom's capabilities continue to expand we may want the repo to host a variety of demos (e.g. we could host a demo of using LocusZoom for a PheWAS plot right now, but that's not exactly high priority). I could see migrating all demos into a demos directory or some such and renaming / updating them to highlight specific features of the library. For now it's not too hard to keep the three in sync with correct data sources.

MrFlick commented 8 years ago

You're right. More demos would probably be better. We just need to ensure that we keep them all in sync.