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

gz-sim: buoyancy.sdf #212

Closed
2 tasks done
azeey opened this issue Aug 31, 2023 · 5 comments
Closed
2 tasks done

gz-sim: buoyancy.sdf #212

azeey opened this issue Aug 31, 2023 · 5 comments

Comments

@azeey
Copy link
Contributor

azeey commented Aug 31, 2023

Setup

  • Os: Ubuntu Jammy
  • BuildType: Binary

Links

Checks

  • Instructions work
  • Images (if there are any) match the result

You can find the code used to generate this test case here

@matosinho
Copy link

Hello!

For the default buoyancy.sdf file the example is working fine. Here is a gif:
buoyancy

Although, changing the buoayancy a little the example crashed. For a density of 800, here is what I get after some time:
crash_buoancy

It is a very similar problem from the one I'm having here: #299 (comment)

Maybe is on the damping plugin or control surfaces plugins...

If any additional test is needed, let me know.

@azeey
Copy link
Contributor Author

azeey commented Sep 13, 2023

@mabelzhang or @arjo129 do you know why this might be happening?

@iche033
Copy link

iche033 commented Sep 13, 2023

Although, changing the buoayancy a little the example crashed.

Looking at the error msg from #299 (comment):

 ODE INTERNAL ERROR 1: assertion "aabbBound >= dMinIntExact && aabbBound < dMaxIntExact" failed in collide() [collision_space.cpp:460]

I think the models probably continued to fall / float so their positions end up in really large values, resulting in nan or inf during collision checking.

@arjo129
Copy link

arjo129 commented Sep 13, 2023

There is no damping in this example. As @iche033 said if you keep running it the model will continue to fall/float resulting in NaNs. Not sure what we should do about it. #299 is different. Its a result of an example of a broken plugin interface.

@azeey
Copy link
Contributor Author

azeey commented Sep 14, 2023

Okay, I'll go ahead and close this since the example, as it's written, works fine. Fixing the damping issue is out of scope for this issue. Feel free to create an issue on gz-sim if necessary.

@azeey azeey closed this as completed Sep 14, 2023
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

4 participants