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.

Sign up for GitHub

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

Jump to bottom

Add command line interface for file parsing and conversion #57

Merged
merged 3 commits into from
Dec 17, 2024
Merged

Add command line interface for file parsing and conversion #57

merged 3 commits into from
Dec 17, 2024

Conversation

flferretti
Copy link
Collaborator

This PR introduces a command line interface to quickly examine model attributes and for URDF/SDF conversion.

@flferretti flferretti requested a review from xela-95 December 13, 2024 14:16
@flferretti flferretti self-assigned this Dec 13, 2024
CarlottaSartore
CarlottaSartore approved these changes Dec 16, 2024
View reviewed changes
Copy link
Contributor

@CarlottaSartore CarlottaSartore left a comment

Choose a reason for hiding this comment

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

Only minor comments.

I think it would make sense to add an usage example in the readme

src/rod/__main__.py Outdated Show resolved Hide resolved
src/rod/__main__.py Show resolved Hide resolved
flferretti and others added 2 commits December 17, 2024 10:51
@flferretti @CarlottaSartore
Address suggestions from code review
4e76bca 
Co-authored-by: Carlotta Sartore <carlotta.sartore@iit.it>
@flferretti
Add reference to CLI in the README
ce160d3
@flferretti flferretti merged commit f580e7d into main Dec 17, 2024
29 checks passed
@flferretti flferretti deleted the feature/cli branch December 17, 2024 09:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@CarlottaSartore CarlottaSartore CarlottaSartore approved these changes

@xela-95 xela-95 Awaiting requested review from xela-95

Assignees

@flferretti flferretti

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@flferretti @CarlottaSartore