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

General performance issues #18

Closed
ariel-phet opened this issue Jan 5, 2015 · 6 comments
Closed

General performance issues #18

ariel-phet opened this issue Jan 5, 2015 · 6 comments
Assignees

Comments

@ariel-phet
Copy link

Our student tester reports:

There are performance issues on all three basic platforms however (Max OSX safarai, Windows Chrome, and iPad). Both OSX and Windows run fairly well but there is a noticeable stutter upon opening the simulation. The iPad runs slower than both OSX and Windows and also contains the same stutter as the other two. Besides the performance issue I would say this sim is looking great.

@andrey-zelenkov
Copy link
Contributor

We made performance optimization at the stage of internal testing (redraw elements only when it's visible, prevent extra model updates). Please сheck the code and if there are any issues or lack about performance improvement let us know. We'll work on it. Thanks in advance!

@jonathanolson
Copy link
Contributor

Noticed a good amount of low-hanging fruit to optimize, mostly related to the period trace shape (and other shapes like arrows).

jonathanolson added a commit to phetsims/scenery-phet that referenced this issue Dec 18, 2015
…hs that are already equivalently pickable because they are duplicated. Improves hit testing for performance (phetsims/pendulum-lab#18)
@jonathanolson
Copy link
Contributor

iPad 3 testing shows 25 FPS with just pendulum moving, 20 FPS with period trace on, 20 FPS with energy graph visible, 18 FPS with both visible. The works (velocity/acceleration arrows, energy graph, ruler, stopwatch running, period timer active showing a period trace, and 2 pendulums) is running at 11 FPS.

@ariel-phet, do you mind testing to see if performance is acceptable? It will be harder to squeeze out much more, but still possible.

@ariel-phet
Copy link
Author

@arouinfar assigning to you to check out performance on iPad - see what you think in something like the lab scenario with both pendula and all options checked

@arouinfar
Copy link
Contributor

iPad 2 performance is pretty good. I had both pendulums going on the Lab screen with the vectors turned on and energy graph open, and everything looked smooth. The frame rate was pretty consistently 20 fps, except when the period timer was used (~13fps). Despite the low frame rates, the sim didn't seem laggy at all. The only noticeable issue was that the time displayed on the period timer seemed to be a passing a bit slowly compared to other platforms, but that was still pretty minor.

I'll take a look at the iPad 3 and report back.

@arouinfar
Copy link
Contributor

Testing on an iPad 3 had similar results. The sim performs pretty well unless "the works" is running as @jonathanolson described. In that case, there's some lag, but I think it's acceptable given the hardware constraints of the iPad 3. I'll go ahead and close this issue.

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

No branches or pull requests

5 participants