forward_command_controller#87
Conversation
Codecov Report
@@ Coverage Diff @@
## master #87 +/- ##
==========================================
+ Coverage 33.12% 37.26% +4.13%
==========================================
Files 12 23 +11
Lines 1123 2383 +1260
Branches 740 1470 +730
==========================================
+ Hits 372 888 +516
- Misses 103 180 +77
- Partials 648 1315 +667
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
e505e69 to
17dba2b
Compare
|
@jordan-palacios can I help you with this? Testing/ debugging/ coding ... whatever you need. |
Thanks @AndyZe but this PR it's pretty much done. Right now is blocked until #90 is merged. |
|
Great! I will make it my goal to get this running in simulation with fake_joint_driver, then |
* Check joints and interface_name params * Check requested joints exist in the robot hardware * Initialize joint handles * Create command topic subscriber
Also throttled loggin macro in update()
538bcc4 to
fbacbc9
Compare
|
|
||
| namespace | ||
| { | ||
| constexpr auto kFCCLoggerName = "forward command controller"; |
There was a problem hiding this comment.
as much as I like this being called "KFC_clogger_name" I think this could be a const member of the controller?
There was a problem hiding this comment.
The logger name is already a member variable, though non const so the controllers that will inherit from this one may change it. I could make it to be default initialized to "forward command controller" and remove the kFCCLoggerName altogether.
What do you think?
Port of forward_command_controller from ROS1.
Depends #90.
Closes #78.