Skip to content

feat: Rework CharacterController based on Bepu's#3195

Open
Eideren wants to merge 1 commit into
stride3d:masterfrom
Eideren:character_controller
Open

feat: Rework CharacterController based on Bepu's#3195
Eideren wants to merge 1 commit into
stride3d:masterfrom
Eideren:character_controller

Conversation

@Eideren
Copy link
Copy Markdown
Collaborator

@Eideren Eideren commented May 28, 2026

PR Details

The implementation for bepu's character controller was a bit wonky;

  • The gravity property was mutated internally to prevent the body from sliding down slopes
  • Moving surfaces would not carry the body along with them
  • Moving past a slope would cause the body to fly off instead of continuing horizontally

Related Issue

None logged

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.

N.B.: CharacterControllers.cs, CharacterMotionConstraint.cs and CharacterMotionConstraint.tt are effectively copies from bepu's demo. Mostly left as is to make porting new features and fixes introduced over there easier.

@Eideren
Copy link
Copy Markdown
Collaborator Author

Eideren commented May 28, 2026

@xen2 looks like CI is broken, a bunch of unrelated nuget errors ?

@mihai-ene-public
Copy link
Copy Markdown

hi, since you're doing a rework; could be possible that either in this PR or another to have an Enabled property on Collidables? related to this: #2923
:)

@Eideren
Copy link
Copy Markdown
Collaborator Author

Eideren commented May 28, 2026

There are thousands of things that I want to tackle, and I already don't have time to take care of those. I'm sure you'll figure this out

@xen2
Copy link
Copy Markdown
Member

xen2 commented May 29, 2026

@xen2 looks like CI is broken, a bunch of unrelated nuget errors ?

Sorry about that, on it!

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.

3 participants