[python-control-developers] [python-control/python-control] 242d2b: Speed up freqresp for transfer functions by an ord...

[Clancy R. <cwr...@pr...>]

  Branch: refs/heads/master
  Home:   https://github.com/python-control/python-control
  Commit: 242d2b43252a2cde07ad0d0bc05aaff1305ca238
      https://github.com/python-control/python-control/commit/242d2b43252a2cde07ad0d0bc05aaff1305ca238
  Author: Clancy Rowley <cwr...@pr...>
  Date:   2014-08-13 (Wed, 13 Aug 2014)

  Changed paths:
    A control/bench/time_freqresp.py
    M control/xferfcn.py

  Log Message:
  -----------
  Speed up freqresp for transfer functions by an order of magnitude

When determining the frequency response, the list of frequencies
is now stored as a numpy array, instead of a list, and evaluated
all at once, instead of using `map`.  This is much faster.

The improved performance can be measured using a script
`bench/time_freqresp.py`.  On my machine, I get a speedup of
more than a factor of 20 (from 3.9 seconds to 0.18 seconds) for
a 10th-order transfer function.


  Commit: 6be2368cd31c55875821c4411b22d2817bc50e37
      https://github.com/python-control/python-control/commit/6be2368cd31c55875821c4411b22d2817bc50e37
  Author: Clancy Rowley <cwr...@pr...>
  Date:   2014-08-13 (Wed, 13 Aug 2014)

  Changed paths:
    M control/statesp.py

  Log Message:
  -----------
  Speed up freqresp for state-space models

Now `freqresp` converts a state-space system to a transfer
function before evaluating the frequency response.  This is much
faster than computing directly from the state-space system, which
involves solving an n-dimensional linear system at each frequency.


  Commit: e54a81bcbb8856da3d1dfbf297f4a9d4401a0317
      https://github.com/python-control/python-control/commit/e54a81bcbb8856da3d1dfbf297f4a9d4401a0317
  Author: Clancy Rowley <cwr...@pr...>
  Date:   2014-08-13 (Wed, 13 Aug 2014)

  Changed paths:
    M control/tests/matlab_test.py
    M control/timeresp.py

  Log Message:
  -----------
  Quiet down noisy MATLAB tests

Fixes #31, suppressing warnings for converting from MIMO to SISO
if the user has explicitly specified the input to use, in
`step_response`, `initial_response`, and `impulse_response`.

If a test should give a warning (such as an impulse response for
a system with a direct feedthrough term), test that a warning was issued,
but suppress the warning from the output of the test.


  Commit: 5f36a55dbf46b48b5e2854b65086f8ba356981b6
      https://github.com/python-control/python-control/commit/5f36a55dbf46b48b5e2854b65086f8ba356981b6
  Author: Clancy Rowley <cwr...@pr...>
  Date:   2014-08-13 (Wed, 13 Aug 2014)

  Changed paths:
    M control/tests/matlab_test.py
    M control/tests/test_control_matlab.py

  Log Message:
  -----------
  Fix some MATLAB tests

Fixes #30, suppressing some known warnings for systems with direct
feedthrough terms.  Also removes a duplicate test in
`test_control_matlab`, and fixes some previously broken, skipped
tests:

  test_convert_MIMO_to_SISO
  test_impulse

This update also removes a previous test to make sure warning
messages are issued: this turns out to be unreliable, because
the default behavior is for warnings to be issued only once, so
depending on the order in which tests are executed, the warning
may or may not be raised.


Compare: https://github.com/python-control/python-control/compare/03c663703d21...5f36a55dbf46