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

Forces are not equal and opposite #97

Closed
rea-laura opened this issue Dec 20, 2018 · 9 comments
Closed

Forces are not equal and opposite #97

rea-laura opened this issue Dec 20, 2018 · 9 comments
Assignees
Labels

Comments

@rea-laura
Copy link

rea-laura commented Dec 20, 2018

Test Device

Leibniz

Operating System

iOS 12.1

Browser

Safari 12.1

Problem Description

For phetsims/qa#242:
When one mass is moved, the other's force will not update until it is moved as well.

Occurs on all platforms tested so far (iOS 12, MacOS 10.14 Safari and Firefox, Windows 10 chrome)

Steps to Reproduce

  1. open sim
  2. note that at the beginning, forces are equal and opposite.
  3. move one object, or change its charge. The other mass will not change force value. This affects the length of the force arrows, and the position of the figure pulling/pushing.

Visuals

image

@arouinfar
Copy link
Contributor

Yikes, nice find @rea-laura.

Strangely, this bug doesn't appear to be in master, or the other ISLC siblings.

mbarlow12 added a commit to phetsims/inverse-square-law-common that referenced this issue Dec 20, 2018
@mbarlow12
Copy link
Contributor

I had some lines commented out while dealing with the issue for text going off screen. Sorry for the confusion.

@arouinfar
Copy link
Contributor

Since QA is early on in the spot check, would it be better to publish a new rc and test that instead @mbarlow12 @ariel-phet?

@mbarlow12
Copy link
Contributor

I'm investigating just re-releasing the same RC. If that's not ok, I'll bump to RC.8.

@kathy-phet
Copy link

Last time I asked JO about this, he said in general bumping the RC is easiest.

@mbarlow12
Copy link
Contributor

Ok—will do.

@mbarlow12
Copy link
Contributor

@kathy-phet
Copy link

thx

@KatieWoe
Copy link
Contributor

This seems fixed in 1.0.0-rc.8

@ariel-phet ariel-phet removed their assignment Dec 21, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

6 participants