search

elfalem / Leaflet.curve

A Leaflet plugin for drawing Bézier curves and other complex shapes.
Other
218 stars 52 forks source link

Performance issues with a lot of curves #68

Open benb545 opened 1 year ago

benb545 commented 1 year ago

image

When having a lot of curves animating, by adding/removing them from the map every time, the javascript heap size always increases along with the amount of nodes in the DOM.

I am not sure why, but it's definitely not in how I handle the curves. If I remove all layers in the map, the heap size/amount of nodes does not decrease, it's something in the library and I am unsure why.

mvrozanti commented 2 months ago

Hi @benb545 did you fix this or find an alternative?

benb545 commented 2 months ago

@mvrozanti nope still having it. What are you experiencing?

mvrozanti commented 2 months ago

The same, pretty much. I tried using canvas as renderer to see if it made a difference performance-wise and it crashed, I probably did something wrong. I wouldn't mind some examples on how to set it up. I have to draw 30k curves every 2 seconds or so

benb545 commented 2 months ago

I see. I've tried canvas too with the same issue. I think it's a memory leak with the curves plugin itself, didn't have issues with the map without curves. Might write your own curves plugin, or fix it or write your own map with your own rendering

elfalem commented 2 months ago

@benb545 have you tried adding and removing the curves without animating them? That would help us narrow down the possible cause of the possible memory leak.