-
Notifications
You must be signed in to change notification settings - Fork 1.4k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
make a round of performance tuning #538
Comments
These benchmarks useless unless you have very low-end device, in 2020 thinking about Microbenchmarks isn't good + there not a lot of developers which can spent their time to improve performance This comment is just my thought, sorry if i'm wrong |
Where it really matters is when animating a high number of things. Even if we can only improve performance by, say, 25%, then that's awesome because it means we can animated 2500 objects instead of 2000 (for example). The more performance, the better. Some people really do care about it, especially if they are making dynamic experiences like first person shooters or similar. But for the vast majority of use cases (for example, any website with an animation here or there) it probably does not matter. It's similar to React vs Backbone.js: sure, React is faster, but honestly for the majority of use cases Backbone.js with string-based Handlebars templating is perfectly fine and achieves the same thing (however you will definitely not make a performant first person shooter with it). |
Yes, you right. I understand you. Thanks for reply. |
Hehe, true! Gotta weigh the values. (I'm planning to make a web based first person shooter later, so besides this I'd also like to compile Tween.js to WebAssembly. More on that later!) |
Would be great to see tweenjs in WebAssembly :) |
You read my mind! I just merged #544, which converts to TypeScript, and so next steps are to compile the TS with AS. |
|
There's things we can optimize, like preventing new variable creations, using
typeof
instead ofinstanceof
, etc.F.e. here's a test that shows
instanceof
being slower thantypeof
:The text was updated successfully, but these errors were encountered: