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
CMCL-0000: sync recentering with matching times #948
Conversation
Codecov ReportAttention: Patch coverage is
❗ Your organization needs to install the Codecov GitHub app to enable full functionality. Additional details and impacted files@@ Coverage Diff @@
## main #948 +/- ##
=======================================
Coverage 26.80% 26.81%
=======================================
Files 250 250
Lines 27791 27804 +13
=======================================
+ Hits 7449 7455 +6
- Misses 20342 20349 +7 ☔ View full report in Codecov by Sentry. |
PanAxis.UpdateRecentering(deltaTime, PanAxis.TrackValueChange()); | ||
TiltAxis.UpdateRecentering(deltaTime, TiltAxis.TrackValueChange()); | ||
var gotInputX = PanAxis.TrackValueChange(); | ||
var gotInputY = TiltAxis.TrackValueChange(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What happens with the radial axis?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There is no radial axis in PanTilt
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Testing looks good. On the code not sure why only 2 axis are tracked?
Purpose of this PR
Axis recentering happens independently on axes with differing recentering tmes, so that input on one axis does not impact the recentering state of the others.
note: This should be backported to 3.0 branch - done
Testing status