Skip to content

Conversation

Danfoa
Copy link
Contributor

@Danfoa Danfoa commented Aug 25, 2025

Simple change adding Tiago++ from mujoco menagerie

Danfoa added 2 commits August 25, 2025 16:57
- Added TIAGo++ (MJCF) description to CHANGELOG.md
- Updated README.md to include TIAGo++ in the robot descriptions list
@coveralls
Copy link

coveralls commented Aug 25, 2025

Pull Request Test Coverage Report for Build 17236375943

Details

  • 6 of 6 (100.0%) changed or added relevant lines in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.009%) to 98.133%

Totals Coverage Status
Change from base Build 16796111045: 0.009%
Covered Lines: 1209
Relevant Lines: 1232

💛 - Coveralls

@Danfoa Danfoa changed the title Add PAL Tiago++/Tiago-Pro dual MJCF description module Add PAL Tiago++ dual MJCF description module Aug 26, 2025
Copy link
Member

@stephane-caron stephane-caron left a comment

Choose a reason for hiding this comment

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

Thank you for this addition 👍 The changes look good to me, I just have one question re. the updated TOC in the readme.

README.md Outdated
- [Mobile manipulators](#mobile-manipulators)
- [Quadrupeds](#quadrupeds)
- [Wheeled](#wheeled)
- [Robot descriptions in Python](#robot-descriptions-in-python)
Copy link
Member

Choose a reason for hiding this comment

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

The TOC here was only meant to navigate locally between descriptions. Are you suggesting adding a global TOC to the readme?

@Danfoa
Copy link
Contributor Author

Danfoa commented Sep 2, 2025

Hi @stephane-caron,

Sorry it was an automatic formatting of a TOC by a markdown extension I had enabled in VS code.

Last commit returns to original TOC

@stephane-caron stephane-caron merged commit e31e5f4 into robot-descriptions:main Sep 3, 2025
9 of 15 checks passed
@stephane-caron
Copy link
Member

LGTM, thank you for adding this description 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants