Skip to content

Add a nav-mesh pathfinding behavior for 2D and 3D - #8943

Open
D8H wants to merge 49 commits into
masterfrom
nav-mesh
Open

Add a nav-mesh pathfinding behavior for 2D and 3D#8943
D8H wants to merge 49 commits into
masterfrom
nav-mesh

Conversation

@D8H

@D8H D8H commented Aug 10, 2026

Copy link
Copy Markdown
Collaborator

No description provided.

Comment thread Extensions/NavMeshPathfinding/NavMeshObstacleRuntimeBehavior.ts
Comment thread Extensions/NavMeshPathfinding/NavMeshObstacleRuntimeBehavior.ts
Comment thread Extensions/NavMeshPathfinding/NavMeshObstacleRuntimeBehavior.ts Outdated
Comment thread Extensions/NavMeshPathfinding/NavMeshDebugPixiRenderer.ts Outdated
Comment thread Extensions/NavMeshPathfinding/NavMeshObstacleRuntimeBehavior.ts
Comment thread Extensions/NavMeshPathfinding/NavMeshCharacterRuntimeBehavior.ts
Comment thread Extensions/NavMeshPathfinding/NavMeshObstacleRuntimeBehavior.ts
Comment thread Extensions/NavMeshPathfinding/NavMeshObstacleRuntimeBehavior.ts Outdated
Comment thread Extensions/NavMeshPathfinding/JsExtension.js Outdated
),
'',
'CppPlatform/Extensions/AStaricon.png',
'PathfindingBehavior',

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Wrong name?

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
'PathfindingBehavior',
'NavMeshCharacter',

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is the class name only useful when the properties are defined in C++?
Should we leave it to '' for clarity?

Comment thread Extensions/NavMeshPathfinding/JsExtension.js Outdated
if (!instanceContainer.navMeshObstaclesManager) {
// Create the shared manager if necessary.
const initialData =
instanceContainer.getInitialSharedDataForBehavior('NavMeshCharacter');

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pass behaviorData.name no? I think physics does that.

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The debug action and the obstacle behavior also use this manager. We don't have access to it in this context.
I guess it's safe as long as we don't allow to edit the behavior name in the editor.

Comment thread Extensions/NavMeshPathfinding/NavMeshCharacterRuntimeBehavior.ts Outdated
Comment thread Extensions/NavMeshPathfinding/NavMeshCharacterRuntimeBehavior.ts Outdated
Comment thread Extensions/NavMeshPathfinding/JsExtension.js
Comment thread Extensions/NavMeshPathfinding/JsExtension.js Outdated
Comment thread Extensions/NavMeshPathfinding/NavMeshObstacleRuntimeBehavior.ts Outdated
@D8H
D8H marked this pull request as ready for review August 10, 2026 15:11
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