Jodan/ahrs imu#12
Open
Zyodooon wants to merge 15 commits into
Open
Conversation
There was a problem hiding this comment.
Pull request overview
This PR updates the robot documentation/topology to reflect an integrated arm + gripper Dynamixel driver, and introduces a new ROS 2 package for the HWT901B AHRS IMU (arm IMU) that publishes IMU and magnetometer topics.
Changes:
- Update system/network topology diagrams and architecture docs to remove the standalone
gripper_driverand showarm_driverhandling both arm + gripper on a combined Dynamixel bus. - Add a new
hwt901b_imuROS 2 (ament_cmake) package with a C++ node that parses HWT901B serial frames and publishessensor_msgs/Imuandsensor_msgs/MagneticField. - Update root README with an architecture diagram and refresh the hardware interface table.
Reviewed changes
Copilot reviewed 10 out of 10 changed files in this pull request and generated 7 comments.
Show a summary per file
| File | Description |
|---|---|
| main_ws/topology/system_topology.mmd | Updates manipulation graph to route gripper cmd/status through arm_driver and represent unified Dynamixel hardware. |
| main_ws/topology/network_topology.mmd | Updates robot hardware + ROS node connectivity to reflect unified arm+gripper device and removal of gripper_driver. |
| docs/ARCHITECTURE.md | Adjusts component list to describe arm_driver as the integrated arm+gripper driver. |
| README.md | Adds an architecture diagram and updates the hardware table for the integrated arm+gripper setup. |
| main_ws/src/perception/arm_imu/src/hwt901b_imu_node.cpp | New HWT901B IMU ROS 2 node implementation (serial parsing + publishing). |
| main_ws/src/perception/arm_imu/include/hwt901b_imu/hwt901b_imu_node.hpp | Header for the new IMU node. |
| main_ws/src/perception/arm_imu/CMakeLists.txt | Build/install rules for the new hwt901b_imu_node executable. |
| main_ws/src/perception/arm_imu/package.xml | New ROS 2 package manifest for hwt901b_imu. |
| main_ws/src/perception/arm_imu/ARM_IMU_NOTES.md | Sensor notes/spec reference for HWT901B framing and configuration. |
| main_ws/src/perception/arm_imu/todo.md | Initial TODO list for arm IMU work. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Collaborator
Author
|
まだやること残ってたので一旦クローズします |
80191ee to
1d4e435
Compare
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.
No description provided.