path.simplify memory leak / heap pick / GC

paperjs / paper.js

The Swiss Army Knife of Vector Graphics Scripting – Scriptographer ported to JavaScript and the browser, using HTML5 Canvas. Created by @lehni & @puckey

http://paperjs.org

Other

14.5k stars 1.23k forks source link

path.simplify memory leak / heap pick / GC #750

Closed v1r closed 9 years ago

v1r commented 9 years ago

Hi there,

Thanks for this awesome library!

There is a huge performance issue with the path simplify method when its used with drawImage..

How to reproduce :

create a canvas next to the paperjs canvas
Draw path or multiple paths in the paper canvas
onEach mouseUp
- Simplify the path
- use the drawImage method to copy papercanvas on the other canvas.

Thank you to look at this problem asap. Cheers!

lehni commented 9 years ago

Could you provide a test-case? I don't understand why the use of drawImage should cause the issue. simplify() is a complex operation, so I'm not surprised that it creates lag. Are you sure GC is the cause?

v1r commented 9 years ago

Hi, I figured out the main cause. The lag is caused by an agressive time interval that runs every 1/60 ms.. Thank you for your answer.

lehni commented 9 years ago

I am closing this issue then. Feel free to reopen if you think there's a bug still.