Recast Navigation - Sandbox Fork 1.0 #1
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Added a project for the DLL and the exposed methods
…reate correctly the sln Adding CMAKE and premake files to the plugin source, to be able the create correctly the sln
Exposed some API
Made the built dll go directly into the correct location in the package
Wrapped all the geometry data into a struct
TileMesh building v1
Added NavMeshBuildData to handle memory freeing
Decreased code duplication
Renamed some methods
Tile building paralellization v1
Added a mutex to ensure that two tiles are not added at the same time
Fixed some bugs
Fixed some crash + made the chunkyMesh reused
Added a way to build tiles without chunkyTriMesh
minor change
Added a method to get the border of the tiles
Fixed compilation issue
…ation Added the block marking + usage of a query filter for the path computation
Added some methods to get data for the debug drawing
Fixed the area marking
… to the Unity side Added a context to be able to return the timings from the native side to the Unity side
Storing NavMesh and NavMeshQuery by environment id
Cleaned code and added comments
Added comments
AdrienMichon-Sandbox
requested review from
joelkaltman,
KinachoSand and
SebasFeiSB
psrojasfredini
approved these changes
Sep 19, 2024
erhermida
reviewed
Sep 23, 2024
| /// The world position of the center of the block. | ||
| float center[3]; | ||
| /// The type of the block. | ||
| int area; |
There was a problem hiding this comment.
Might be better to name this type or areaType, having center and area makes me think this is a volume of some kind.
There was a problem hiding this comment.
Ok yes, I'll go for areaType. It corresponds to something called area in the NavMesh
| @@ -0,0 +1,252 @@ | |||
| #include <windows.h> | |||
There was a problem hiding this comment.
I don't see this being stripped in non-windows builds, does this compile in linux and mac?
There was a problem hiding this comment.
Probably not. I will check on Mac on Wednesday when I can use the Mac in the office.
emmarojasfredini
approved these changes
Sep 24, 2024
Made the plugin work when using MacOS
Fixed Linux compilation (code to clean later)
erhermida
approved these changes
Dec 10, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
This is the native code that is used by the Unity plugin in this repo
Dev
This (public) repo is a fork of Recast Navigation.
The C++ code is divided in several projects.
The Visual studio projects can be built using premake, in the same way it is mentionned in the original repository's documentation.
Most of the code was inspired/copied from the RecastDemo project, which is not included in the dll used by the Unity plugin.
There are just a few specifities linked to the Sandbox, like the fact that we sometimes use blocks and they have a size of (1;1;1), but that's all.