[codex] Add MPPI path-angle critic#195
Merged
Merged
Conversation
rsasaki0109
added a commit
that referenced
this pull request
Jun 17, 2026
rsasaki0109
added a commit
that referenced
this pull request
Jun 17, 2026
rsasaki0109
added a commit
that referenced
this pull request
Jun 17, 2026
rsasaki0109
added a commit
that referenced
this pull request
Jun 17, 2026
rsasaki0109
added a commit
that referenced
this pull request
Jun 17, 2026
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
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.
Summary
path_angle_weight(default0.25)controller_benchmark ... path_angleplus checked-in CSV/Markdown results, and refresh the ESDF report with the new default critic enabledNotes
The rollout kernel computes a local path tangent around the follow point and adds a light yaw-error stage cost. Reverse samples compare against
path_tangent + pi; the existing backward-motion critic still decides whether reverse motion is worth using.A stronger
path_angle_weight=2.0over-selected stationary straight-heading rollouts in the standalone wall-gap smoke. The checked-in default is tuned to0.25, which keeps the straight/corridor smoke cells successful and improves the correctedu_turnK8192 time-to-goal from 44.75 s to 39.9 s in the local benchmark.Validation
source /opt/ros/humble/setup.bash && colcon build --packages-select cuda_mppi_controller --cmake-args -DCMAKE_BUILD_TYPE=Releasesource /opt/ros/humble/setup.bash && source install/setup.bash && ros2 run cuda_mppi_controller parameter_validation_testsource /opt/ros/humble/setup.bash && source install/setup.bash && ros2 run cuda_mppi_controller mppi_gpu_standalone 2048source /opt/ros/humble/setup.bash && source install/setup.bash && ros2 run cuda_mppi_controller controller_benchmark /tmp/mppi_path_angle_u_turn u_turnsource /opt/ros/humble/setup.bash && source install/setup.bash && ros2 run cuda_mppi_controller controller_benchmark /tmp/mppi_path_angle_esdf esdfsource /opt/ros/humble/setup.bash && source install/setup.bash && ros2 run cuda_mppi_controller controller_benchmark /tmp/mppi_path_angle_bench path_anglepython3 -m py_compile scripts/render_cuda_mppi_esdf_benchmark.py python/src/cudarobotics/__init__.pygit diff --checkPython native extension smoke was not rerun locally because
scikit-build-coreandnanobindare not installed in this environment.