Skip to content

Fix place varga #197

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.

Sign up for GitHub

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

Merged
murrayrm merged 3 commits into from
Dec 22, 2018
Merged

Fix place varga #197

Changes from 1 commit
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
9a42fc0
Add discrete time support to place_varga(). Also, fix issue #177 by c…
rabraker Feb 24, 2018
cfe18ea
Add decorators to skip new tests for place_varga if slycot is not ins…
rabraker Feb 24, 2018
33c59a8
Change DICO parameter to boolean dtime. Update tests as well.
rabraker Nov 22, 2018
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Add decorators to skip new tests for place_varga if slycot is not ins… 
…talled
  • Loading branch information
@rabraker
rabraker committed Nov 22, 2018
commit cfe18eabf1a7e8316d23cea8763679014f95d919
3 changes: 3 additions & 0 deletions control/tests/statefbk_test.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -218,6 +218,7 @@ def testPlace_varga_continuous(self):
P_placed.sort()
np.testing.assert_array_almost_equal(P, P_placed)

@unittest.skipIf(not slycot_check(), "slycot not installed")
def testPlace_varga_continuous_partial_eigs(self):
"""
Check that we are able to use the alpha parameter to only place
Expand All @@ -239,6 +240,7 @@ def testPlace_varga_continuous_partial_eigs(self):
P_placed.sort()
np.testing.assert_array_almost_equal(P_expected, P_placed)

@unittest.skipIf(not slycot_check(), "slycot not installed")
def testPlace_varga_discrete(self):
"""
Check that we can place poles using DICO='D' (discrete time)
Expand All @@ -254,6 +256,7 @@ def testPlace_varga_discrete(self):
P_placed.sort()
np.testing.assert_array_almost_equal(P, P_placed)

@unittest.skipIf(not slycot_check(), "slycot not installed")
def testPlace_varga_discrete_partial_eigs(self):
""""
Check that we can only assign a single eigenvalue in the discrete
Expand Down