Skip to content

Add tags to robot descriptions database #133

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

Merged
merged 12 commits into from
Apr 1, 2025
Merged

Add tags to robot descriptions database #133

merged 12 commits into from
Apr 1, 2025

Conversation

stephane-caron
Copy link
Member

@stephane-caron stephane-caron commented Mar 31, 2025

This PR follows up the suggestion by @drfenixion in #130 by adding a tags field to the list of robot descriptions.

Tags are implemented as sets of strings (rather than enumerations like we do for formats) to allow for various future usages. The current use case for tags is about morphology (e.g. "quadruped", "humanoid", ...), but in the future it may be pertinent to other use cases to filter over other features (e.g. "tendon_driven", "can_jump", ...)

This PR tags quadruped robot descriptions and adds an example showing how to load them all in Pinocchio:

image

@coveralls
Copy link

coveralls commented Mar 31, 2025

Pull Request Test Coverage Report for Build 14196394399

Details

  • 13 of 13 (100.0%) changed or added relevant lines in 3 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.3%) to 97.89%

Totals Coverage Status
Change from base Build 14195396351: 0.3%
Covered Lines: 1067
Relevant Lines: 1090

💛 - Coveralls

@stephane-caron
Copy link
Member Author

Added a similar example with humanoid robot descriptions:

image

@drfenixion
Copy link
Contributor

drfenixion commented Apr 1, 2025

@stephane-caron Great work! It also nice to tag manipulators, grippers, bipeds if you know what packages contain is.

Btw, I am integrating Robot_descriptions to RobotCAD and currently made most of work. I want to make categories in UI based on your tags. I would be grateful if you could merge this quickly.

You can see how is displayed imported models.
models_lib

draco_3
G1_Unitree

B2_Unitree

Anymal_C_Anybotics

A1_Unitree
Barret_hand_Jhu-lcsr-attic
Allegro_hand_Robotlocomotion

@stephane-caron
Copy link
Member Author

Your project looks great 🤩

I've added a tag to each description based on the readme. Feel free to open a PR if you find some tags are missing.

I will merge this PR now, and aim for a release of v1.16.0 of the project by the end of this week.

@stephane-caron stephane-caron merged commit 8e1247c into main Apr 1, 2025
15 checks passed
@stephane-caron stephane-caron deleted the feature/tags branch April 1, 2025 14:04
@drfenixion
Copy link
Contributor

drfenixion commented Apr 2, 2025

Thank you for fast tags feature creating.
You don't have to rush the release, I'm using the repository as source instead of pip anyway.

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