Skip to content

Add mobile-base teleoperation to hangar simulation - #870

Open
picknik-hal-9000-bot[bot] wants to merge 1 commit into
mainfrom
feat/hangar-base-teleop
Open

Add mobile-base teleoperation to hangar simulation#870
picknik-hal-9000-bot[bot] wants to merge 1 commit into
mainfrom
feat/hangar-base-teleop

Conversation

@picknik-hal-9000-bot

Copy link
Copy Markdown

[written by AI]

Motivation

The hangar_sim configuration needs a base-only teleoperation target so operators can jog the Ridgeback base without exposing arm joints. Addresses PickNikRobotics/moveit_pro#21522.

needs: moveit_pro/#21811

Brief description

Adds a distinct mobile_base planning group spanning the planar X, Y, and yaw joints from world to ridgeback_base_link. A dedicated velocity-force controller claims only those base command interfaces, is loaded but inactive at startup, and is mapped to the planar Pose Jog layout introduced by the paired MoveIt Pro PR.

The planning-group and controller configuration—not UI hiding—enforces that selecting mobile_base cannot command arm joints.

How it was tested

  • 43/43 YAML, XML, and cross-file semantic checks passed.
  • Verified the base-only chain, controller resources, acceleration limits, planar Pose Jog mapping, and inactive-at-startup behavior.
  • Applicable pre-commit checks passed with 9 hooks passed, 10 skipped, and 0 failed; Node/Docker hooks were unavailable in the test environment.
  • Robot and simulator launch was intentionally not run during static validation.

Release notes

  • Enhancement: Added planar mobile-base teleoperation to the hangar_sim configuration.

@coderabbitai

coderabbitai Bot commented Aug 20, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 11e168a1-3199-4b80-80f4-0f30d34b2fef

📥 Commits

Reviewing files that changed from the base of the PR and between 205d833 and dcf91ac.

📒 Files selected for processing (4)
  • src/hangar_sim/config/config.yaml
  • src/hangar_sim/config/control/picknik_ur.ros2_control.yaml
  • src/hangar_sim/config/moveit/picknik_ur.srdf
  • src/hangar_sim/config/moveit/pose_jog.yaml

Included review availability: 8 reviews are currently available. Your included PR review attempts over the past 7 days set your current allowance at 10 reviews per hour.


📝 Walkthrough

Summary by CodeRabbit

  • New Features
    • Added mobile-base motion control using velocity and force commands.
    • Added mobile-base support to MoveIt planning and PoseJog controls.
    • Configured planar D-pad controls and motion limits for mobile-base movement.
    • The new controller is available but inactive at startup.

Walkthrough

Changes

The configuration adds a mobile-base velocity-force controller, defines the corresponding MoveIt planning group, and connects both to PoseJog with planar D-pad controls.

Mobile base control

Layer / File(s) Summary
Controller registration and limits
src/hangar_sim/config/config.yaml, src/hangar_sim/config/control/picknik_ur.ros2_control.yaml
Registers and configures mobile_base_velocity_force_controller, including its joints, velocity interface, frames, limits, timeout, and startup state.
MoveIt mobile-base group
src/hangar_sim/config/moveit/picknik_ur.srdf
Adds the mobile_base group from world to ridgeback_base_link.
PoseJog mobile-base integration
src/hangar_sim/config/moveit/pose_jog.yaml
Adds the mobile-base planning group, controller, and planar D-pad layout.

Suggested reviewers: marioprats

Merge Risk: 🔵 Low · up to dcf91

The PR adds a base-only teleoperation controller with a short command timeout. It is mergeable with owner awareness that live activation, command isolation, stop behavior, and timeout scheduling should be verified in the simulator.

🚥 Pre-merge checks | ✅ 4
✅ Passed checks (4 passed)
Check name Status Explanation
Description check ✅ Passed The description clearly explains the mobile-base teleoperation configuration changes and matches the files modified in the pull request.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Human Review Check ✅ Passed The diff only adds mobile-base teleoperation settings in four hangar_sim simulation config files; it does not modify APIs, behaviors, CI/infra, deployment, or launch scripts.
✨ Finishing Touches 💡 1
🛠️ Fix failing CI checks 💡
  • Create stacked PR
  • Commit on current branch

Comment @coderabbitai help to get the list of available commands.

@github-actions

Copy link
Copy Markdown

MoveIt Pro Example WS - Objectives Integration Test Report

  • lab_sim
    • jazzy: no report produced — see run logs
  • hangar_sim
    • jazzy: no report produced — see run logs

@picknik-hal-9000-bot
picknik-hal-9000-bot Bot marked this pull request as ready for review August 20, 2026 23:17
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.

1 participant