phetsims / bending-light

"Bending Light" is an educational simulation in HTML5, by PhET Interactive Simulations.
http://phet.colorado.edu/en/simulation/bending-light
GNU General Public License v3.0
8 stars 8 forks source link

Incorrect wave direction #359

Closed phet-steele closed 7 years ago

phet-steele commented 8 years ago

This is in the published version. Seen on iOS 10.0.2 iPad Air 2. EDIT: the problem may be the retina display. @ariel-phet @samreid this seems kind of important. I think why we have not seen any user reports is because I do NOT see this on iOS 9.3.5, only 10.

Intro and More Tools screen. The reflected light, in wave form, has its lines going the wrong way: img_0051

Blocking the beam with the intensity sensor makes them go the correct way: img_0052

For phetsims/tasks/issues/682.

samreid commented 8 years ago

I've seen related problems like this--first thing to try will be to make the webgl canvas smaller and see if the problem goes away.

phet-steele commented 8 years ago

@samreid I've edited the issue to reflect this, but it is not iOS 10 specific. I checked 9.3.5 on an iPad 2 and did not see the problem, so I assumed it was only a problem for 10. But the issue IS prevalent on 9.3.5 on an iPad Air 2. These means the only commonality between sightings is the iPad model and a retina display. I'm willing to bet it is a problem with the retina display.

samreid commented 8 years ago

Unassigning until I'm devoting time to Bending Light.

phet-steele commented 7 years ago

Kathy's comments from #368:

Note, this bug does not appear in webgl=false version, only webgl version.

@jonathanolson - When there is a lull due to testing or other things you are waiting on, can you take a look at this error since it is only in WebGL code?

It actually seems like it might be something general in WebGL node that should be corrected, and just manifests itself here for some reason.

And a comment from @ariel-phet:

Marking at low priority since this is a "as time allows" bug and has not been reported to phethelp

samreid commented 7 years ago

One more note: it seems like the problem is related to the total size of the canvas, and that reducing the webgl drawable area may make the problem go away. (mentioned in https://github.com/phetsims/bending-light/issues/359#issuecomment-255230720 )

samreid commented 7 years ago

@kathy-phet said:

It actually seems like it might be something general in WebGL node that should be corrected, and just manifests itself here for some reason.

I have seen several occasions where having the geometry "too big" causes rendering artifacts like this. I'm not sure how big is too big, but the problem sometimes (always?) goes away when reducing the geometry size.

samreid commented 7 years ago

This problem occurs on my iPhone 5S running iOS 10.2 but not on my iPad3 running 9.3.5.

samreid commented 7 years ago

This problem was identified for the transmitted wave in https://github.com/phetsims/bending-light/issues/147 but the solution wasn't applied to the reflected wave. I'll apply it to the reflected wave shortly.

samreid commented 7 years ago

I published an RC here which includes the geometry fix and the wave color fix from @jonathanolson: http://www.colorado.edu/physics/phet/dev/html/bending-light/1.1.4-rc.2/bending-light_en.html

I'll create a test task.

samreid commented 7 years ago

Task is mentioned above. On hold until that's complete.

kathy-phet commented 7 years ago

Works great here. Thx, k

Sent from my iPhone

On Feb 9, 2017, at 10:00 PM, Sam Reid notifications@github.com<mailto:notifications@github.com> wrote:

Task is mentioned above. On hold until that's complete.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHubhttps://github.com/phetsims/bending-light/issues/359#issuecomment-278858217, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AE3FZMiRVJW9QrOrcg5qtQftBZPvrPgMks5ra-7zgaJpZM4Kclbu.

samreid commented 7 years ago

Ready for RC test in https://github.com/phetsims/tasks/issues/786

phet-steele commented 7 years ago

Wave direction was correct on (rc.3):

Closing, thanks @samreid!