CommonplaceRobotics / iRC_ROS

ROS2 packages for the igus Robot Control
Apache License 2.0
15 stars 7 forks source link

Improve startup routine regarding error reset and referencing #38

Closed cpr-fer closed 1 year ago

cpr-fer commented 1 year ago

The current PR state already fixes the issue, it still needs some work to be good code and some explaination what was done

cpr-fer commented 1 year ago

Only thing left before merging this is to test it with a DP 5 to check if the referencing patch works

cpr-fer commented 1 year ago

There appears to be an issue with referencing the open loop modules due to a firmware bug. Needs further investigation and special handling before this can be merged.

cpr-fer commented 1 year ago

Rebase once #43 is merged into humble

cpr-fer commented 1 year ago

The issues with the DP 5 and other open loop modules not referencing realiable will be addressed in a future PR, see issue #44 for future work in that regard. Since the intended things are addressed here and work with the ReBeLs I will merge this now.