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.

Sign up for GitHub

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

Jump to bottom

Implement Navigation using Recast/Detour #1606

Open
wants to merge 64 commits into
base: master
Choose a base branch
from
Open

Implement Navigation using Recast/Detour #1606

wants to merge 64 commits into from

Conversation

Maxwell175
Copy link
Contributor

Issue description

This PR implements Navigation starting from the work that was done on the gsoc/2020 branch.

Solution description

All commits have been rebased on master.

Checklist

I have done my best to ensure that…

  • …I have familiarized myself with the CONTRIBUTING.md file
  • …this change follows the coding style and design patterns of the codebase
  • …I own the intellectual property rights to this code
  • …the intent of this change is clearly explained
  • …existing uses of the Panda3D API are not broken
  • …the changed code is adequately covered by the test suite, where possible.

@Maxwell175 Maxwell175 mentioned this pull request Jan 25, 2024
Closed
6 tasks
ash1215 and others added 29 commits January 25, 2024 15:10
@ash1215 @Maxwell175
makepanda for recastdetour integration
d504d5c
@ash1215 @Maxwell175
rearranged folders
2b319bc
@ash1215 @Maxwell175
newly arranged classes
e712132
@ash1215 @Maxwell175
added new line at end
c093b53
@ash1215 @Maxwell175
worked on suggestions
d036941
@ash1215 @Maxwell175
Build errors
b943dc8
@ash1215 @Maxwell175
IGATEFILES changes
65b9b02
@ash1215 @Maxwell175
added OPTS before TargetAdd('libp3navigation.dll',opts=OPTS)
d123858
@ashwini-clumio @Maxwell175
Working in python. Tests added
497d41e
@ashwini-clumio @Maxwell175
Added python sample program
a96b08b
@ashwini-clumio @Maxwell175
Divided into 2 libraries
219e203
@ashwini-clumio @Maxwell175
BAM errors
30774a5
@ashwini-clumio @Maxwell175
BAM Serialization done
b803722
@ashwini-clumio @Maxwell175
nav_query accepts nodepath
0e0062a
@ashwini-clumio @Maxwell175
added class NavMeshParams and added visualization in NavMesh Class
710c5af
@ashwini-clumio @Maxwell175
added test files for bam
c246a55
@ashwini-clumio @Maxwell175
fixing sample and completing sample code
5cc8935
@ashwini-clumio @Maxwell175
PR review changes
d6e0769
@ashwini-clumio @Maxwell175
modified test files
d6faf1b
@ashwini-clumio @Maxwell175
added docstrings
162bda7
@ashwini-clumio @Maxwell175
removed BuildSettings, added more functions to set and get settings, …
ddd874d 
…partition to enum
@ashwini-clumio @Maxwell175
Added add_polygon() function
d06031f
@ashwini-clumio @Maxwell175
removed unnecessary comments, added navigation_cat.error() and added …
4cf96ca 
…add_plygon with PTA_LVecBase3f as input
@ashwini-clumio @Maxwell175
added navmeshgen_cat.error() and navmeshgen_cat.info()
5cadc85
@ashwini-clumio @Maxwell175
Removed unused PRC variable ConfigVariableInt
11e235a
@ashwini-clumio @Maxwell175
added MAKEPROPERTY for all properties
012e602
@ashwini-clumio @Maxwell175
added matrix to convert coordinate system
f986cc2
@ashwini-clumio @Maxwell175
renamed RC_MESH_NULL_IDX
caa6468
@ashwini-clumio @Maxwell175
added cull_callback, added doctrings to classes, used nullptr for poi…
18fe03c 
…nters
@Maxwell175
Implement CMake and makepanda build using thirdparty repo.
f9d03bb
@Maxwell175
Prevent test failure when building without navigation.
7126647
@Maxwell175
Add missing sample model.
3adbd91 
[skip ci]
@Maxwell175
Fix weird instability in release mode.
dce85ca
@Maxwell175
Add optional collision mask parameter to filter used collision nodes.
4dd1245
@Maxwell175
Fix comment on cmake Find* modules.
f84c36b
@Maxwell175
Fix skipping tests when navigation/navmeshgen are not built.
ec93c4c
@Maxwell175
Implement filtering Polys.
50312d0
@Maxwell175
Fix segfault when calling NavMeshPoly
b917372
@Maxwell175
Update NavMesh debug draw to show poly borders and allow to color ind…
50563b2 
…ividual polys.
@Maxwell175
Call correct base functions from pvector.
f96f7fa
@Maxwell175
Add optional extents parameter to be able to adjust how far to snap t…
96af77f 
…o the navmesh.
@Maxwell175
Exclude stashed nodes then processing adding a node path to the builder.
2710e9c
@Maxwell175
Extents must be positive. Abs() after transforming to y-up.
3d830e5
@Maxwell175
Implement Tiling and dynamically updating the navmesh during runtime.
78051c3
@Maxwell175
Fix smooth path being constructed incorrectly.
c38584b
@Maxwell175
Refactor to combine navmeshgen into navigation. Clean up API.
80b7077
@Maxwell175
Fix saving and loading bam files.
bf1f923
@Maxwell175
Implement NavMesh Obstacles.
30c4c41
@Maxwell175
Add obstacle debug visualization.
c3b512c
@Maxwell175
Fix debug viz. Properly transform radius and height of Cylinder.
9f3a51c 
Also replaced usages of xform_point_general_in_place with xform_point_in_place.
Don't pass in a shared_ptr to a function that doesn't save the pointer.
@Maxwell175
Cleanups.
7476028
@Maxwell175
Cleanups
a124afc
@Maxwell175
NavObstacleCylinderNode radius and height cannot be negative.
3627145
@Maxwell175
Fix makepanda build
4d77baa
@Maxwell175
Clean up API
56f974f
@Maxwell175
Fix build.
85d2b37
@Maxwell175
Fix indentation.
6ec950f
@Maxwell175
Implement BAM read/write.
b419479
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@Maxwell175 @ash1215 @ashwini-clumio