Refine governor validation details

Copilot · web-flow · commit 74d49ac5b33e · 2026-07-01T19:22:18.000Z
diff --git a/examples/steam_engine_governor.py b/examples/steam_engine_governor.py
@@ -34,6 +34,7 @@
     'load_time': 5.0,
     'sim_time': 25.0,
 }
+SETTLING_TOLERANCE = 0.02
 
 
 def create_governor_system(params):
@@ -109,7 +110,7 @@ def summarize_response(system, time, outputs, target):
     final_speed = speed[-1]
     peak_speed = np.max(speed)
     minimum_speed = np.min(speed)
-    tolerance = max(0.02 * max(abs(target), abs(final_speed)), 1e-3)
+    tolerance = max(SETTLING_TOLERANCE * max(abs(target), abs(final_speed)), 1e-3)
     if abs(target) < 1e-9:
         overshoot = 0.0
     elif target > 0:
@@ -118,6 +119,7 @@ def summarize_response(system, time, outputs, target):
         overshoot = max(0.0, (target - minimum_speed) / abs(target) * 100)
     steady_state_error = target - final_speed
     poles = list(ct.poles(system))
+    # Use the least stable (rightmost) poles as the dominant dynamics.
     dominant_poles = sorted(poles, key=lambda pole: pole.real)[-min(2, len(poles)):]
     damping_ratios = []
     for pole in dominant_poles:
@@ -147,7 +149,7 @@ def summary_text(metrics):
         if np.isfinite(metrics['settling_time']) else 'not settled'
     )
     return '\n'.join([
-        f'Settling time (2% band): {settling}',
+        f'Settling time ({SETTLING_TOLERANCE:.0%} band): {settling}',
         f'Overshoot: {metrics["overshoot"]:.2f}%',
         f'Steady-state error: {metrics["steady_state_error"]:.4f}',
         f'Dominant poles: {poles}',
