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

ARE: added conf files for Gazebo #672

Merged
merged 1 commit into from
Jul 17, 2020
Merged

ARE: added conf files for Gazebo #672

merged 1 commit into from
Jul 17, 2020

Conversation

pattacini
Copy link
Member

@pattacini pattacini commented Jul 17, 2020

Hi @Nicogene @vvasco 👋

This PR adds the conf files required to start up ARE along with Gazebo correctly.
Basically, Gazebo doesn't have the skin on the hand yet, thus ARE needs to use analog sensors for contact detection.

Please revise and merge.

Copy link

@vvasco vvasco left a comment

Choose a reason for hiding this comment

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

By changing grasp_model_type to springy in the conf file (see below), the module works properly.

impedence_torso_stiffness (0.0 0.9 0.9)
impedence_torso_damping (0.06 0.06 0.06)
arm_target_tol 0.01
grasp_model_type tactile
Copy link

Choose a reason for hiding this comment

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

grasp_model_type should be set to springy, otherwise the module looks for /icubSim/skin/left_hand.

Copy link
Member Author

Choose a reason for hiding this comment

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

Right! I'll do that, thanks!

@pattacini pattacini force-pushed the feat/are-gazebo-conf branch from 9906eac to e7f86c5 Compare July 17, 2020 20:13
@pattacini pattacini merged commit 6c1e164 into devel Jul 17, 2020
@pattacini pattacini deleted the feat/are-gazebo-conf branch July 17, 2020 20:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants