Skip to content
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

V2: Perform a performance sweep and code review #814

Closed
7 tasks
Rycochet opened this issue Oct 25, 2017 · 1 comment
Closed
7 tasks

V2: Perform a performance sweep and code review #814

Rycochet opened this issue Oct 25, 2017 · 1 comment

Comments

@Rycochet
Copy link
Collaborator

Between getting to beta and marking for a full release, we need to go through and check various specific things to improve performance and reduce memory footprint. Most of these things will require some form of profiling and / or jsperf style tests for speed.

Don't Prematurely Optimise!

  • Temporary Arrays and Objects waste memory.
  • Reducing the size of needed Objects where possible.
  • Loop unrolling.
  • Caching unused Objects while animations are running and they can be re-used / saved from the GC.
  • Public API is not exposing private data, and is exposing useful methods.
  • Public API comments are well formatted for the developer docs.
  • All options and functions have tests that work properly.

...this list is subject to grow and change as ideas occur.

@Rycochet Rycochet added this to the Velocity 2 milestone Oct 25, 2017
@Rycochet
Copy link
Collaborator Author

Closing due to #927

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant