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
[Servo] Make applyJointUpdate()
a free function
#2121
Conversation
4784f44
to
b338b3f
Compare
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## main #2121 +/- ##
==========================================
- Coverage 50.84% 50.81% -0.03%
==========================================
Files 391 391
Lines 32165 32167 +2
==========================================
- Hits 16352 16343 -9
- Misses 15813 15824 +11
... and 2 files with indirect coverage changes Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report in Codecov by Sentry. |
b338b3f
to
cbdc532
Compare
cbdc532
to
9d32b93
Compare
Description
Fixes #2120
Should be merged after #2106
This PR makes the
applyJointUpdate()
method ofServoCalcs
a free function and moves it toutilities
.This is done in order to keep the core servo code cleaner and enable better testing.
Checklist