Skip to content

Commit 5f88af3

Browse files
committed
Add unit tests
Add 7 unit tests for saturation and anti-windup feature.
1 parent 7ab99c2 commit 5f88af3

File tree

2 files changed

+426
-20
lines changed

2 files changed

+426
-20
lines changed

control_toolbox/src/pid.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -298,7 +298,7 @@ double Pid::compute_command(double error, double error_dot, const double & dt_s)
298298
gains.trk_tc_ = gains.p_gain_/gains.i_gain_;
299299
}
300300
i_term_ += dt_s * (gains.i_gain_ * error + 1/gains.trk_tc_ * (cmd_ - cmd_unsat_));
301-
} else if (gains.antiwindup_strat_ == "conditioning_technique") {
301+
} else if (gains.antiwindup_strat_ == "conditioning_technique" && gains.i_gain_ != 0) {
302302
if (gains.trk_tc_ == 0.0) {
303303
// Default value for tracking time constant for conditioning technique
304304
gains.trk_tc_ = gains.p_gain_;

0 commit comments

Comments
 (0)