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

Waterways demo causes additional forces on the simulated body #11

Closed
fire opened this issue Jan 17, 2021 · 3 comments
Closed

Waterways demo causes additional forces on the simulated body #11

fire opened this issue Jan 17, 2021 · 3 comments

Comments

@fire
Copy link
Collaborator

fire commented Jan 17, 2021

Throw a duck from the sky.

E 0:00:25.075   Face3::split_by_plane: Condition "above_count >= 4 && below_count >= 4" is true. Returned: 0
  <C++ Source>  core\math\face3.cpp:83 @ Face3::split_by_plane()

EXPLOSION!

Use https://github.com/fire/WaterwaysDemo/tree/hydro

@fire fire changed the title Waterways demo causes addition forces on the simulate body Waterways demo causes additional forces on the simulated body Jan 17, 2021
@jonri
Copy link
Collaborator

jonri commented Jan 18, 2021

I've reproduced this. For what it's worth, the Face3::split_by_plane warning is a red herring, the Godot source claims this error condition is hit due to a "bug in the algo", but it also happens when a face on your hull is totally coplanar with the water (which won't cause us a problem).

@jonri
Copy link
Collaborator

jonri commented Jan 18, 2021

This is probably another manifestation of #3, although I'd have to do some additional testing to see if the duck had achieved terminal velocity.

@fire
Copy link
Collaborator Author

fire commented Sep 4, 2022

Duplicate of #3.

@fire fire closed this as not planned Won't fix, can't repro, duplicate, stale Sep 4, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants