add legacy settings for nyquist

murrayrm · murrayrm · commit 67c1f3f68944 · 2022-04-15T17:53:29.000-07:00
diff --git a/control/config.py b/control/config.py
@@ -267,6 +267,15 @@ def use_legacy_defaults(version):
     #
     reset_defaults()            # start from a clean slate
 
+    # Version 0.9.2:
+    if major == 0 and minor < 9 or (minor == 9 and patch < 2):
+        from math import inf
+
+        # Reset Nyquist defaults
+        set_defaults('nyquist', indent_radius=0.1, max_curve_magnitude=inf,
+                     max_curve_offset=0, primary_style=['-', '-'],
+                     mirror_style=['--', '--'], start_marker_size=0)
+
     # Version 0.9.0:
     if major == 0 and minor < 9:
         # switched to 'array' as default for state space objects
diff --git a/control/tests/nyquist_test.py b/control/tests/nyquist_test.py
@@ -337,6 +337,17 @@ def test_linestyle_checks():
     with pytest.raises(ValueError, match="invalid 'mirror_style'"):
         ct.nyquist_plot(sys, mirror_style=0.2)
 
+@pytest.mark.usefixtures("editsdefaults")
+def test_nyquist_legacy():
+    ct.use_legacy_defaults('0.9.1')
+
+    # Example that generated a warning using earlier defaults
+    s = ct.tf('s')
+    sys = (0.02 * s**3 - 0.1 * s) / (s**4 + s**3 + s**2 + 0.25 * s + 0.04)
+
+    with pytest.warns(UserWarning, match="indented contour may miss"):
+        count = ct.nyquist_plot(sys)
+
 
 if __name__ == "__main__":
     #
