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

Performance Issues #130

Closed
mithi opened this issue Jul 20, 2020 · 2 comments · Fixed by #133
Closed

Performance Issues #130

mithi opened this issue Jul 20, 2020 · 2 comments · Fixed by #133
Labels
PRIORITY Do this first

Comments

@mithi
Copy link
Owner

mithi commented Jul 20, 2020

Performance is still not great, largely due to the huge overhead made by the Plotly library

There are some micro-optimizations that we can still do like lazy loading and memoization, using pure components, and preventing needless rerenderings, but these really won't make a difference unless we somehow address the overhead made by Plotly library.

Will need to benchmark it with: #106
and see what we can incorporate

Loading the landing page (Includes plot and animated gif)

Screen Shot 2020-07-21 at 5 48 02 AM

Screen Shot 2020-07-21 at 5 49 32 AM

Screen Shot 2020-07-21 at 5 49 10 AM

Screen Shot 2020-07-21 at 5 48 33 AM

Screen Shot 2020-07-21 at 5 48 50 AM

@mithi
Copy link
Owner Author

mithi commented Jul 20, 2020

Loading forward kinematics page

Screen Shot 2020-07-21 at 5 50 52 AM

Screen Shot 2020-07-21 at 5 51 10 AM

Screen Shot 2020-07-21 at 5 51 22 AM

Screen Shot 2020-07-21 at 5 51 31 AM

Screen Shot 2020-07-21 at 5 51 54 AM

@mithi mithi added the PRIORITY Do this first label Jul 22, 2020
@mithi mithi pinned this issue Jul 22, 2020
@mithi mithi linked a pull request Jul 26, 2020 that will close this issue
@mithi
Copy link
Owner Author

mithi commented Jul 26, 2020

Screen Shot 2020-07-26 at 11 24 57 PM

Screen Shot 2020-07-26 at 11 27 57 PM

@mithi mithi unpinned this issue Aug 5, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
PRIORITY Do this first
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant