Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Additional I/O system type conversions#672

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
murrayrm merged 5 commits intopython-control:masterfrommurrayrm:ios-mixedops
Nov 27, 2021

Conversation

murrayrm
Copy link
Member

@murrayrmmurrayrm commentedNov 21, 2021
edited
Loading

This PR adds some type conversations for algebraic operations involving I/O systems, motivated by an example I was putting together where I wanted to generate a Nyquist plot for a linearized nonlinear I/O system (typeLinearIOSystem) and a simple lead compensator (typeTransferFunction). UsingP * C generated a type conversation error butC * P was OK as wasct.tf(P) * C. It seemed annoying to have to do the explicit typecasting, so I decided to implement some of the operations that were needed to makeP * C work (generates aLinearICSystem, which is a subclass ofStateSpace).

The following functionality has been added:

  • Added__sub__, so that you can write G1 - G2 for I/O systems
  • Added conversions for scalars, arrays, state space, and transfer function systems so that they can be combined with I/O systems on either the left or right.
  • Set the Numpy__array_priority__ value so that operations involving ndarrays are handled as I/O system operations and I/O system operations take precedence over LTI operations involving ndarrays.

I also did some PEP8 cleanup oniosys.py while I was at it, and put in unit tests for the code.

@murrayrm
Copy link
MemberAuthor

Rebased on top of latest master...

@coveralls
Copy link

coveralls commentedNov 21, 2021
edited
Loading

Coverage Status

Coverage increased (+0.04%) to 90.119% when pullinga44d1a4 on murrayrm:ios-mixedops into56cecc0 on python-control:master.

Copy link
Contributor

@bnavigatorbnavigator left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others.Learn more.

LGTM

@murrayrmmurrayrm merged commit20088d1 intopython-control:masterNov 27, 2021
@murrayrmmurrayrm deleted the ios-mixedops branchNovember 27, 2021 18:35
@murrayrmmurrayrm added this to the0.9.1 milestoneDec 30, 2021
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers

@bnavigatorbnavigatorbnavigator left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
0.9.1
Development

Successfully merging this pull request may close these issues.

3 participants
@murrayrm@coveralls@bnavigator

[8]ページ先頭

©2009-2025 Movatter.jp