Leaflet.heat icon indicating copy to clipboard operation
Leaflet.heat copied to clipboard

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

Open Harvinator opened this issue 8 years ago • 10 comments

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

Harvinator avatar Nov 24 '16 00:11 Harvinator

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

The pull request works perfectly here.

rrooij avatar Jan 13 '17 12:01 rrooij

Bumping this, would appreciate some update here.

dchhetri0 avatar Jan 31 '17 17:01 dchhetri0

Any update on this? Need this working !

nakiabrewer avatar Mar 07 '17 06:03 nakiabrewer

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

garciacastano09 avatar Jun 08 '17 21:06 garciacastano09

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.

ajhbh avatar Aug 13 '18 14:08 ajhbh

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

instantaphex avatar Mar 13 '20 16:03 instantaphex

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

tsafs avatar Apr 22 '20 17:04 tsafs

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

adrianmxb avatar Aug 09 '21 19:08 adrianmxb

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

danielbeckera avatar Jan 19 '22 16:01 danielbeckera

@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!

yohanboniface avatar Oct 05 '23 14:10 yohanboniface