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

Improve interference from "reflection" in single wave light screen for v2 #349

Open
kathy-phet opened this issue Mar 11, 2019 · 20 comments
Open

Comments

@kathy-phet
Copy link

When we just have one source in the first or second screens, the "reflection" or lack of complete absorption that is happening on the far right of the lattice is causing interference to be visible on the light screen. I would like to spend some time investigating options for reducing this for this v2 work (see oscillatory behavior here). Can the model be tweaked so that after the wave has propogated out of the window and to the edge it can no longer propogate backwards? Perhaps consulting with Mike D for additional options on suppressing that backward reflection? (Note - I am connecting this to a reflection of the back of the lattice because it takes a while for the oscillations to appear and at first they are not there).
image

@arouinfar
Copy link
Contributor

@samreid is it possible that increasing the lattice size in #315 could be the culprit here?

The apparent interference pattern in screen on Waves was previously discussed/addressed in #171 and #197. The brightness of the screen has also been extensively discussed in #161. Would we need to revisit/recalibrate the work done in these issues?

@arouinfar arouinfar removed their assignment Mar 11, 2019
@arouinfar
Copy link
Contributor

Sorry for the potato quality, but here's a comparison of dev.61 (top) and rc.2 (bottom).

dev 61-screen
rc 2-screen

@samreid
Copy link
Member

samreid commented Apr 1, 2019

On Thursday, March 21, @kathy-phet reiterated that it would be good to eliminate this interior reflection that is leading to a pattern in the signal. I previously experimented with several approaches to eliminate or reduce reflections at the edge of the barriers, and the best approach we have implemented so far is described in this comment: #197 (comment)

This approach has better behavior than any of the exponential decay function envelopes we tried, but it does not eliminate reflections 100%.

I mentioned to @kathy-phet that as a last resort, we could superimpose the desired result on the screen and graph.

It would be good to touch base on this issue before I spend more time on it.

@samreid samreid removed their assignment Apr 1, 2019
@samreid
Copy link
Member

samreid commented Apr 1, 2019

The brightness of the screen has also been extensively discussed in #161. Would we need to revisit/recalibrate the work done in these issues?

Note we expect to perform some recalibration in #323 (comment)

@samreid
Copy link
Member

samreid commented Apr 4, 2019

In discussion, @arouinfar and I decided to try a hollywood or larger averaging window, only for the waves screen. You would still see this problem on the interference screen if one source is on, but that is a rarer case.

@samreid samreid self-assigned this Apr 4, 2019
@samreid
Copy link
Member

samreid commented Apr 5, 2019

For the screens with a single source, I increased the averaging window for the screen. This smooths out the noise. @arouinfar can you please review?

@arouinfar
Copy link
Contributor

@samreid this looks great! Since the Waves screen has its own averaging window, I think it would be good to also change the brightness calibration for Waves too. With the amplitude at 50%, the brightness is quite low, and would seem closer to the brightness I would expect at 10%.

Amplitude Screen Brightness
100%
50%
10%

@arouinfar arouinfar assigned samreid and unassigned ariel-phet and arouinfar Apr 5, 2019
@samreid
Copy link
Member

samreid commented Apr 5, 2019

If I multiply the overall brightness scaling by a factor of 3, the 50% value seems better:

image

but even 25% seems very dark:

image

Do you think we will need to introduce a nonlinearity for this display?

@samreid samreid assigned arouinfar and unassigned samreid Apr 5, 2019
@samreid
Copy link
Member

samreid commented Apr 5, 2019

@arouinfar and I reviewed this and decided we would like to try a piecewise linear function to get better behavior across the range of intensities.

@samreid samreid self-assigned this Apr 5, 2019
@arouinfar
Copy link
Contributor

@samreid the updated brightness on Waves looks great!

However, the screen is broken elsewhere in the sim. The intensity graph appears undisturbed, though.
image
image
image
image

@arouinfar arouinfar assigned samreid and unassigned arouinfar Apr 8, 2019
@samreid
Copy link
Member

samreid commented Apr 8, 2019

Good catch, thanks! I'll take a look.

@samreid
Copy link
Member

samreid commented Apr 8, 2019

I committed a proposed fix, can you please take a look on phettest?

@arouinfar
Copy link
Contributor

Looks great @samreid! I'll tag for design meeting review.

@arouinfar arouinfar removed their assignment Apr 8, 2019
@samreid
Copy link
Member

samreid commented Apr 11, 2019

Raw notes for archival purposes, in case we come back to this.

80% (default) 0.13
100% 0.21 (oversaturated)
50% => 0.05 (looks good)
40% => 0.036 (this is what 25% should look like)
30% => 0.019 (this is what 10% should look like)

Maybe take sqrt for the display? This idea did not pan out quickly, we investigated linear piecewise instead.

amplitude intensity brightess                            desired brightness
100% 0.2096934692889395 0.9063853228610702 fully bright  0.9
80% 0.13422634397144692 0.5801839634279833 good          0.5
50% 0.05372277438413573 0.23221292666039975 too dim      0.4
20% 0.008937089269640335 0.03862994193652949             0.3
0% 0.000000000000000000 0.00000000000000000 off          0.0

new piecewiselinearfunction([0,0,  0.008, 0.3,  0.05,0.4,  0.134,0.5, 0.2,0.9]  )
a b
0 0
0.00223709 0.4
0.00893709 0.5
0.05372277 0.64
0.13422634 0.8
0.20969347 0.9
1 1

image

@samreid
Copy link
Member

samreid commented Apr 11, 2019

In the design meeting, we agreed this is looking much better. @kathy-phet said she will take a closer look later on.

@kathy-phet asks if it is important to be dimmer on the edges, @ariel-phet says not that important.

@samreid
Copy link
Member

samreid commented Jun 18, 2019

@ariel-phet can you help move this issue forward?

@samreid samreid added the wave-interference:2.0 Issues that will be addressed for the 2.0 release label Jun 18, 2019
@ariel-phet
Copy link

@samreid said proposed version is in Master. Will take a look.

@ariel-phet
Copy link

@samreid @arouinfar I am tempted to go ahead and close this issue. The fix appears to address the areas of concern. I will note that even with the fix the interference can still be seen with a single source that is slightly off center. But really on the interference screen both sources being "on" is the pedagogically useful case.

interference

@arouinfar
Copy link
Contributor

Discussed in 6/20/19 design meeting, and decided that we can defer this issue.

The current behavior is an improvement over 1.0, but we would like to leave this open for the possibility of future investigation/improvements.

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

No branches or pull requests

4 participants