Skip to content
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

Dotdensity #283

Open
wants to merge 8 commits into
base: master
Choose a base branch
from
Open

Dotdensity #283

wants to merge 8 commits into from

Conversation

wpears
Copy link
Member

@wpears wpears commented Feb 5, 2015

Wrote in a dotdensity renderer for the HMDA data. Runs off voronoi polygons generated from the tract centroids. Decent way to look at the data where data is dense.. Not as much where data is sparse or zoom level is high.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 76.84% when pulling 6606590 on wpears:dotdensity into db7d344 on cfpb:master.

@wpears
Copy link
Member Author

wpears commented Feb 5, 2015

It should be noted that the voronoi approach is obviously wrong near boundaries (see: points in the ocean). These would have to be clipped to a geometry which currently isn't returned by the API (and at that point, tracts could be used instead of voronoi polygons for the dot density input).

@cmc333333
Copy link
Contributor

This sounds really interesting. Mind providing a screen shot?

@wpears
Copy link
Member Author

wpears commented Mar 20, 2015

@sephcoster has been working on incorporating this upstream.
When we demo'd this this at the beginning of February it looked something like this

@cmc333333
Copy link
Contributor

Very nice!

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.

3 participants