Style cleanup

bnavigator · web-flow · commit 1e3109fcf574 · 2021-03-25T01:06:23.000+01:00
diff --git a/control/tests/timeresp_test.py b/control/tests/timeresp_test.py
@@ -301,10 +301,10 @@ def mimo_ss_step_matlab(self):
                          'SteadyStateValue': -0.5394},
                          {'RiseTime': 0.0000, # (*)
                          'SettlingTime': 3.4000,
-                         'SettlingMin': -0.4350,# -0.1935 in MATLAB. (is wrong) 
+                         'SettlingMin': -0.4350,  # (*)
                          'SettlingMax': -0.1485,
                          'Overshoot': 132.0170,
-                         'Undershoot': 0, # 0. in MATLAB (is correct)
+                         'Undershoot': 0,
                          'Peak': 0.4350,
                          'PeakTime': .2,
                          'SteadyStateValue': -0.1875}]]
diff --git a/control/timeresp.py b/control/timeresp.py
@@ -926,7 +926,7 @@ def step_info(sysdata, T=None, T_num=None, yfinal=None,
                 # Undershoot 
                 y_us = (sgnInf*yout).min()
                 y_us_index = (sgnInf*yout).argmin()
-                if (sgnInf * yout[y_us_index]) < 0: # must have oposite sign
+                if (sgnInf * yout[y_us_index]) < 0:  # InfValue and undershoot must have opposite sign
                     undershoot = np.abs(100. * np.abs(y_us) / InfValue)
                 else:
                     undershoot = 0
