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

[CODAP] Ammeter sign doesn't match sensor data #1021

Open
Nancy-Salpepi opened this issue Nov 15, 2024 · 1 comment
Open

[CODAP] Ammeter sign doesn't match sensor data #1021

Nancy-Salpepi opened this issue Nov 15, 2024 · 1 comment

Comments

@Nancy-Salpepi
Copy link

Test device
MacBook Air M1 chip

Operating System
15.0.1

Browser
Chrome

Problem description
For phetsims/qa#1170, when the Signed ammeter preference is selected, and then a battery is flipped, only the ammeter attached to the battery will give negative values in CODAP even though all ammeters show a negative value in the sim.

Steps to reproduce

  1. Create a circuit similar to the one in the screenshots
  2. Check all the variables
  3. Press Start Auto Collect
  4. Press Stop Auto Collect after a few seconds
Screenshot 2024-11-15 at 3 30 04 PM
  1. In Preferences, select Signed for Ammeter Readout
  2. Flip the battery
  3. Press Start Auto Collect -- only the ammeter next to the battery will have negative numbers in CODAP even though the sim shows all negative values
Screenshot 2024-11-15 at 3 30 38 PM 8. Flip the battery again -- the negative value column will switch to positive values while the other columns switch to negative values (sim only shows positive values) Screenshot 2024-11-15 at 3 32 04 PM
@kathy-phet
Copy link

@samreid - Please investigate this one ... I suspect the wrapper needs to listen to the same property that the current ammeter is listening too ... or it needs to be adjusted in some way to match.

@samreid samreid removed their assignment Nov 22, 2024
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

3 participants