-
Notifications
You must be signed in to change notification settings - Fork 3
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
Tweak autocollision threshold in JointInit #3
Comments
Hi Yoan, I don't if it is still usefull but I think that the topic for this message should be /bridge_init instead of /machine_init. Using /bridge_init I'm able to see the ACK from the joints. |
This piece of code was sent by a Comau engineer working on eDo so I highly doubt that he made a mistake.
If you have some time, that would be very interesting to code a nice |
Hi Yoan, |
Hi @Bracewind
before And changing the spin method of JointTrajectoryActionServer to:
I tested it on eDO without gripper version 3.0, I also modified the code from Yoan as you did in your pull request. Anyway I'm going to test it on an eDO with a gripper tomorrow morning probably. |
I tried your code and it did not work for me. When I put a mask of 127, an error occur and the robot must be reboot. When I put a mask of 63, It works as before, i.e it works but there is times when the robot falls. I put the code between self._joint_calibration_command_pub and rospy.Subscriber('/machine_movement_ack'... |
I have done some tests this morning on a model with the gripper, it seems that Have you modified also the spin method as I wrote in the previous comment? In my previous tests on a model without the gripper it seemed necessary. |
Yes, and I just tried with your value, and the issue is not solved for me :/. The robot is still falling at times. |
Don't know if it makes any difference but I had the code to disable collisions among I have modified the About the robot falling I don't really know what else could be causing the problem at this point. On an edo without gripper it has been working well for a long time now. Are you using the tablet app to initialize and calibrate edo? Does the app prompt something about brakes check? Are you connected by cable to edo? Offtopic(?): Comau has just published an sdk to interact with edo using python |
no difference :/ I'm using wifi, I doubt that it is the cause though. Anyway, thanks for all the help you have given, if you can push your edo_control/edo_core package code somewhere i will track the differences, if not I will try something else |
On Friday I couldn't do the tests on the model with the gripper. I did the tests today and I have again the same problems that you are describing. It seems like the solution that was working for the version w/o gripper for some reason does not work with a gripper attached. Eventually I will try to reach to Comau if I can't find an obvious solution in a reasonable time.
I can't share the modified code right now, but I'm working on a way to share it. |
@Bracewind this code works for me on eDO with gripper:
|
It works perfectly, thanks a lot |
If you have some spare time, it would be nice to integrate this to a Also, it looks like the last eDo messages have some collision-related messages here Comau/eDO_core_msgs@0daad46#diff-af3b638bc2a3e6c650974192a53c7291R75 |
I have the edo message package contained in the robot, so I don't know it it compiles with the one from the git repository. |
|
I can confirm that this works on e.DO with gripper on firmware v2.5. Only thing is that edo_core_msgs must be obtained from the e.DO itself for anyone else trying this because CollisionThreshold message is missing from the Comau github. Thanks a lot for the fix! |
We should be able to tweak the autocollision threshold from the dynamic reconfigure server so that the robot does not inerrrupts by itself during some high speed motions. Info from Comau engineers:
The text was updated successfully, but these errors were encountered: