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

add a tiago with fixed wheels & casters #178

Merged
merged 1 commit into from
Mar 25, 2022
Merged

Conversation

jbmouret
Copy link
Collaborator

Add the previous Tiago URDF, which is useful for whole-body control or when we want a fixed Tiago. The only difference is that the wheels and casters are 'fixed joints'.

@costashatz
Copy link
Member

@jbmouret shouldn't we add a robot class for this?

@jbmouret
Copy link
Collaborator Author

I would also say that I would use the normal Tiago for simulation and this one for whole-body control. I need the URDF but not robot class.

If needed, we can use the current class with a different URDF. I would try to avoid having many different classes for variants of the same robot.

That said, if the wheels are parts of the Normal Tiago, with its own interface (which does not make sense for this specific Tiago), this could make sense to have a class.

@costashatz
Copy link
Member

OK I agree. Let's merge this...

@costashatz costashatz merged commit e5474c2 into master Mar 25, 2022
@costashatz costashatz deleted the tiago_no_wheels branch March 25, 2022 10:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants