-
Notifications
You must be signed in to change notification settings - Fork 3
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
Memory Test for phet brand #36
Comments
First run: Tested using the built_all_debug version from master, SHA was 3a7c86f, common code was whatever was on master today, 07/13/2021.
This seems like it is a reasonable memory growth profile. With @jbphet, we compared the post-fuzz and pre-fuzz snapshots and saw no leakage of sim-specific classes. I will leave this issue open, and let @pixelzoom close at their discretion when doing the code review. |
I agree, looks reasonable. QA will verify again during dev/RC testing, as part of their standard procedures, so I won't spend time re-running tests. Closing. |
Standard code review issue. Created for #35. I'll perform the memory test and put the data here soon.
The text was updated successfully, but these errors were encountered: