-
Notifications
You must be signed in to change notification settings - Fork 22
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 our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Saturated feedback controllers #273
Saturated feedback controllers #273
Conversation
@cmichelenstrofer - for the off-diagonal elements, are you doing anything to enforce symmetry? It'd probably be good to do this optionally. |
Pull Request Test Coverage Report for Build 6829198216Warning: This coverage report may be inaccurate.We've detected an issue with your CI configuration that might affect the accuracy of this pull request's coverage report.
💛 - Coveralls |
example notebook: saturated_pi.ipynb.zip |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
probably worth a quick meeting to discuss this
@cmichelenstrofer - I can't seem to assign you as the reviewer for this, but can you please take a final look at what I've added in |
Description
This PR
Type of PR
Checklist for PR
Additional details
Saturated PI controller has been requested multiple times. See #269