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 #628

Merged
merged 3 commits into from Jun 25, 2016

Conversation

Projects
None yet
2 participants
@k-okada
Copy link
Member

commented Jun 25, 2016

@pazeshun #627

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.

This PR should fail for all distro since robot-state is not update until we explicitly
call :robot-interface-simulation-callback within :wait-until-update-all-joints (jsk-ros-pkg/jsk_pr2eus#238)

  • baxtereus: run test only for hydro
  • Wait until all joint states are updated before moving

@k-okada k-okada force-pushed the k-okada:wait-until-js-update branch from e102746 to ff3c862 Jun 25, 2016

@k-okada k-okada force-pushed the k-okada:wait-until-js-update branch from ff3c862 to b8463ce Jun 25, 2016

@k-okada k-okada merged commit 655ca3f into jsk-ros-pkg:master Jun 25, 2016

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details

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

@k-okada k-okada referenced this pull request Oct 15, 2016

Merged

update gazebo simulation #78

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.