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

Weird sphere movement #1

Closed
osrf-migration opened this issue Oct 4, 2018 · 5 comments
Closed

Weird sphere movement #1

osrf-migration opened this issue Oct 4, 2018 · 5 comments
Labels
bug Something isn't working

Comments

@osrf-migration
Copy link

Original report (archived issue) by Louise Poubel (Bitbucket: chapulina, GitHub: chapulina).


Pull request #51 adds an example world which has a sphere on top of a large box representing the ground plane. I'd expect the sphere, which has no link, visual, collision or inertia poses, to sit still on top of the floor, but instead it sways back and forth as if its CoM wasn't centered or its collision shape a smooth sphere:

moving_sphere.gif

I'm ticketing this here because I'm not sure if the issue is coming from ign-physics or the integration.

@osrf-migration
Copy link
Author

Original comment by Michael Grey (Bitbucket: mxgrey, GitHub: mxgrey).


One possibility that jumps to mind is that dartsim by default will mesh-ify primitives when using FCL as a collision detector, because it used to have known issues with primitive shapes. This was fixed as of version 0.4, but the the Xenial repo distributes 0.3 (although Bionic distributes 0.5, so maybe we can start to avoid this problem). If the sphere is being converted into a mesh, then it might be bouncing back and forth between two corners of that mesh.

But I'm just speculating about this being the cause. I'll take a deep dive into this example and see if I can identify the real issue.

@osrf-migration
Copy link
Author

Original comment by Louise Poubel (Bitbucket: chapulina, GitHub: chapulina).


That does look like a meshified sphere. +1 to using the version in Bionic if that solves the issue.

@osrf-migration
Copy link
Author

Original comment by Michael Grey (Bitbucket: mxgrey, GitHub: mxgrey).


A potential fix for this issue can be found here: https://osrf-migration.github.io/ignition-gh-pages/#!/ignitionrobotics/ign-physics/pull-requests/36/

@osrf-migration
Copy link
Author

Original comment by Nate Koenig (Bitbucket: Nathan Koenig).


@chapulina , did the above PR fix this issue for you?

@osrf-migration
Copy link
Author

Original comment by Louise Poubel (Bitbucket: chapulina, GitHub: chapulina).


  • changed state from "new" to "resolved"

Yes, the problem is solved, thanks @mxgrey !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant