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

Created OMPL bindings for local_pathfinding using Pybind11 #452

Merged
merged 20 commits into from
Nov 3, 2024

Conversation

SPDonaghy
Copy link
Contributor

Description

In this PR custom python bindings were created for the specific functionalities that the local_pathfinding ros package requires from OMPL.

Some modifications were also made to build and setup scripts to properly set up the environment for the python bindings to be found by ROS and Python and used.

Verification

  • [On Windows] Rebuilt container, ran setup task, ran build task, ran test task, ros2 launch local_pathfinding main_launch.py with no errors
  • [On rpi] Rebuilt container, ran setup task, ran build task, ran test task, ros2 launch local_pathfinding main_launch.py with no errors

@SPDonaghy SPDonaghy self-assigned this Nov 2, 2024
@SPDonaghy SPDonaghy added the path Pathfinding team label Nov 2, 2024
@FireBoyAJ24 FireBoyAJ24 mentioned this pull request Nov 2, 2024
2 tasks
@SPDonaghy SPDonaghy enabled auto-merge (squash) November 3, 2024 01:19
@SPDonaghy
Copy link
Contributor Author

CI passed. Please approve when you're able. Thanks!

@SPDonaghy SPDonaghy merged commit 6c5d7f3 into main Nov 3, 2024
10 checks passed
@SPDonaghy SPDonaghy deleted the SPDonaghy/test-pybind11-simple branch November 3, 2024 06:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
path Pathfinding team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

OMPL Python bindings not properly installed on ARM devices
2 participants