Skip to content

Dev/obs - #7

Merged
ZGZ713912 merged 8 commits into
mainfrom
dev/obs
Sep 27, 2026
Merged

ZGZ713912 merged 8 commits into
mainfrom
dev/obs

Conversation

@ZGZ713912

@ZGZ713912 ZGZ713912 commented Sep 27, 2026 •

Copy link
Copy Markdown
Member

摘要

  • 扩展 ONNX 模型契约,允许输入和输出的批次维度为 1 或动态维度;特征维度仍须为正整数。运行时按单样本形状 [1, N] 创建张量。
  • 将相对关节位置观测归一化到主值范围,减少跨越角度边界时的跳变。
  • 更新 executor.yaml,加入 WheelLegRlImu,并调整策略频率、观测与动作配置、模型路径及控制高度范围。
  • 新增模型工具环境安装脚本和 ONNX 元数据盖章脚本,用于检查布局、写入元数据并复验模型契约。
  • 更新 CI 负向测试,使其检查 FAIL obs feature shape。

测试

未提供测试运行结果。

Recover the dangling commit de8d983 (detached HEAD, not on any branch):
- executor.yaml: command height range, policy rate, obs size 35, joint order
  and default joint positions aligned with the V1 policy
- onnxruntime_inference.hpp: accept dynamic batch dimension and normalize it
  to 1 for the single-observation deployment path
- check_policy_contract.py: allow batch 1 or dynamic, validate feature dims

Also revert observation.cpp includes to <rmcs_rl/rl_bridge/...>; the absolute
paths are what the bridge target expects and unblock the build.
check_policy_contract.py now labels the check 'obs feature shape' to
allow a dynamic batch dimension; update the negative-test grep so CI
does not fail on the stale 'obs shape' string.
@coderabbitai

coderabbitai Bot commented Sep 27, 2026

Copy link
Copy Markdown

Review in Change Stack →

Navigate logical layers of code changes, visualize relationships, and explore their blast radius.

Note

Currently processing new changes in this PR. This may take a few minutes, please wait...

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Advanced

Run ID: 244f1efb-f4c8-4f8f-a704-74a78d269c52

📥 Commits

Reviewing files that changed from the base of the PR and between 8b72200 and 8f854eb.

📒 Files selected for processing (10)
  • .github/workflows/ci.yml
  • config/executor.yaml
  • include/rmcs_rl/onnxruntime_inference.hpp
  • models/deformable_sps_V2.onnx
  • models/wheel_leg_v1.onnx
  • models/wheel_leg_v2.onnx
  • src/rl_bridge/observation.cpp
  • tool/check_policy_contract.py
  • tool/install_model_tools.sh
  • tool/stamp_model.sh
 ____________________________________________________________
< If you don't finish then you're just busy, not productive. >
 ------------------------------------------------------------
  \
   \   \
        \ /\
        ( )
      .( o ).
✨ Finishing Touches
📝 Generate docstrings
  • Commit to this branch
  • Create a new PR
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR

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

@CodeRabbit summary

@ZGZ713912
ZGZ713912 merged commit 9213204 into main Sep 27, 2026
1 of 2 checks passed
@coderabbitai

coderabbitai Bot commented Sep 27, 2026 •

Copy link
Copy Markdown
✅ Action performed

Pull request is closed.

@ZGZ713912
ZGZ713912 deleted the dev/obs branch September 28, 2026 20:53
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