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

Added support for Ignition #1

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from

Conversation

ahcorde
Copy link

@ahcorde ahcorde commented Jan 27, 2022

Hi

I have been playing around with panda and ign_ros_control. I made this changes to make it work.

  • Added inertias
  • Added a new param to setup the simulation
    panda_moveit

Signed-off-by: ahcorde ahcorde@gmail.com

Signed-off-by: ahcorde <ahcorde@gmail.com>
Signed-off-by: ahcorde <ahcorde@gmail.com>
@ahcorde
Copy link
Author

ahcorde commented Feb 14, 2022

friendly ping @marcbone

@marcbone
Copy link

Hey @ahcorde, thanks for your contribution. We didnt forgot it, but are sadly super busy with other stuff right now. We did not take a detailed look into it, but our main concern is that you are using the position interface that we cannot implement for the real robot at the moment. So you cannot write a controller that works with the real robot and in the simulator. On the other hand, using the position interface allows at least to run moveit in simulation. Those are our initial thoughts on it without testing it.

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