search

Leaflet / Leaflet.heat

A tiny, simple and fast heatmap plugin for Leaflet.
http://leaflet.github.io/Leaflet.heat/demo
BSD 2-Clause "Simplified" License
1.55k stars 525 forks source link

#43 Patch to fix intensity/altitude issue (auto max) #78

Open Harvinator opened 7 years ago

Harvinator commented 7 years ago

This patch works by automatically setting the max as it changes on each zoom level as points get combined/separated.

You no longer need to manually set options.max

When calculating the max value it considers all points (even if they are off the screen). This way you always get the same results at a particular zoom level.

Thanks to @spacemansteve for pointing me in the right direction ca1f43026545f7e413c653e376b1fa8bf3a2b6a6

rrooij commented 7 years ago

Any update on this? It would be nice to see it included.

The pull request works perfectly here.

dchhetri0 commented 7 years ago

Bumping this, would appreciate some update here.

nakiabrewer commented 7 years ago

Any update on this? Need this working !

garciacastano09 commented 7 years ago

I strongly suggest a disclaimer in the readme explaining this bug if it is not going to be be covered. This is a major bug

ajhbh commented 6 years ago

This pull request is a must have! I am using it and it is much more intuitive to use the heatmaps when the intensity adjusts based on the current zoom. This really deserves to be merged, even after all this time.

instantaphex commented 4 years ago

It's been almost 3 and 1/2 years. Any plans on merging this in?

tsafs commented 4 years ago

At this point it might be worthwhile seeing if anyone else might want to manage this plugin whilst being active?

adrianmxb commented 3 years ago

Can this PR be approved? Really unintuitive the way it is now.

danielbeckera commented 2 years ago

Please approve this PR, this is a major bug that needs to be fixed.

yohanboniface commented 1 year ago

@mourner do you have a chance to have a quick check ?

I've tested it on uMap and it seems to have more friendly results. The implemented change seems logic to me, but I can't say I master the topic, so a check from you would help! And if it's good for you, I'm up for releasing :)

Thanks in advance, and take care, amigo!