8000 Switch default state space matrix type to 'array' (instead of 'matrix') by murrayrm · Pull Request #480 · python-control/python-control · GitHub
[go: up one dir, main page]
Skip to content

Switch default state space matrix type to 'array' (instead of 'matrix') #480

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

Jump to bottom
Merged
merged 2 commits into from
Dec 28, 2020
Merged

Switch default state space matrix type to 'array' (instead of 'matrix') #480

merged 2 commits into from
Dec 28, 2020

Conversation

murrayrm
Copy link
Member

This is the initial commit to the 0.9.0 release, which changes the default value of state space matrices to be numpy.array (really ndarray) intead of numpy.matrix.

I also updated the use_legacy_defaults code to be a bit more general, so that as we make changes we can allow backward compatibility.

@coveralls
Copy link
coveralls commented Dec 28, 2020 •
edited
Loading

Coverage Status

Coverage increased (+0.04%) to 86.367% when pulling 2849413 on murrayrm:array-by-default into 35f1e64 on python-control:master.

bnavigator
bnavigator approved these changes Dec 28, 2020
View reviewed changes
Copy link
Contributor
@bnavigator bnavigator left a comment

Choose a reason for hiding this comment

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

More work created for the necessary rebase of #438, but that's okay.

@bnavigator bnavigator merged commit 37d1121 into python-control:master Dec 28, 2020
@murrayrm murrayrm deleted the array-by-default branch December 29, 2020 00:41
@murrayrm murrayrm added this to the 0.9.0 milestone Mar 20, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@bnavigator bnavigator bnavigator 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.
3 participants
@murrayrm @coveralls @bnavigator
0