-
Notifications
You must be signed in to change notification settings - Fork 5
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
Should the ?theory lines show on the "Interference" screen? #196
Comments
Good question @samreid. I think If you agree, feel free to close @samreid. Otherwise, please tag this issue for further consideration at design meeting. |
That part doesn't bother me at all, for the same reason it is acceptable showing the origin at the center of the barrier on the slits screen (for two slits). I agree this feature will be very helpful to teachers that want to leverage it for the slits screen, but it could be confusing to teachers that want to draw correspondence between the slits screen and the interference screen. That being said, I don't think this query parameter will be used by a large fraction of teachers and it also doesn't seem essential for 1.0. If it was a more important feature, I'd suggest we add a user interface control for it in the simulation, but we decided against that in a design meeting. I estimate it would take a couple of hours to add it to the Interference screen. I won't work on it until we discuss it further. |
@samreid I think we could defer |
Agreed, I'll self-assign for now to document accordingly, then we can decide how best to address for 1.1+. |
Documented accordingly, and deferring until 1.1+ |
In my opinion, the value of this feature would not exceed its cost and I recommend to close without fixing. Assigning for design meeting to discuss. |
I don't think |
@kathy-phet says it would be nice to make sure the empirical result matches the theoretical result, and it could potentially be something to share with teachers as well. @samreid gave a time estimate of 2-3 hours. |
I adapted the existing Testing the max frequency for sound: Testing the min frequency for light: Correspondence seems reasonable. Reassigning to @arouinfar to take a closer look, let me know if there's more to do for this issue. |
@samreid the correspondence in the screenshots you've posted looks fine to me. However, the lines don't appear to shift when the separation changes. I'm also seeing a pretty big delay between changing the frequency and the shift in the |
Ah, got it. Thanks for the clarification.
Since I was fine with limiting |
@kathy-phet can you review the behavior of
@kathy-phet is this acceptable? @samreid also proposed this alternative:
|
@ariel-phet can you please help move this issue forward? |
Discussed at the 6/20/19 design meeting, and we decided that @samreid should proceed with this. |
In the commit, I made it so the theory lines update immediately for water. @arouinfar can you please fully test the |
@samreid In latest (1.0.4) there is good agreement between Recreating the same situation in master, the |
Good discovery @arouinfar, I'll look into it. |
Proposed fix is committed, @arouinfar please review. |
Looking good in master @samreid! |
From #191 and #136, the
?theory
query parameter only shows for the "Slits" screen. Would we like it to show fo the "Interference" screen as well?The text was updated successfully, but these errors were encountered: