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

[bug] Using Gazebo set_model_configuration srv messes up joint_states #225

Open
1 task
rickstaa opened this issue Feb 16, 2022 · 1 comment
Open
1 task

Comments

@rickstaa
Copy link
Contributor

rickstaa commented Feb 16, 2022

During my RL training, the robot sometimes becomes stuck. When this happens, I use the /gazebo/set_model_configuration service to set the Panda robot to a free robot position. However, when doing this, the joint positions are sometimes reported to be outside their joint limits. This prevents third-party tools like MoveIt from functioning.

Why this happens is very clear by looking at the code:

this->position += angles::shortest_angular_distance(this->position, position);

Every time the /gazebo/set_model_configuration changes the joint positions, they get added to the joint->positions attribute. As a result, when the change made by the /gazebo/set_model_configuration is too big, the reported joint positions are pushed outside the joint limits.

I understand why this is implemented, but providing users with a way to reset the joint positions would be very helpful since it is a common use case.

I implemented a PR to add a set_franka_model_configuration service in #226.

See the behaviour in action

  1. Clone the https:/rickstaa/franka_ros/tree/show_gazebo_set_model_config_problem.
  2. Build the catkin workspace.
  3. Source the catkin workspace.
  4. Start the panda simulation roslaunch franka_gazebo panda.launch use_gripper:=false physics:=dart controller:='force_example_controller'.
  5. Start the franka_gazebo/scripts/log_joint_violations.py script (i.e. rosrun franka_gazebo log_joint_violations.py).
  6. Start the franka_gazebo/scripts/set_random_joint_positions.py script (i.e. rosrun franka_gazebo set_random_joint_positions.py) script.
  7. See the reported joint positions being pushed outside the joint limits.

Note

This branch also includes #211 because the ODE physics engine is not stable when performing force control (see #160 (comment)).

See the fix in action

  1. Checkout the test_fix_gazebo_set_model_config_problem branch.
  2. Perform set 2-5 above.
  3. Start the franka_gazebo/scripts/set_random_joint_positions_226.py script (i.e. rosrun franka_gazebo set_random_joint_positions_226.py).
  4. See that now the joints stay within the joint limits.

TODOs

rickstaa added a commit to rickstaa/franka_ros that referenced this issue Feb 16, 2022
This commit implements a `reset_joint_states` service. This service can
be used to reset the join states when they are pushed outside of the
joint limits. This can for example happen when Gazebo's
`set_model_configuration` is used (see
frankaemika#225 for more
information).
rickstaa added a commit to rickstaa/franka_ros that referenced this issue Feb 16, 2022
This commit implements a `reset_joint_states` service. This service can
be used to reset the joint states when they are pushed outside of the
joint limits. This can for example happen when Gazebo's
`set_model_configuration` is used (see
frankaemika#225 for more
information).
rickstaa added a commit to rickstaa/ros-gazebo-gym that referenced this issue Feb 16, 2022
This commit adds a fix that prevents the joint positions being
pushed outside of the joint limits when Gazebo's
'set_model_configuration' service is used. See
frankaemika/franka_ros#225 for more
information.
rickstaa added a commit to rickstaa/franka_ros that referenced this issue Feb 18, 2022
This commit implements a `set_franka_model_configuration` service. This
service can be used to set the Franka configuration in the Gazebo
simulation. Under the hood, this service calls Gazebo's
'set_model_configuration' service while ensuring that the reported joint
positions stay within joint limits (see
frankaemika#225 for more
information).
rickstaa added a commit to rickstaa/franka_ros that referenced this issue Feb 18, 2022
This commit implements a `set_franka_model_configuration` service. This
service can be used to set the Franka configuration in the Gazebo
simulation. Under the hood, this service calls Gazebo's
'set_model_configuration' service while ensuring that the reported joint
positions stay within joint limits (see
frankaemika#225 for more
information).
rickstaa added a commit to rickstaa/franka_ros that referenced this issue Feb 18, 2022
This commit implements a `set_franka_model_configuration` service. This
service can be used to set the Franka configuration in the Gazebo
simulation. Under the hood, this service calls Gazebo's
'set_model_configuration' service while ensuring that the reported joint
positions stay within joint limits (see
frankaemika#225 for more
information).
rickstaa added a commit to rickstaa/franka_ros that referenced this issue Feb 18, 2022
This commit implements a `set_franka_model_configuration` service. This
service can be used to set the Franka configuration in the Gazebo
simulation. Under the hood, this service calls Gazebo's
'set_model_configuration' service while ensuring that the reported joint
positions stay within joint limits (see
frankaemika#225 for more
information).
rickstaa added a commit to rickstaa/franka_ros that referenced this issue Mar 18, 2022
This commit implements a `set_franka_model_configuration` service. This
service can be used to set the Franka configuration in the Gazebo
simulation. Under the hood, this service calls Gazebo's
'set_model_configuration' service while ensuring that the reported joint
positions stay within joint limits (see
frankaemika#225 for more
information).
rickstaa added a commit to rickstaa/franka_ros that referenced this issue Sep 4, 2023
This commit implements a `set_franka_model_configuration` service. This
service can be used to set the Franka configuration in the Gazebo
simulation. Under the hood, this service calls Gazebo's
'set_model_configuration' service while ensuring that the reported joint
positions stay within joint limits (see
frankaemika#225 for more
information).
rickstaa added a commit to rickstaa/franka_ros that referenced this issue Sep 4, 2023
This commit implements a `set_franka_model_configuration` service. This
service can be used to set the Franka configuration in the Gazebo
simulation. Under the hood, this service calls Gazebo's
'set_model_configuration' service while ensuring that the reported joint
positions stay within joint limits (see
frankaemika#225 for more
information).
rickstaa added a commit to rickstaa/franka_ros that referenced this issue Sep 4, 2023
This commit implements a `set_franka_model_configuration` service. This
service can be used to set the Franka configuration in the Gazebo
simulation. Under the hood, this service calls Gazebo's
'set_model_configuration' service while ensuring that the reported joint
positions stay within joint limits (see
frankaemika#225 for more
information).
rickstaa added a commit to rickstaa/franka_ros that referenced this issue Sep 4, 2023
This commit implements a `set_franka_model_configuration` service. This
service can be used to set the Franka configuration in the Gazebo
simulation. Under the hood, this service calls Gazebo's
'set_model_configuration' service while ensuring that the reported joint
positions stay within joint limits (see
frankaemika#225 for more
information).
rickstaa added a commit to rickstaa/franka_ros that referenced this issue Sep 4, 2023
This commit implements a `set_franka_model_configuration` service. This
service can be used to set the Franka configuration in the Gazebo
simulation. Under the hood, this service calls Gazebo's
'set_model_configuration' service while ensuring that the reported joint
positions stay within joint limits (see
frankaemika#225 for more
information).
rickstaa added a commit to rickstaa/franka_ros that referenced this issue Sep 4, 2023
This commit implements a `set_franka_model_configuration` service. This
service can be used to set the Franka configuration in the Gazebo
simulation. Under the hood, this service calls Gazebo's
'set_model_configuration' service while ensuring that the reported joint
positions stay within joint limits (see
frankaemika#225 for more
information).
rickstaa added a commit to rickstaa/franka_ros that referenced this issue Sep 4, 2023
This commit implements a `set_franka_model_configuration` service. This
service can be used to set the Franka configuration in the Gazebo
simulation. Under the hood, this service calls Gazebo's
'set_model_configuration' service while ensuring that the reported joint
positions stay within joint limits (see
frankaemika#225 for more
information).
rickstaa added a commit to rickstaa/franka_ros that referenced this issue Sep 4, 2023
This commit implements a `set_franka_model_configuration` service. This
service can be used to set the Franka configuration in the Gazebo
simulation. Under the hood, this service calls Gazebo's
'set_model_configuration' service while ensuring that the reported joint
positions stay within joint limits (see
frankaemika#225 for more
information).
rickstaa added a commit to rickstaa/franka_ros that referenced this issue Sep 4, 2023
This commit implements a `set_franka_model_configuration` service. This
service can be used to set the Franka configuration in the Gazebo
simulation. Under the hood, this service calls Gazebo's
'set_model_configuration' service while ensuring that the reported joint
positions stay within joint limits (see
frankaemika#225 for more
information).
rickstaa added a commit to rickstaa/franka_ros that referenced this issue Sep 4, 2023
This commit implements a `set_franka_model_configuration` service. This
service can be used to set the Franka configuration in the Gazebo
simulation. Under the hood, this service calls Gazebo's
'set_model_configuration' service while ensuring that the reported joint
positions stay within joint limits (see
frankaemika#225 for more
information).
rickstaa added a commit to rickstaa/franka_ros that referenced this issue Sep 5, 2023
This commit implements a `set_franka_model_configuration` service. This
service can be used to set the Franka configuration in the Gazebo
simulation. Under the hood, this service calls Gazebo's
'set_model_configuration' service while ensuring that the reported joint
positions stay within joint limits (see
frankaemika#225 for more
information).
@rickstaa rickstaa changed the title [bug] Using Gazebo 'set_model_configuration' srv messes up joint_states [bug] Using Gazebo set_model_configuration srv messes up joint_states Sep 5, 2023
@rickstaa
Copy link
Contributor Author

rickstaa commented Jan 3, 2024

I've recently observed an issue stemming from the changes introduced in commit 89d2571. Specifically, after invoking the original Gazebo set_model_configuration, the robot reverts to its initial position (see gif below). I adapted #226 to allow users to set the Franka configuration still.

gazeb_set_model_configuration_problem

How to reproduce

  1. Clone and build the franka_ros catking workspace.
  2. Start the franka gazebo simulation (i.e. roslaunch franka_gazebo panda.launch).
  3. Execute the following /gazebo/set_model_configuration call:
rosservice call /gazebo/set_model_configuration "model_name: 'panda'
urdf_param_name: 'robot_description'
joint_names:
- 'panda_joint2'
- 'panda_joint3'
joint_positions:
- 0
- 0.5"
  1. See the panda robot being set to the desired position and directly returning to the position before the service is called.

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

No branches or pull requests

1 participant