Intensity doesn't work correctly about leaflet.heat HOT 8 CLOSED

Hi @danielfox , my pull requests (issue #8 ) shows a relative heat map based on the minimums and maximums of the visible points instead of the zoom. I tested it with random data and looks good for few points

from leaflet.heat.

+1, a relative heatmap would be a huge improvement

from leaflet.heat.

I noticed after updating to the new version, our heat didn't look very good. It's related to this issue. Why was https://github.com/Leaflet/Leaflet.heat/blob/gh-pages/src/HeatLayer.js/#L145 changed? It used to be:

alt = this._latlngs[i].alt || this._latlngs[i][2];
      k = alt || (1 * v);

Why is this._latlngs[i][2] no longer supported?

@mourner, if there is not a pressing reason for it to have been changed, would you accept a PR to put it back? If there was a reason, we'll need to restructure all of our data. Please let me know.

from leaflet.heat.

Why is this._latlngs[i][2] no longer supported?

It's accidental probably. There's a pull for this already: #20
Will merge soon.

from leaflet.heat.

Okay, awesome. Thanks!

from leaflet.heat.

Hi,

At his line, I don't understand why we take the minimum between the cell's value and 1:
https://github.com/Leaflet/Leaflet.heat/blob/gh-pages/src/HeatLayer.js/#L172

Shouldn't it be

Math.min(cell[2], this.options.max)

instead of

Math.min(cell[2], 1)

?

from leaflet.heat.

@fredericbonifas: I think so, too. The change works for me, so here's a minimal pull request.

from leaflet.heat.

@mourner: That was quick 👍

from leaflet.heat.