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

fix(crash): Update parry2d to 0.13.7 (use rapier fork) for crash fix in QBVH #960

Merged
merged 1 commit into from
Apr 6, 2024

Conversation

MaxCWhitehead
Copy link
Collaborator

I've been frequently hitting a crash / bug in parry2d QBVH involving incremental update. Sometimes it crashes, sometimes is a framerate death spiral / fully locks up.

thread 'main' panicked at /Users/max/.cargo/registry/src/index.crates.io-6f17d22bba15001f/parry2d-0.13.6/src/partitioning/qbvh/update.rs:367:54:
attempt to add with overflow

Appears to be been fixed at dimforge/parry@7df2728 in parry2d 0.13.7. Updated rapier to use a fork with parry2d update as not updated in rapier yet.

@MaxCWhitehead MaxCWhitehead enabled auto-merge April 6, 2024 03:28
@MaxCWhitehead MaxCWhitehead added this pull request to the merge queue Apr 6, 2024
Merged via the queue into fishfolk:main with commit d89e045 Apr 6, 2024
8 checks passed
@MaxCWhitehead MaxCWhitehead deleted the update-parry2d branch April 6, 2024 03:35
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

Successfully merging this pull request may close these issues.

1 participant