Skip to content

Conversation

@zhangwt20011015
Copy link
Contributor

@zhangwt20011015 zhangwt20011015 commented Jan 26, 2026

adding the teleop-ros2 to github page

Summary by CodeRabbit

  • 文档
    • 在开源项目列表的 Teleoperation 部分新增 “wuji-teleop-ros2” 条目,包含标题、链接、简要描述与 Stars/Forks 徽章;该条目已插入到 Teleoperation 条目之后、Hardware Info 条目之前,其他条目保持不变。
    • 调整了 Teleoperation 展示表格的行合并设置,以容纳新增条目并提升列表显示的整齐性。

✏️ Tip: You can customize this high-level summary in your review settings.

adding the teleop-ros2 to github page
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Jan 26, 2026

Walkthrough

在 Teleoperation 分类中新增仓库 wuji-teleop-ros2:向 repos-config.yml 添加该条目,并在 profile/README.md 的 Teleoperation 表格中将原单元格改为 rowspan="2" 并新增对应行含链接、描述与 Stars/Forks 徽章。无其他改动。

Changes

内聚组 / 文件(s) 变更摘要
文档:README 表格
profile/README.md
在开源项目表格的 Teleoperation 区块将原单元格改为 rowspan="2",并新增 wuji-teleop-ros2 行,包含项目链接、描述 “Ros2 Teleop system: Full teleoperation system with Tianji arm based on ROS2.” 及 Stars/Forks 徽章,新增行位于现有 Teleoperation 条目之后、Hardware Info 之前。
配置:仓库列表
repos-config.yml
categoriesTeleoperationrepos 下新增仓库条目 wuji-teleop-ros2

Sequence Diagram(s)

(无序列图 — 仅文档与配置项新增,未引入新的多组件控制流。)

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

Suggested reviewers

  • chenjunnn
  • christopheralex-cc

Poem

🐰 我在 README 上轻轻跳,
新添一行连星耀,
wuji 遥控转臂舞蹈,
徽章闪烁像灯梢,
开源田间笑声飘。

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed 标题准确反映了主要变更:向README添加wuji-teleop-ros2项目,与修改内容完全匹配。
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.


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

Copy link
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 1

🤖 Fix all issues with AI agents
In `@profile/README.md`:
- Around line 76-80: In the table row containing the link text
"wuji-teleop-ros2", remove the trailing ".git" from the href URL so it matches
other GitHub links, and change the description text "Ros2 Teleop system: Full
teleoperation system with Tianji arm based on Ros2." to use consistent casing
"ROS2" (e.g., "ROS2 Teleop system: Full teleoperation system with Tianji arm
based on ROS2.").

Copy link
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 0

Caution

Some comments are outside the diff and can’t be posted inline due to platform limitations.

⚠️ Outside diff range comments (1)
profile/README.md (1)

69-83: 修复表格结构:将“Teleoperation”表头合并为 rowspan=2,避免 脱离
当前 Lines 76-78 的 <td> 不在任何 <tr> 内,会导致表格渲染异常。建议把上一行表头的 rowspan 改为 2,并移除多余的表头单元格。

🔧 建议修改
-    <tr>
-        <td colspan="1" rowspan="1" align="center" class="ai-notebooks-table-points ai-orange-link">
+    <tr>
+        <td colspan="1" rowspan="2" align="center" class="ai-notebooks-table-points ai-orange-link">
             Teleoperation
         </td>
         <td><a href="/wuji-technology/wuji-retargeting" target="_blank"> wuji-retargeting </a> <br> Hand pose retargeting system: high-precision retargeting system based on DexRetargeting algorithm, supporting Vision Pro hand tracking for real-time teleoperation.  </td>
         <td><img alt="Stars" src="https://img.shields.io/github/stars/wuji-technology/wuji-retargeting?style=flat-square"/></td>
         <td><img alt="Forks" src="https://img.shields.io/github/forks/wuji-technology/wuji-retargeting?style=flat-square"/></td>
     </tr>
-    <td colspan="1" rowspan="1" align="center" class="ai-notebooks-table-points ai-orange-link">
-        Teleoperation
-    </td>
     <tr>
         <td><a href="/wuji-technology/wuji-teleop-ros2" target="_blank"> wuji-teleop-ros2 </a> <br> Ros2 Teleop system: Full teleoperation system with Tianji arm based on ROS2.  </td>
         <td><img alt="Stars" src="https://img.shields.io/github/stars/wuji-technology/wuji-teleop-ros2?style=flat-square"/></td>
         <td><img alt="Forks" src="https://img.shields.io/github/forks/wuji-technology/wuji-teleop-ros2?style=flat-square"/></td>
     </tr>
♻️ Duplicate comments (1)
profile/README.md (1)

80-80: 统一大小写:将 “Ros2” 改为 “ROS2”
与其它条目一致性问题(已在历史评论中提出)。

zhangwt20011015 and others added 3 commits January 26, 2026 19:45
- ROS2 should be uppercase (proper noun)
- Description follows format: "Noun phrase: lowercase description."

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Copy link
Contributor Author

@zhangwt20011015 zhangwt20011015 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

感谢 CodeRabbit 的审查!所有问题已在最新提交中解决:

  1. ROS2 大小写问题 - 已修复,"Ros2" 改为 "ROS2"
  2. 表格结构 (rowspan) - 已正确设置 rowspan="2"<td> 在正确的 <tr>
  3. URL .git 后缀 - 当前 URL 无 .git 后缀

当前 description 已按规范更新为:

ROS2 teleoperation package: full teleoperation system integrating Wuji Hand with Tianji arm for real-time control.

@zhangwt20011015
Copy link
Contributor Author

根据飞书 GitHub 对外文案规范,建议将 wuji-teleop-ros2 的描述更新为与 repo description 一致:

当前 PR 中的描述:

ROS2 teleoperation package: full teleoperation system integrating Wuji Hand with Tianji arm for real-time control.

建议修改为:

ROS2 teleoperation package: real-time Wuji Hand control via Vision Pro and MANUS Gloves input.

原因:

  1. Tianji arm 支持在 README 中标注为 "Coming Soon",尚未正式发布
  2. 当前实际支持的输入设备是 Vision Pro 和 MANUS Gloves
  3. 与刚更新的 repo description 保持一致

已将 wuji-technology/wuji-teleop-ros2 的 GitHub description 更新为符合规范的版本。

@zhangwt20011015
Copy link
Contributor Author

更新:建议描述修改为:

ROS2 teleoperation package: real-time Wuji Hand and robot arm control with multiple input devices.

不强调具体输入设备品牌,保持通用性。已同步更新 repo description。

@zhangwt20011015
Copy link
Contributor Author

确认:profile/README.md 的改动应该撤销,由 update-profile.yml workflow 自动处理。

正确流程:

  1. PR 只保留 repos-config.yml 的改动(添加 wuji-teleop-ros2 到列表)
  2. 撤销 profile/README.md 的手动改动
  3. PR 合并后,手动触发 Update Profile README workflow(在 Actions 页面点击 "Run workflow")
  4. Workflow 会自动从各仓库拉取 description 并创建新 PR 更新 README

这样可以确保 description 与各仓库设置保持同步。我已将 wuji-teleop-ros2 的 repo description 更新为:

ROS2 teleoperation package: real-time Wuji Hand and robot arm control with multiple input devices.

Let the update-profile workflow handle README updates automatically.
Only keep repos-config.yml changes.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@ThirteenLLB ThirteenLLB requested a review from chenjunnn January 29, 2026 07:52
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.

2 participants