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

[baxter-interface.l] Wait until all joint states are updated before moving #627

Closed
wants to merge 1 commit into from

Conversation

Projects
None yet
2 participants
@pazeshun
Copy link
Contributor

commented Jun 24, 2016

Thanks to #622
This PR is for baxter joint trajectory action server which needs the initial joint states.
Without this PR, arms sometimes move suddenly when several types of /robot/joint_states exist.

@pazeshun

This comment has been minimized.

Copy link
Contributor Author

commented Jun 25, 2016

I don't know why travis fails...

@k-okada

This comment has been minimized.

Copy link
Member

commented Jun 25, 2016

Ok, I'll take care of this

◉ Kei Okada

2016/06/25 11:54、pazeshun notifications@github.com のメッセージ:

I don't know why travis fails...


You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

k-okada added a commit that referenced this pull request Jun 25, 2016

[baxter-interface.l] Wait until all joint states are updated before m…
…oving #627 (#628)

* Wait until all joint states are updated before moving

* baxtereus/CMakeLists.txt: re-write using CATKIN_ENABLE_TESTING

* baxtereus/test/test-baxter.l: test-baxter-interface only available after indigo
@k-okada

This comment has been minimized.

Copy link
Member

commented Jun 25, 2016

Thanks merged in #628

@k-okada k-okada closed this Jun 25, 2016

@k-okada

This comment has been minimized.

Copy link
Member

commented Jun 25, 2016

memo: the error on travis in this PR only for simulation mode, for real robot, the PR is ok.

@pazeshun pazeshun deleted the pazeshun:wait-until-js-update branch Jun 25, 2016

@pazeshun

This comment has been minimized.

Copy link
Contributor Author

commented Jun 25, 2016

@k-okada Thank you

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.