Skip to content

Fix margin indexing#425

Merged
murrayrm merged 5 commits into
python-control:masterfrom
bnavigator:fix-407
Jul 11, 2020
Merged

Fix margin indexing#425
murrayrm merged 5 commits into
python-control:masterfrom
bnavigator:fix-407

Conversation

@bnavigator

Copy link
Copy Markdown
Contributor

Fixes #407

@coveralls

coveralls commented Jun 29, 2020

Copy link
Copy Markdown

Coverage Status

Coverage decreased (-0.2%) to 83.975% when pulling dc4871c on bnavigator:fix-407 into a9d869e on python-control:master.

@bnavigator

bnavigator commented Jun 29, 2020

Copy link
Copy Markdown
Contributor Author

First commit is just the adjustment of the check within the list construction, but then I could not let it go and cleaned up the whole code block. Now see if I can come up with a meaningful unit test...

clean up margin_test: obsolete assert_almost_equal override; imports
@murrayrm murrayrm merged commit 711a512 into python-control:master Jul 11, 2020
@bnavigator bnavigator deleted the fix-407 branch July 26, 2020 14:39
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.

stability_margins Fails for Many FRD Systems at Stab Margins

3 participants