Skip to content
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

mecanum_drive: use mesh wheels in example world #2250

Merged
merged 2 commits into from
Nov 21, 2023

Conversation

scpeters
Copy link
Member

🦟 Bug fix

Improves wheel visuals in the mecanum_drive.sdf example world.

Summary

Use the Mecanum wheel meshes from the "Mecanum lift" model on Gazebo Fuel in the mecanum_drive example world.

Screenshot 2023-11-18 at 4 10 51 PM

Checklist

  • Signed all commits for DCO
  • Added tests
  • Updated documentation (as needed)
  • Updated migration guide (as needed)
  • Consider updating Python bindings (if the library has them)
  • codecheck passed (See contributing)
  • All tests passed (See test coverage)
  • While waiting for a review on your PR, please help review another open pull request to support the maintainers

Note to maintainers: Remember to use Squash-Merge and edit the commit message to match the pull request summary while retaining Signed-off-by messages.

Use the Mecanum wheel meshes from the "Mecanum lift"
model on Gazebo Fuel in the mecanum_drive example world.

Signed-off-by: Steve Peters <[email protected]>
@github-actions github-actions bot added the 🌱 garden Ignition Garden label Nov 19, 2023
Copy link

codecov bot commented Nov 19, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (81850ce) 64.75% compared to head (a7bb50b) 64.72%.

Additional details and impacted files
@@             Coverage Diff             @@
##           gz-sim7    #2250      +/-   ##
===========================================
- Coverage    64.75%   64.72%   -0.04%     
===========================================
  Files          357      357              
  Lines        29143    29142       -1     
===========================================
- Hits         18873    18861      -12     
- Misses       10270    10281      +11     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link
Contributor

@ahcorde ahcorde left a comment

Choose a reason for hiding this comment

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

Just a minor comment but LGTM

<geometry>
<sphere>
<!--sphere>
Copy link
Contributor

Choose a reason for hiding this comment

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

remove spheres and add a comment about the size of the wheels?

Copy link
Member Author

Choose a reason for hiding this comment

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

* Add comment about scale of mesh visual
* Remove commented box, sphere geometries

Signed-off-by: Steve Peters <[email protected]>
@scpeters scpeters merged commit 3a21904 into gz-sim7 Nov 21, 2023
9 of 11 checks passed
@scpeters scpeters deleted the scpeters/mecanum_example_mesh branch November 21, 2023 00:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🌱 garden Ignition Garden
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

2 participants