Skip to content

Direct streaming#16

Closed
ghost wants to merge 24 commits intogroovy-develfrom
unknown repository
Closed

Direct streaming#16
ghost wants to merge 24 commits intogroovy-develfrom
unknown repository

Conversation

@ghost
Copy link
Copy Markdown

@ghost ghost commented Sep 25, 2013

These are the changes that were discussed in https://groups.google.com/forum/#!topic/swri-ros-pkg-dev/rRL9vwQ8yK0
The commit tree is a bit messy (ie "bug fix" and "typo fix") but it might be too late to rebase it...

…erface.h.

Added a ros::param for this timeout parameter.
…erface.h.

Added a ros::param for this timeout parameter.
…Utilisateurs/Stages/2013 Controleur generique/Teleoperation/motoman into direct_streaming
…Utilisateurs/Stages/2013 Controleur generique/Teleoperation/motoman into direct_streaming
…Utilisateurs/Stages/2013 Controleur generique/Teleoperation/motoman into direct_streaming
…his->mutex_.lock(); trajectoryStop(); this->mutex_.unlock();"

Now if a new non-empty trajectory is received while in TransferStates::STREAMING, there is a warning message instead of error, and the new trajectory is directly sent to the robot (jointTrajectoryCB in industrial_robot_client/joint_trajectory_streamer.cpp).
…Utilisateurs/Stages/2013 Controleur generique/Teleoperation/motoman into direct_streaming
…Utilisateurs/Stages/2013 Controleur generique/Teleoperation/motoman into direct_streaming
… the positions from the robot and the positions we ask for is the same when switching from IDLE to STREAMING.

Added a warning for the frame_id which must be 0 at this switch.
…Utilisateurs/Stages/2013 Controleur generique/Teleoperation/motoman into direct_streaming
…Utilisateurs/Stages/2013 Controleur generique/Teleoperation/motoman into direct_streaming
@shaun-edwards
Copy link
Copy Markdown
Member

shaun-edwards commented Jun 5, 2016

#88 updates this PR for indigo, closing

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants