GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — I/O system enhancements by murrayrm · Pull Request #710 · python-control/python-control

Conversation

@murrayrm

@murrayrm murrayrm commented Mar 16, 2022

Copy link
Copy Markdown
Member

This PR adds some new functionality for I/O systems that I used in a recent course on optimization-based control that I taught at Caltech:

  • Modify the ss(), rss(), and drss() functions to return LinearIOSystem objects (instead of StateSpace objects). This makes it easier to create LTI state space systems that can be combined with other I/O systems without having to add a conversation step. Since LinearIOSystem objects are also StateSpace objects, no functionality is lost. (This change is implemented through the introduction of a internal NamedIOSystem class, to avoid import cycles.)

  • Added a new function create_statefbk_iosystem() that creates an I/O system for implementing a linear state feedback controller of the form u = ud - Kp(x - xd). The function returns an I/O system that takes xd, ud, and x as inputs and generates u as an output. The integral_action keyword can be used to define a set of outputs y = C x for which integral feedback is also included: u = ud - Kp(x - xd) - Ki(C x - C xd).

  • The lqr and dlqr commands now accept an integral_action keyword that allows outputs to be specified for implementing integral action. The resulting gain matrix has the form K = [Kp, Ki]. (This is useful for combining with the integral_action functionality in create_statefbk_iosystem()).

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

All reactions
murrayrm added 6 commits March 12, 2022 22:38
@murrayrm
ss, rss, drss return LinearIOSystem
f712096
@murrayrm
create _NamedIOSystem, _NamedIOStateSystem parent classes
4b0584d
@murrayrm
add create_statefbk_iosystem + unit tests
2b13747
@murrayrm
add integral_action option to lqr/dlqr
213905c
@murrayrm
update create_statefbk_iosys/lqr/dlqr errors + unit tests
ca77cca
@murrayrm
add documentation on integral action to lqr/dqlr
effd6b6
@coveralls

coveralls commented Mar 16, 2022 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown

Coverage Status

Coverage decreased (-0.001%) to 93.593% when pulling 316945d on murrayrm:lti_iosys into e10855e on python-control:master.

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@murrayrm murrayrm mentioned this pull request Mar 18, 2022
Change InputOutputSystem repr to show type, name, inputs, and outputs #699
Closed
@murrayrm murrayrm added this to the 0.9.2 milestone Mar 19, 2022
@murrayrm
use super() for LTI functions
316945d
@murrayrm murrayrm linked an issue Mar 22, 2022 that may be closed by this pull request
Consider using super idiom for child classes #124
Closed
@murrayrm

murrayrm commented Mar 23, 2022

Copy link
Copy Markdown
Member Author

I have some changes that build on this one, so I'm accepting the PR. If you have comments on anything here, feel free to post and I can update with another PR to address.

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@murrayrm
murrayrm merged commit 2f3ca18 into python-control:master Mar 23, 2022
@murrayrm murrayrm mentioned this pull request Mar 23, 2022
Stochastic systems additions #714
Merged
@murrayrm
murrayrm deleted the lti_iosys branch March 24, 2022 05:19
@murrayrm murrayrm mentioned this pull request Apr 9, 2022
Updated system class functionality #721
Merged
@sawyerbfuller

sawyerbfuller commented Apr 12, 2022

Copy link
Copy Markdown
Contributor

Am a little late on this, but it looks good to me.

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Uh oh!

There was an error while loading. Please reload this page.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

0.9.2

Development

Successfully merging this pull request may close these issues.

Consider using super idiom for child classes

Uh oh!

There was an error while loading. Please reload this page.

3 participants

@murrayrm @coveralls @sawyerbfuller
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
You are visitor 01402503 — GeezerVison does not store the pages you look up.
Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.