Skip to content

feat: Allow Kinematic-Kinematic Contact generation to use as trigger in bepu#3111

Draft
Nicogo1705 wants to merge 1 commit intostride3d:masterfrom
Nicogo1705:k-k-Contact
Draft

feat: Allow Kinematic-Kinematic Contact generation to use as trigger in bepu#3111
Nicogo1705 wants to merge 1 commit intostride3d:masterfrom
Nicogo1705:k-k-Contact

Conversation

@Nicogo1705
Copy link
Copy Markdown
Contributor

feat: Allow Kinematic-Kinematic contact generation to be used as trigger using Bepu

Previously, AllowContactGeneration only returned true when at least one of the two collidables was Dynamic. This meant that two Kinematic bodies would never generate contacts, making it impossible to use them as triggers against each other.

This fix adds the bothKinematic case so that contact generation is also allowed when both bodies are Kinematic, enabling Kinematic-Kinematic trigger detection.

Also includes a minor #if DEBUG indentation cleanup in ConfigureContactManifold.

Related Issue

Types of changes

  • Docs change / refactoring / dependency upgrade
  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Checklist

  • My change requires a change to the documentation.
  • I have added tests to cover my changes.
  • All new and existing tests passed.
  • I have built and run the editor to try this change out.

@Eideren
Copy link
Copy Markdown
Collaborator

Eideren commented Apr 5, 2026

Afaik this is WIP given your last message on discord, I'll mark it as such, feel free to change it back when it's ready :)

@Eideren Eideren marked this pull request as draft April 5, 2026 16:53
@Eideren Eideren changed the title Allow Kinematic-Kinematic Contact generation to use as trigger [BEPU] feat: Allow Kinematic-Kinematic Contact generation to use as trigger in bepu Apr 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants