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.

Sign up for GitHub

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

Jump to bottom

Change cell rendering #4

Closed
spacemansteve opened this issue Mar 23, 2016 · 5 comments
Closed

Change cell rendering #4

spacemansteve opened this issue Mar 23, 2016 · 5 comments

Comments

@spacemansteve
Copy link
Owner

Consider making individual heatmap cells circular and having them bleed/blend together like this site.
@spacemansteve
Copy link
Owner Author

Leaflet-solr-heatmap is a Leaflet layer. Perhaps it could optionally hand off the rendering to another Leaftlet layer. Currently it supports the rendering types geojsonGrid and clusters. Clusters requires LeafletMarkerCluster. Along those lines, should we add a new type (heatmap?) that uses Leaflet.heat?

@spacemansteve
Copy link
Owner Author

While the above suggestions is workable, it is a bit of a hack. SolrHeatmap extends L.GeoJSON. The suggestion means that in "heatmap" mode, the SolrHeatmap layer wouldn't contain any features (like it does for geojson and cluster) but instead create and manage an instance of Leaflet.heat. Perhaps the Solr specific code should be refactored into a separate object that can be easily used with various kinds of layers?

spacemansteve added a commit that referenced this issue Mar 24, 2016
@spacemansteve
#4 initial bubble based heatmap rendering
a615b6b 
added initial support for type: heatmap
creates an object of Leaflet.heat, which is a bit of a hack
ideally, the solr code would be refactor into another class and support multiple renderers.
colors are quite right due to bug in Leaflet.heat:
http://stackoverflow.com/questions/26767722/leaflet-heat-issue-with-adding-points-with-intensity
causes intermediate colors to be under represented
@spacemansteve
Copy link
Owner Author

One of the better looking heatmaps:
leafletheatmap3

@spacemansteve
Copy link
Owner Author

L.heat has a known issue with rendering intensity: Leaflet/Leaflet.heat#43. There's a modified version floating around that I used for the screenshot below. I will compare it to the L.heat code in github.

leafletheatmapconkling

@spacemansteve
Copy link
Owner Author

With fix to Leaflet.Heat, bubble based heatmaps look good.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@spacemansteve