Skip to content

Adaptive gain click criterion and zoom bug fix#302

Merged
murrayrm merged 9 commits intopython-control:masterfrom
icam0:sisotool_final
May 25, 2019
Merged

Adaptive gain click criterion and zoom bug fix#302
murrayrm merged 9 commits intopython-control:masterfrom
icam0:sisotool_final

Conversation

@icam0
Copy link
Copy Markdown
Contributor

@icam0 icam0 commented May 22, 2019

Fixed 2 bugs in the current sistool implementation with the help of @repagh :

  1. Added an adaptive gain criterion which makes clicking root locus around poles and zeros easier
  2. Fixed a bug where the initial zoom update was not always computed right due to the fact that the button-release event still for the zoom event still had the old limits.

@coveralls
Copy link
Copy Markdown

coveralls commented May 22, 2019

Coverage Status

Coverage increased (+0.005%) to 78.21% when pulling 4959a84 on icam0:sisotool_final into ace1683 on python-control:master.

@repagh repagh requested review from murrayrm May 22, 2019 07:45
Copy link
Copy Markdown
Member

@murrayrm murrayrm left a comment

Choose a reason for hiding this comment

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

Changes look good. There are some PEP8 warnings, but not urgent to fix now (and mainly unrelated).

@murrayrm murrayrm merged commit 7188a9c into python-control:master May 25, 2019
@murrayrm murrayrm added this to the 0.8.3 milestone Jan 4, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants