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

[ROS_Client][wip] Add point head method (dependent on PR2 PointHead action). #427

Closed
wants to merge 4 commits into from

Conversation

130s
Copy link
Contributor

@130s 130s commented Feb 1, 2016

Do NOT merge yet.

Hinted at http://wiki.ros.org/pr2_controllers/Tutorials/Moving%20the%20Head

  • Action server not returns. Head does not move as a result.
  • Add tests.

@130s
Copy link
Contributor Author

130s commented Feb 1, 2016

Added hironx_head_action pkg, similar as pr2_head_action pkg. Now action server seems to be trying to do the work but still yields error:

[ERROR] [1454350266.914027519, 221.234999999]: Service call to query controller trajectory failed.

while sending commands like this:

$ ipython -i `rospack find hironx_ros_bridge`/scripts/hironx.py
:
In [13]: ros.look_at('WAIST', 8.2, 0.1, 0.01)
[INFO] [WallTime: 1454350253.282177] [207.510000] goal sent via aclient_point_head
[INFO] [WallTime: 1454350253.282725] [207.510000] done aclient_point_head

In [14]: ros.look_at('WAIST', 8.2, 0.1, 1.01)
[INFO] [WallTime: 1454350263.056392] [217.370000] goal sent via aclient_point_head
[INFO] [WallTime: 1454350263.056962] [217.370000] done aclient_point_head

In [15]: ros.look_at('WAIST', 1.2, 0.1, 1.01)
[INFO] [WallTime: 1454350266.912696] [221.235000] goal sent via aclient_point_head
[INFO] [WallTime: 1454350266.913651] [221.235000] done aclient_point_head

This error is returned from the action server I'm adding:
https://github.com/start-jsk/rtmros_hironx/pull/427/files#diff-e4974846b35b9f6fa59a02a56c7775e2R291

+    if (!cli_query_traj_.call(traj_state))
+    {
+      ROS_ERROR("Service call to query controller trajectory failed.");
+      gh.setRejected();
+      return;
+    }

@k-okada
Copy link
Member

k-okada commented Sep 5, 2016

rewrited ad #468

@k-okada k-okada closed this Sep 5, 2016
@130s 130s deleted the add/test_tf branch March 5, 2017 05:29
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