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

Possible to get floating soccer balls #82

Closed
KatieWoe opened this issue Mar 7, 2022 · 15 comments
Closed

Possible to get floating soccer balls #82

KatieWoe opened this issue Mar 7, 2022 · 15 comments
Assignees
Labels

Comments

@KatieWoe
Copy link

KatieWoe commented Mar 7, 2022

Test device
Dell
Operating System
Win 11
Browser
Chrome
Problem description
For phetsims/qa#781. Seen when using ?speed=.1 but it may be possible, if hard, during normal use.
When a soccer ball is about to land in a spot, dragging another on/off that spot as it lands can cause the positions to overlap, making it look like a ball is floating and/or missing. This requires precise timing and luck, even with the sim slowed down, so I'm not sure how likely it is during normal use.
Steps to reproduce

  1. Use the query parameter ?speed=.1
  2. One the first screen, use the kick 5 button
  3. When one ball lands, drag it to where the next ball will land
  4. As the second ball lands drop the first
  5. It may also happen if you pick up the first ball from the landing spot as the second lands
  6. This may take several tries

Visuals
lastsecgrab
floating

Troubleshooting information:

!!!!! DO NOT EDIT !!!!!
Name: ‪Center and Spread‬
URL: https://phet-dev.colorado.edu/html/center-and-spread/1.0.0-dev.11/phet/center-and-spread_all_phet.html
Version: 1.0.0-dev.11 2022-03-07 01:50:02 UTC
Features missing: applicationcache, applicationcache, touch
Flags: pixelRatioScaling
User Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/98.0.4758.102 Safari/537.36
Language: en-US
Window: 1280x649
Pixel Ratio: 1.5/1
WebGL: WebGL 1.0 (OpenGL ES 2.0 Chromium)
GLSL: WebGL GLSL ES 1.0 (OpenGL ES GLSL ES 1.0 Chromium)
Vendor: WebKit (WebKit WebGL)
Vertex: attribs: 16 varying: 30 uniform: 4096
Texture: size: 16384 imageUnits: 16 (vertex: 16, combined: 32)
Max viewport: 32767x32767
OES_texture_float: true
Dependencies JSON: {}

@KatieWoe KatieWoe added the type:bug Something isn't working label Mar 7, 2022
@samreid samreid self-assigned this Mar 11, 2022
@amanda-phet
Copy link
Contributor

I've wondered if this might happen, but wasn't fast enough to make it happen on my own (and didn't know about the speed qp). I think this is a potentially strange issue if someone is just moving balls around and not trying to make it happen, so it would be helpful to think through a solution sooner rather than later, if possible.

@amanda-phet amanda-phet added this to the 1.1 Publication milestone Apr 1, 2022
@samreid samreid removed their assignment Apr 2, 2022
@KatieWoe
Copy link
Author

KatieWoe commented Apr 6, 2022

As a data point, I managed to reproduce this in phetsims/qa#795 without the speed parameter.
reproduced

@samreid
Copy link
Member

samreid commented Apr 6, 2022

Good discovery! @amanda-phet would you recommend investigation into this for the current milestone?

@amanda-phet
Copy link
Contributor

I think we can keep this on the 1.1 milestone.

@samreid samreid removed this from the Publish 1.1 milestone Apr 21, 2023
@samreid
Copy link
Member

samreid commented Apr 21, 2023

@amanda-phet and @catherinecarter is the right kind of solution to investigate: when a ball is moved, detect if there are any floating soccer balls, and animate them to the ground?

@catherinecarter
Copy link
Contributor

If there's a way to do that, your proposition seems best. We'd need to consider when there is a stack of floating balls like in the 2nd image above (two balls are stacked) and make sure to move the entire stack down.

@samreid
Copy link
Member

samreid commented May 1, 2023

I have seen this bug more and more after recent refactorings.

@samreid
Copy link
Member

samreid commented May 5, 2023

Even after other changes, this is still buggy by the instructions above (drag lower ball away while landed ball animating up)

image

@samreid
Copy link
Member

samreid commented May 5, 2023

Fixed and ready for review. After reviewing and discussing the commits, let's reassign to @KatieWoe for testing.

@samreid
Copy link
Member

samreid commented May 5, 2023

I'm still seeing problems on multikicks.

@samreid
Copy link
Member

samreid commented May 6, 2023

The recent commits seem to help a lot. I also opened a side issue. Probably ready for review again?

@samreid samreid removed their assignment May 6, 2023
@samreid
Copy link
Member

samreid commented May 8, 2023

Today I got a floating soccer ball by dragging one from a stack while a ball was animating up the stack. Maybe also be tested in #188

@matthew-blackman
Copy link
Contributor

There is some buggy behavior here, but I'm no longer able to get floating soccer balls. Continuing this in #188

@samreid
Copy link
Member

samreid commented May 10, 2023

@marlitas can this issue be closed?

@marlitas
Copy link
Contributor

Closing as completed.

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

7 participants