WenheLI / p5.gif

P5.js gif helper
MIT License
38 stars 4 forks source link

Discussion about removing limit of framerate #17

Open NHibiki opened 5 years ago

NHibiki commented 5 years ago

This issue relates to #7.

@WenheLI, Do you think it is necessary to refactor our previous design that to limit the frame rate? (I guess in some specific situations, we need high-framerate gif to be recorded.)

We might think of other ways to prevent too much work load of CPU, like to limit the frame count? Or giving an option of the “unsafe mode” to allow these dangerous code?

WenheLI commented 5 years ago

I think instead of giving a limit on the framerate we should limit the total frames in the RAM to avoid browser crash.

NHibiki commented 5 years ago

@WenheLI Yeah, that's a great idea. 👍