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

new dynamics and output functions for statespace and iosystems#558

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 5 commits intopython-control:masterfromsawyerbfuller:dynamics2
Mar 19, 2021

Conversation

sawyerbfuller
Copy link
Contributor

@sawyerbfullersawyerbfuller commentedMar 11, 2021
edited
Loading

revised version of#546.

  • makest argument required following discussion there. not sure yet that this is the best, but I was almost there already so I just submitted this PR.
  • benchmark results in the discussion of that PR that showed that creating a separate callable that skips argument processing had only a small performance advantage, so I decided to leavedynamics as a method.
  • leaves code and usage of_rhs and_out unchanged iniosys. The idea is to leave them as fast internal methods and to havedynamics andoutput serve as more user-friendly versions where speed is of less importance.

@coveralls
Copy link

coveralls commentedMar 11, 2021
edited
Loading

Coverage Status

Coverage increased (+0.01%) to 88.872% when pulling23c4b09 on sawyerbfuller:dynamics2 into92de12d on python-control:master.

@murrayrmmurrayrm added this to the0.9.0 milestoneMar 13, 2021
@murrayrmmurrayrm merged commit22b9953 intopython-control:masterMar 19, 2021
@sawyerbfullersawyerbfuller linked an issueMar 19, 2021 that may beclosed by this pull request
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers

@murrayrmmurrayrmmurrayrm approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
0.9.0
Development

Successfully merging this pull request may close these issues.

Some way of stepping a system?
3 participants
@sawyerbfuller@coveralls@murrayrm

[8]ページ先頭

©2009-2025 Movatter.jp