Point streaming and multiple robot groups about motoman

[rr-mark]

That seems to work. My code is here rivelinrobotics@d34b4ad if anyone is interested in cherry-picking it into a PR.

[rr-mark]

Looks like MotomanJointTrajectoryStreamer::create_message uses a setRobotID method, which is presumably getting the wrong id from somewhere. I'll see if I can find out where and how to fix it.

[rr-mark]

Okay, I think I understand what's happening. JointTrajectoryStreamer::jointCommandCB calls MotomanJointTrajectoryStreamer::create_message(..., JointTrajectoryPoint, ...), which uses setRobotId(robot_id_).

What I think I need is MotomanJointTrajectoryStreamer::create_message(..., DynamicJointsGroup, ...), which uses setRobotId(pt.group_number).

So I guess I need to write a JointTrajectoryStreamer::jointCommandCB(DynamicJointTrajectoryConstPtr).

[gavanderhoorn]

Thanks for reporting on your progress + solution.

I noticed rivelinrobotics#1 was closed, not merged.

Did you discover something that was not correct in the end, or was the PR just for running CI?

[gavanderhoorn]

I have fixed the various MoveIt connections which were causing most of the issues in #585, such that we don't need #259 or #488.

late perhaps, but I would really recommend you do in fact incorporate the changes from #488.

The multi-group implementation in HEAD is incomplete, and #488 does fix some of those issues.

[rr-mark]

Thanks for reporting on your progress + solution.

👍

I noticed rivelinrobotics#1 was closed, not merged.
Did you discover something that was not correct in the end, or was the PR just for running CI?

This was just because the branching in our fork is a mess. We decided to make a new master branch, rather than updating our kinetic-devel branch. We are now using the contents of that PR.

If we need to keep using motoman1 for much longer we will need to tidy up our fork, but we're hoping to migrate to motoros2 before that is necessary.

[gavanderhoorn]

we're hoping to migrate to motoros2 before that is necessary.

in this context feedback/input on Yaskawa-Global/motoros2#186 would be highly welcome.

[rr-mark]

I have fixed the various MoveIt connections which were causing most of the issues in #585, such that we don't need #259 or #488.

late perhaps, but I would really recommend you do in fact incorporate the changes from #488.

The multi-group implementation in HEAD is incomplete, and #488 does fix some of those issues.

Our current solution appears to be working, so we're reluctant to make more changes unless something stops working. If we end up sticking with motoros1 for much longer and come across any problems we will look at #488 again.

[rr-mark]

we're hoping to migrate to motoros2 before that is necessary.

in this context feedback/input on Yaskawa-Global/motoros2#186 would be highly welcome.

Sure thing. We are about half way through our migration to ROS2, and we haven't got to the motoman migration yet. No doubt we will have lots of feedback/input when we get to that stage.

[rr-mark]

Actually, you're right, that discussion does look relevant to us at this stage. I'll discuss with the team, and hopefully someone will contribute soon.