Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

Added option for computing anti-stabilizing solutions of Riccati equations#281

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
murrayrm merged 6 commits intopython-control:masterfromdbankmann:master
Apr 7, 2019

Conversation

dbankmann
Copy link
Contributor

Sometimes also anti-stabilizing solutions of Riccati equations are of interest. The slicot routine offers to compute these solutions, so I added an option in thecare anddare routine, which both default back to the stabilizing case.

@coveralls
Copy link

coveralls commentedMar 21, 2019•
edited
Loading

Coverage Status

Coverage decreased (-0.04%) to 78.343% when pullingbe598c1 on dbacc:master into31e6944 on python-control:master.

@moorepants
Copy link

Can you please add unit tests to demonstrate the new functionality?

@dbankmann
Copy link
ContributorAuthor

Sure, done ;)

@murrayrm
Copy link
Member

Thanks for your contribution! The unit test needs to go incontrol/tests and use theunittest framework. I suggest adding this tostatefbk-test.py since that is where other unit tests associated with controller synthesis currently live.

@dbankmann
Copy link
ContributorAuthor

now it should be at the correct location ;)

btw. Why is@coveralls telling that coverage decreased?

@murrayrm
Copy link
Member

@dbacc Can you merge changes from latest master to get Travis CI working again (see PR#280 for more info). Thanks.

@murrayrm
Copy link
Member

Changes look good. Will include in 0.8.2 release after verifying there are no conflicts.

@murrayrmmurrayrm added this to the0.8.2 milestoneApr 6, 2019
@murrayrmmurrayrm merged commit1dd0ca2 intopython-control:masterApr 7, 2019
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
0.8.2
Development

Successfully merging this pull request may close these issues.

5 participants
@dbankmann@coveralls@moorepants@murrayrm@repagh
[8]ページ先頭
©2009-2025 Movatter.jp