-
Notifications
You must be signed in to change notification settings - Fork 4
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
Aliasing of curves #242
Comments
@liammulh Could you please comment on whether you see similar aliasing on other platforms? Or was this specific to iPad? |
I only saw it on the iPad. |
I'll note that the number of points for this display is 300, which could be increased, without impacting performance much. However, since the curve shown above is linear, it may not address the problem, although it may be worth trying. Also increasing the linewidth may reduce the aliasing. |
@veillette do you have an iPad to test the above hypotheses? If not, I can have a go at it. |
@pixelzoom: No I dont. Yes, give it a go. |
Zooming in on the screenshot that @liammulh provided, it's easier to see the aliasing. I reproduced this on my iPad6. |
Adding more points does not improve things. With the default 300 points, I took a screenshot of the sim (macOS + Chrome), then zoomed in on that screenshot in macOS Preview: Here's the same thing with 600 points, pretty much identical: I seems unlikely that a larger |
Using Here's a screenshot from my iPad6: Zooming in on that screenshot in macOS Preview: Compared to zooming in on the screenshot that @liammulh provided: And I confirmed that there's no noticeable difference (visual or performance) on other platforms. Closing. |
In phetsims/qa#898 (comment), @liammulh reported aliasing on iPad. I'm not sure what we can do about this, since any diagonal line will always be somewhat more aliased. And I'm not clear about whether it's specific to iPad. Assigning to @veillette and myself for discussion.
These notes apply to:
...
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.
...
The text was updated successfully, but these errors were encountered: