Title: Updated system class functionality by murrayrm · Pull Request #721 · python-control/python-control · GitHub
Open Graph Title: Updated system class functionality by murrayrm · Pull Request #721 · python-control/python-control
X Title: Updated system class functionality by murrayrm · Pull Request #721 · python-control/python-control
Description: This PR contains a refactoring of the code that is used to implement the I/O and LTI systems classes, building on some of the changes started in PR #710 and adding new functionality (and consistency). There are quite a few changes, but everything is either new functionality or backward compatible and so existing code does not need to change. Summary of changes: Changed the LTI class to use poles() and zeros() for retrieving poles and zeros, with pole() and zero() generating a PendingDeprecationWarning (which is ignored by default in Python). (The MATLAB compatibility module still uses pole() and zero().) The TimeResponseData and FrequencyResponseData objects now implement a to_pandas() method that creates a simple pandas dataframe. The FrequencyResponseData class is now used as the output for frequency response produced by freqresp() and a new function frequency_response has been defined, to be consistent with the input_output_response function. A FrequencyResponseData object can be assigned to a tuple to provide magnitude, phase, and frequency arrays, mirroring TimeResponseData functionality from PR #649. The LTI class is now a subclass of NamedIOSystem (formerly _NamedIOSystem), which allows state space and transfer functions systems to have system and signal names. In addition, the separate _NamedIOStateSystem class was eliminated in favor of using nstates to distinguish a system with no state space functionality (nstates == None, used for TransferFunction and FrequencyResponseData systems) from a system defined in state space (nstates >= 0, used for InputOutputSystem and StateSpace systems). The __repr__ method for NamedIOSystem has been updated to implement the changes in draft PR #699 and the __str__ method has been updated for linear I/O system to combine the information for an I/O system and a state space system. In particular, I/O systems are now displayed as SystemClass:sysname:[input_labels]->[output_labels]>. Common functionality for the LTI and InputOutputSystem classes is now pulled up into the NamedIOSystem class (and namedio.py) and a common parsing function _process_namedio_keywords() is used for consistent behavior across system creation functions. The drss, rss, ss2tf, tf2ss, tf2io, and ss2io functions now all accept system and signal name arguments (via _process_namedio_keywords(). The rss function can now accept a dt argument and can create either continuous-time or discrete-time random systems. The drss function just calls rss() with dt=True. The ss function can now accept function names as arguments, in which case it creates a NonlinearIOSystem (I'm not sure how useful this is, but ss is a sort of wrapper function that calls the appropriate class constructor, so it was easy enough to implement.) The interconnect function now has a warn_duplicate flag that can be used to turn on/off warning messages about duplicate objects. The default value is None, in which case warnings are suppressed for duplicated systems with generic names (of the form sys[n]. (The let's you do things like H = G * G without getting warning messages for state space systems without specified names.) The SystemTrajectory class for flat systems now has a response() method that generates a TimeDataResponse object. Added some missing type conversions between systems by using the __array_priority__ class variable to allow the right operand to override the left operand when needed. This mainly affect the FrequencyResponseData class. Updated unit tests, docstrings, user documentation, and examples to match new functionality. The following diagram (from the user documentation) illustrates the relationship between the various classes:
Open Graph Description: This PR contains a refactoring of the code that is used to implement the I/O and LTI systems classes, building on some of the changes started in PR #710 and adding new functionality (and consistenc...
X Description: This PR contains a refactoring of the code that is used to implement the I/O and LTI systems classes, building on some of the changes started in PR #710 and adding new functionality (and consistenc...
Opengraph URL: https://github.com/python-control/python-control/pull/721
X: @github
Domain: github.com
| route-pattern | /:user_id/:repository/pull/:id/checks(.:format) |
| route-controller | pull_requests |
| route-action | checks |
| fetch-nonce | v2:3ab13094-5ec3-44a5-344e-2615252ce170 |
| current-catalog-service-hash | 87dc3bc62d9b466312751bfd5f889726f4f1337bdff4e8be7da7c93d6c00a25a |
| request-id | B5C4:1DB585:1A3804:2291B9:697AFFCF |
| html-safe-nonce | 0948c9f332cfdb769e865f0c512a9cdb97786a1b8ad98ee3ec271ca607bfda7b |
| visitor-payload | eyJyZWZlcnJlciI6IiIsInJlcXVlc3RfaWQiOiJCNUM0OjFEQjU4NToxQTM4MDQ6MjI5MUI5OjY5N0FGRkNGIiwidmlzaXRvcl9pZCI6IjEyMzEwMjc4ODE5OTMxNzQ1NSIsInJlZ2lvbl9lZGdlIjoiaWFkIiwicmVnaW9uX3JlbmRlciI6ImlhZCJ9 |
| visitor-hmac | 12f200d795fd9259810219bacc74d27012938eaff30ce214c8e3815373e27fd7 |
| hovercard-subject-tag | pull_request:904518964 |
| github-keyboard-shortcuts | repository,pull-request-list,pull-request-conversation,pull-request-files-changed,checks,copilot |
| google-site-verification | Apib7-x98H0j5cPqHWwSMm6dNU4GmODRoqxLiDzdx9I |
| octolytics-url | https://collector.github.com/github/collect |
| analytics-location | / |
| fb:app_id | 1401488693436528 |
| apple-itunes-app | app-id=1477376905, app-argument=https://github.com/python-control/python-control/pull/721/checks |
| twitter:image | https://avatars.githubusercontent.com/u/293362?s=400&v=4 |
| twitter:card | summary_large_image |
| og:image | https://avatars.githubusercontent.com/u/293362?s=400&v=4 |
| og:image:alt | This PR contains a refactoring of the code that is used to implement the I/O and LTI systems classes, building on some of the changes started in PR #710 and adding new functionality (and consistenc... |
| og:site_name | GitHub |
| og:type | object |
| hostname | github.com |
| expected-hostname | github.com |
| None | 7ce8ed0a54c4730aeca4e6abacfc6490365fc42e25480b86883054df3f9181c8 |
| turbo-cache-control | no-preview |
| go-import | github.com/python-control/python-control git https://github.com/python-control/python-control.git |
| octolytics-dimension-user_id | 2285872 |
| octolytics-dimension-user_login | python-control |
| octolytics-dimension-repository_id | 22791752 |
| octolytics-dimension-repository_nwo | python-control/python-control |
| octolytics-dimension-repository_public | true |
| octolytics-dimension-repository_is_fork | false |
| octolytics-dimension-repository_network_root_id | 22791752 |
| octolytics-dimension-repository_network_root_nwo | python-control/python-control |
| turbo-body-classes | logged-out env-production page-responsive full-width full-width-p-0 |
| disable-turbo | false |
| browser-stats-url | https://api.github.com/_private/browser/stats |
| browser-errors-url | https://api.github.com/_private/browser/errors |
| release | d7bfc78137af9a4828305e52ab993fce981d7085 |
| ui-target | full |
| theme-color | #1e2327 |
| color-scheme | light dark |
Links:
Viewport: width=device-width