Skip to content

Refactor omni-b and add omni-c hardware - #95

Open
ZGZ713912 wants to merge 86 commits into
dev/robotsfrom
merge/deformable
Open

Refactor omni-b and add omni-c hardware#95
ZGZ713912 wants to merge 86 commits into
dev/robotsfrom
merge/deformable

Conversation

@ZGZ713912

@ZGZ713912 ZGZ713912 commented Jul 22, 2026

Copy link
Copy Markdown
Member

PR 摘要

  • 重构 omni-b 硬件为上下位双板架构,并新增 omni-c 硬件实现、插件注册及完整 bringup 配置。
  • 新增遥控器仲裁、VT13 输入解析、状态监视、CSV 写入和环形缓冲区预览等硬件与工具基础设施。
  • 扩展底盘模式,加入快速/慢速旋转、对齐及无线充电模式,并支持无线充电角度补偿和导航速度控制。
  • 重构云台自动瞄准、双偏航编码器状态机及偏心云台求解器,完善导航与自瞄输入链路。
  • 调整射击、供弹、摩擦轮和热量控制逻辑,支持单次射击、预装载就绪及新的摩擦轮配置。
  • 新增静态扭矩、扫频识别控制器及配套灰盒拟合脚本,支持输出 CSV/YAML/JSON 识别结果。
  • 更新 omni、omni-b、omni-c、sentry 等硬件配置,修正摩擦参数、PID、自动瞄准、火控和无线充电参数。
  • 清理旧 SteeringHero/SteeringInfantry 插件与配置,并更新相关子模块版本。
  • 新增本地/远程上下文发布、自动瞄准调试、开发工作流和远程扫描改进脚本。

@coderabbitai

coderabbitai Bot commented Jul 22, 2026

Copy link
Copy Markdown

Review Change Stack

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: f54b57cf-27ed-4b76-8863-9e61732a31dd

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

Walkthrough

该 PR 更新了底盘、云台、射击和硬件设备控制逻辑,新增遥控仲裁、识别工具、板卡组件及运行脚本,并同步调整 ROS 配置、插件注册、界面状态映射与子模块指针。

Changes

运行工具与识别流程

Layer / File(s) Summary
运行脚本与参数识别工具
.script/*, .gitmodules
新增调试、上下文、tmux 工作流及摩擦轮、重力扭矩、灰盒扫频识别工具;更新容器管理和远程扫描行为。
底盘模式与功率控制
rmcs_ws/src/rmcs_core/src/controller/chassis/*, rmcs_ws/src/rmcs_msgs/include/rmcs_msgs/*
新增底盘模式、导航、无线充电和功率控制路径,并调整攀爬控制。
云台与射击控制
rmcs_ws/src/rmcs_core/src/controller/gimbal/*, rmcs_ws/src/rmcs_core/src/controller/shooting/*
重构双偏航、自动瞄准、IMU 保持、单次射击、拨片状态机和射击统计。
硬件设备与遥控仲裁
rmcs_ws/src/rmcs_core/src/hardware/device/*, rmcs_ws/src/rmcs_utility/include/*
新增 RemoteControlVt13CsvWriter、状态监视和环形缓冲遍历接口,并扩展 DJI 电机与 DR16 状态管理。
板卡与硬件组件整合
rmcs_ws/src/rmcs_core/src/hardware/*
重构多类硬件组件的板卡、CAN、IMU、遥控和状态服务编排,新增 DeformableInfantryOmniC
插件、配置与界面接线
rmcs_ws/src/rmcs_core/plugins.xml, rmcs_ws/src/rmcs_bringup/config/*, rmcs_ws/src/rmcs_core/src/referee/app/ui/*
新增组件注册和多套 bringup 配置,更新自动瞄准界面、底盘模式颜色及硬件参数。

Estimated code review effort: 5 (Critical) | ~120 minutes

Sequence Diagram(s)

sequenceDiagram
  participant Vt13
  participant RemoteControl
  participant HardwareComponent
  participant Controllers
  Vt13->>RemoteControl: 提供有效遥控帧与控制模式
  RemoteControl->>HardwareComponent: 输出统一遥控快照
  HardwareComponent->>Controllers: 更新底盘、云台与射击输入
  Controllers->>HardwareComponent: 输出控制力矩与状态
Loading

Possibly related PRs

Suggested reviewers: creeper5820

Poem

小兔挥耳,控制器醒,
新的板卡跑不停。
云台寻光,底盘转,
CSV 里参数闪。
远控合流,工具齐,
代码月光甜如梨。

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch merge/deformable

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

@ZGZ713912

Copy link
Copy Markdown
Member Author

@coderabbitai summary

@coderabbitai

coderabbitai Bot commented Jul 22, 2026

Copy link
Copy Markdown
✅ Action performed

Summary regeneration triggered.

@ZGZ713912
ZGZ713912 force-pushed the merge/deformable branch 4 times, most recently from 9f23fea to c3fc3ae Compare July 31, 2026 20:09
@ZGZ713912
ZGZ713912 force-pushed the merge/deformable branch 3 times, most recently from c66163c to a359b1c Compare August 3, 2026 22:35
@ZGZ713912

Copy link
Copy Markdown
Member Author

@coderabbital summary

creeper5820 and others added 6 commits August 12, 2026 01:06
- Sync both omni and omni-b to BottomBoard/TopBoard/ImuBoard architecture
- Update board serial filters and DeformableSuspension config in YAMLs
- Remove Vt13/RemoteControl; use Dr16 direct /remote/* outputs
- Clean up unrelated changes from feat/deformable-infantry
ZGZ713912 and others added 29 commits August 12, 2026 01:11
…ected leg lifting

- X/WIRELESS_CHARGING mode now raises the offset-aligned leg and its diagonal
  counterpart, with Q toggling between hardcoded high/low charging poses
- Leg azimuths unified to CCW [0,360): LF=45, LB=135, RB=225, RF=315
- wireless_charging_offset_deg is now a required parameter (no default)
- Offset direction corrected (subtract in chassis follow)
- Offset ownership moved to DeformableChassisModeManager; chassis consumes via getter
- Add WIRELESS_CHARGING to hero chassis fallthrough chain
- Add BLACK UI color for WIRELESS_CHARGING mode in deformable referee UI
- Add referee 0x301 RobotInteractionData
- Add vtm-link 0x310 CustomMsg
- Remove high/low wireless charging posture selection
- Add mouse wheel friction wheel speed adjustment (Ctrl+F)
- Add CAN2 packet CSV logging (0x200/0x141/0x142)
- Tune auto-aim parameters and expose debug_log_yaw_packet config
…p script; remove unused yaw feedback timeouts from gimbal controller and chassis files
- Remove debug_log_* switches and their supercap/wheel/joint/yaw-packet
  logging code from the deformable-infantry-omni hardware and configs
- Drop write-only leftovers (wheel_status_received_, latest_supercap_status_)
  whose only readers were the removed debug logs
- Align omni-c can_receive_callback with omni/-b: reject extended/remote
  CAN frames upfront
- Remove unused getters and a self-assignment no-op from deformable_mode.hpp
- Document the intentionally always-true correction-inverted expression in
  deformable_chassis.cpp
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Todo

Development

Successfully merging this pull request may close these issues.

7 participants