allwpilib icon indicating copy to clipboard operation
allwpilib copied to clipboard

Published 20 hours ago • verified publisher icon wpilibsuite

The `Sendable` for `PIDController` does not contain the values for the integrator range

Open chauser opened this issue 10 months ago • 1 comments

The Sendable for PIDController allows adjusting all of the values that affect calculate's result except for m_minimumIntegral and m_maximumIntegral. When tuning a controller these may need adjustment, so having them as part of the Sendable would be useful.

Can this be changed unilaterally in PIDController, or does it require changes to the code for the PIDController type on various dashboards?

chauser avatar Mar 27 '24 22:03 chauser

It requires changes across all the dashboards as well.

calcmogul avatar Mar 27 '24 22:03 calcmogul