Skip to content

Commit f0087b2

Browse files
committed
Fix gains validate
1 parent a043cfe commit f0087b2

File tree

2 files changed

+4
-4
lines changed

2 files changed

+4
-4
lines changed

control_toolbox/include/control_toolbox/pid.hpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -422,7 +422,7 @@ class Pid
422422

423423
bool validate(std::string & error_msg) const
424424
{
425-
if (u_min_ >= u_max_) // is false if any value is nan
425+
if (u_min_ > u_max_) // is false if any value is nan
426426
{
427427
error_msg =
428428
fmt::format("Gains: u_min ({}) must be less than or equal to u_max ({})", u_min_, u_max_);

control_toolbox/test/pid_ros_parameters_tests.cpp

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -170,7 +170,7 @@ TEST(PidParametersTest, InitPidTestBadParameter)
170170
bool ret;
171171
ASSERT_NO_THROW(
172172
ret = pid.initialize_from_args(
173-
P, I, D, TF U_MAX_BAD, U_MIN_BAD, ANTIWINDUP_STRAT, I_METHOD, D_METHOD, false));
173+
P, I, D, TF, U_MAX_BAD, U_MIN_BAD, ANTIWINDUP_STRAT, I_METHOD, D_METHOD, false));
174174
ASSERT_FALSE(ret);
175175

176176
rclcpp::Parameter param;
@@ -210,14 +210,14 @@ TEST(PidParametersTest, InitPidTestBadParameter)
210210
ANTIWINDUP_STRAT.i_min = 5.;
211211
ASSERT_NO_THROW(
212212
ret = pid.initialize_from_args(
213-
P, I, D, TF U_MAX_BAD, U_MIN_BAD, ANTIWINDUP_STRAT, I_METHOD, D_METHOD, false));
213+
P, I, D, TF, U_MAX_BAD, U_MIN_BAD, ANTIWINDUP_STRAT, I_METHOD, D_METHOD, false));
214214
ASSERT_FALSE(ret);
215215

216216
ANTIWINDUP_STRAT.i_max = -5.;
217217
ANTIWINDUP_STRAT.i_min = 10.;
218218
ASSERT_NO_THROW(
219219
ret = pid.initialize_from_args(
220-
P, I, D, TF U_MAX_BAD, U_MIN_BAD, ANTIWINDUP_STRAT, I_METHOD, D_METHOD, false));
220+
P, I, D, TF, U_MAX_BAD, U_MIN_BAD, ANTIWINDUP_STRAT, I_METHOD, D_METHOD, false));
221221
ASSERT_FALSE(ret);
222222
}
223223

0 commit comments

Comments
 (0)