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

Convex decomposition can panic #645

Open
fr33zing opened this issue Jan 31, 2025 · 1 comment
Open

Convex decomposition can panic #645

fr33zing opened this issue Jan 31, 2025 · 1 comment
Labels
A-Collision Relates to the broad phase, narrow phase, colliders, or other collision functionality P-Crash A sudden unexpected crash

Comments

@fr33zing
Copy link

Hi, I have been running into some panics with Collider::convex_decomposition_from_mesh_with_config. There are some settings (like resolution) that just do not agree with certain models, and instead of returning None it panics. I have the builtin monkey model from blender, and I've been able to crash it with a lot of random resolutions like 13. It isn't very reproducible though so I just try random values until it crashes. I ended up making a wrapper function that does the V-HACD in a panic::catch_unwind but I feel this function should at least mention that it can panic.

Thanks!

@fr33zing
Copy link
Author

I just realized this might be the wrong repo to post this in. This is the error:

.../parry3d-0.17.5/src/transformation/convex_hull3/convex_hull.rs:11:29:
called `Result::unwrap()` on an `Err` value: MissingSupportPoint

@Jondolf Jondolf added A-Collision Relates to the broad phase, narrow phase, colliders, or other collision functionality P-Crash A sudden unexpected crash labels Jan 31, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Collision Relates to the broad phase, narrow phase, colliders, or other collision functionality P-Crash A sudden unexpected crash
Projects
None yet
Development

No branches or pull requests

2 participants