Ekumen-OS / andino

Open-source diff drive robot ROS 2 compatible
165 stars 45 forks source link

Fixed wheel diameter in URDF #230

Closed JesusSilvaUtrera closed 6 months ago

JesusSilvaUtrera commented 6 months ago

This is just a test pull request to start getting used to the workflow, just changed the value of the wheel diameter to be coherent with the wheel radius in the YAML.

I have reviewed other modules and there is n sense in loading the YAML into the URDF because none of the paramters in it are used in this xacro file, so regarding that this is the only place the wheel_diameter is used it would not be bad to hardcode it.

jballoffet commented 6 months ago

On the other hand, how was this tested? Were you able to run the simulation? If so, I would appreciate it if you could attach a screenshot or gif where the changes can be depicted. Thanks!

francocipollone commented 6 months ago

On the other hand, how was this tested? Were you able to run the simulation? If so, I would appreciate it if you could attach a screenshot or gif where the changes can be depicted. Thanks!

Adding to this: The change will be only noticed in the odometry calculation and velocity of the wheels. Not visual change here.

JesusSilvaUtrera commented 6 months ago

@jballoffet I was able to run the simulation and teleoperate the robot in Gazebo using 'teleop_twist_keyboard', but as this change is minimal and it can be easily appreciated I did not include any evidence, sorry. In future PRs I will ensure to do this, thanks Javi for the feedback!

JesusSilvaUtrera commented 6 months ago

@francocipollone I could add the output of an 'echo' in the odometry or velocity topics if you want

francocipollone commented 6 months ago

@francocipollone I could add the output of an 'echo' in the odometry or velocity topics if you want

Not needed! LGTM for me . Thanks for contributing :smiley: