-
Notifications
You must be signed in to change notification settings - Fork 0
Issue #30: Fix URDF #37
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This pull request is intended to address Issue #30 by updating and correcting various aspects of the robot’s URDF description. Key changes include:
- Adjusted base and wheel properties, visual meshes, and joint specifications in the robot_core.xacro.
- Removed unused xacro includes in robot.urdf.xacro and replaced the lidar Collada mesh with an STL.
- Updated material definitions and modified the parent link and origin settings for the lidar and IMU.
Reviewed Changes
Copilot reviewed 9 out of 9 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| description/robot_core.xacro | Revised dimensions, mass calculations, visual meshes, and joint type adjustments. |
| description/robot.urdf.xacro | Removed camera and water_tank includes to streamline the URDF. |
| description/meshes/lidar.dae | Removed outdated Collada file as the lidar mesh is now provided in STL format. |
| description/materials.xacro | Added a new yellow material. |
| description/lidar.xacro | Updated the joint’s parent link and origin, and switched to an STL mesh for the lidar. |
| description/imu.xacro | Updated the IMU origin to reflect the corrected sensor placement. |
Comments suppressed due to low confidence (3)
description/robot_core.xacro:86
- Changing the wheel joint type from 'continuous' to 'fixed' disables rotational actuation. Confirm that this change is intended for the robot's design.
<joint name="${prefix}_joint" type="fixed">
description/lidar.xacro:5
- The parent link for the laser_joint was changed from 'water_tank_link' to 'base_link'. Verify that this update correctly reflects the intended physical connection in the robot.
<parent link="base_link"/>
description/imu.xacro:6
- The IMU origin was shifted from '0.0 0 0.068' to '-0.036 0.039 0.015'. Please ensure that these new offsets accurately represent the sensor's physical placement.
<origin xyz="-0.036 0.039 0.015" rpy="0 0 0"/>
|
I have errors related to ros2 control: |
|
And also gazebo cannot find meshes: |
|
@ChVictoria |
|
I run it in the simulation without using Docker: |
No description provided.