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

feat: DotRecast for Bepu physics navigation and nav mesh generation #2529

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Doprez
Copy link
Contributor

@Doprez Doprez commented Nov 23, 2024

PR Details

  • re-adds the bepu navigation library to the source.
  • Moves the core Dotrecast logic out of the Bepu library for others to use elsewhere.
  • Adds extra configuration to pathfinding options
  • Adds the ability to build multiple nav meshes with different configurations
  • Adds the Collision mask to the build when gathering colliders
  • Lots of clean up for better readability

Related Issue

None.

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.

Remaining work

Remaining changes I want to make for this PR:

  • Dedicated collider tracker (similar to how Bullet sends an event to notify navigation build events)
  • conversions to DotRecast colliders (Will be more useful with dynamic meshes)
  • Dynamic meshes.
  • A couple more separation of concern changes for the current builder

@Eideren Eideren self-assigned this Nov 23, 2024
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.

2 participants