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

Enable lifts to work with TPE #250

Merged
merged 1 commit into from
Oct 30, 2020

Conversation

mrushyendra
Copy link
Contributor

Modifies lift plugin to issue either joint or model velocity commands based on the physics engine used.

Modifies lift plugin to issue either joint or model velocity commands
based on the physics engine used.
Copy link
Member

@luca-della-vedova luca-della-vedova left a comment

Choose a reason for hiding this comment

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

The code itself looks good, there is no way to test it yet though since link level commands in TPE are still a work in progress.
I guess we would either need a simple test world with lifts that have a single door (maybe a branch on a rmf_demos fork?) or we would need to wait for the feature to make it in ign-physics first.
What do you think?

@mrushyendra
Copy link
Contributor Author

I was able to test just the lift movement on TPE (minus the door opening/closing) with the lift panel tab on Rviz. Combined with the full testing with DART, would that be sufficient? If not, I can try creating a demo with a lift that has a single door.

@luca-della-vedova
Copy link
Member

Good point, I managed to test it through full testing in DART and partial testing (manually placing the robot inside the lift) through RVIZ as you mentioned and it looks great!

@luca-della-vedova luca-della-vedova merged commit 20a9cd6 into open-rmf:master Oct 30, 2020
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.

None yet

2 participants