-
Notifications
You must be signed in to change notification settings - Fork 8
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
Dev-Lite Test: Calculus Grapher 1.0.0-dev.13 #898
Comments
These notes apply to:
Specific Things
Not noticeably worse with tangent checkbox checked.
Not noticeably worse with area under curve checkbox checked.
See notes about lab screen below. General NotesThe performance on the derivative, integral, and advanced screens was fine. The lab screen's performance is slightly worse. The circled line in the below screenshot is a bit "fuzzy". This happens to the curves/lines in this sim a lot on the iPad I was using to test. Integral ScreenWith the area under curve checkbox checked, I've seen some weirdness. If you click buttons A and B, make a tall "high frequency" curve, then press button C, make the curve as vertical as possible, then click button D to move the curve to the top of the screen (or the bottom), then press button C again to mess around with the curve, you start to see some interesting dots: These dots move around as you move the curve, and they're not completely solid; you can see the background color coming through with some of them. Lab ScreenInitially, with no configuration, the performance is noticeably worse than the other screens. However, if you click the button that allows you to create a piecewise function, the performance degrades to the point where the curve being drawn is well behind where your finger is on the screen. |
Testing with the chromebook I had the same experience as Liam, except that I did not have the dots on the Integral screen. On the Lab screen, the performance is significantly worse (mostly with piecewise function) but I would still consider it usable. This chromebook is not touchscreen, and the lag might just be less obtrusive with trackpad input than touch input. |
Thank you QA. Next step is to review these results, and break into separate GitHub issues. @veillette and I will do that when he returns. |
@liammulh's report of "interesting dots" is being tracked in phetsims/calculus-grapher#241. @liammulh's report of lines being "a bit fuzzy" (antialiasing) is being tracked in phetsims/calculus-grapher#242. @liammulh and @stemilymill's performance feedback/concerns are being tracked in phetsims/calculus-grapher#243. I believe that covers all of the QA feeback from above comments. |
In 2/27/2023 Calculus Grapher design meeting, we decided that we need further performance testing with some PhET-iO-only features enabled ( |
@stemilymill @liammulh please do another round of performance testing when you have time. 30-60 minutes should be sufficient. Here are the links for the new dev version:
Again we are interested primarily in evaluating responsiveness of the sim when clicking or click-dragging in the graph labeled f(x). What's different about this test is that we'll be turning on features that are only available via PhET-iO. These features are called "labeled lines" and "labeled points". Here's a screenshot that shows these features made visible using query parameter Please test on the devices that you used previously, or similar devices if they are not available. Please note the device when reporting your results. Specific things to test: (1) Compare the brand=phet version with and without query parmaters (2) Repeat (1) with brand=phet-io version, by clicking the Simulation link in the wrapper index. This runs the standalone PhET-iO version. How does performance compare to brand=phet? Is the sim usable? |
My number one priority right now is fixing Rosetta. And then I have another issue I have to finish before the sprint is over. I probably won't get to this today. |
No problem @liammulh. FYI, our goal is to start QA dev testing by Monday, March 20. So if we can get this performance testing done by end-of-day Tuesday, March 14, that would be great. |
Performance seems improved from dev.13. Even in phet-io standalone and with The only odd thing I did notice was that the labeled points are visible when the plotted curve extends beyond the area of the graph (github is not allowing me to add images again but i will attempt again later) |
In phetsims/calculus-grapher#278, we identified (and addressed) a significant amount of redundant work that was being done while the user is manipulating the curve. The sim is now noticeably snappier and more responsive. So rather than have @liammulh test what is now an old/irrelevant dev versions, we're going to stop this test here, and close this issue. @stemilymill @liammulh thanks for your testing efforts. |
Dev-Lite Test
Mentions: @veillette @amanda-phet @kathy-phet @KatieWoe
Simulation links
Test Matrix
Features included
PhET-iOAlternative InputUI SoundSonificationDescriptionVoicingFocus and Special Instructions
This is intended to be an early/short dev test, since this sim is still being implemented.
The focus of this dev test is responsiveness of the user interface. In each screen, the graph labeled f(x) is interactive. You can click or click-drag in this graph to modify the curve. We are interested in whether these interactions are sufficiently responsive, especially on performance-challenged platforms.
Please choose test platforms as you see fit, but start with performance-challenged platforms. In the Test Matrix section above, indicate the specific devices that you used for testing, including:
In addition to overall subjective impression of responsiveness, here are some specific things to test:
Issues to Verify
None.
For QA...
General features
What to Test
PhET-iO features
What to Test
sure the simulation loads without crashing or throwing errors.
Accessibility features
What to Test
Specific instructions can be found above.
Make sure the accessibility (a11y) feature that is being tested doesn't negatively affect the sim in any way. Here is
a list of features that may be supported in this test:
Test all possible forms of input.
Screen Readers
This sim may support screen readers. If you are unfamiliar with screen readers, please ask Katie to introduce you to
screen readers. If you simply need a refresher on screen readers, please consult the
QA Book, which should have all of the information
you need as well as a link to a screen reader tutorial made by Jesse. Otherwise, look over the a11y view before opening
the simulation. Once you've done that, open the simulation and make sure alerts and descriptions work as intended.
Platforms and Screen Readers to Be Tested
Critical Screen Reader Information
We are tracking known screen reader bugs in
here. If you find a
screen reader bug, please check it against this list.
Keyboard Navigation
This sim supports keyboard navigation. Please make sure it works as intended on all platforms by itself and with a
screen reader.
Magnification
This sim supports magnification with pinch and drag gestures on touch screens, keyboard shortcuts, and mouse/wheel
controls. Please test magnfication and make sure it is working as intended and well with the use cases of the
simulation. Due to the way screen readers handle user input, magnification is NOT expected to work while using a screen
reader so there is no need to test this case.
FAQs for QA Members
There are multiple tests in this issue... Which test should I do first?
Test in order! Test the first thing first, the second thing second, and so on.
How should I format my issue?
Here's a template for making issues:
Who should I assign?
We typically assign the developer who opened the issue in the QA repository.
My question isn't in here... What should I do?
You should:
The text was updated successfully, but these errors were encountered: