Contributor

Coverage decreased (-10.01%) to 76.606% when pulling f9d4a9c on bnavigator:deprecate-matrix into 51c797e on python-control:master.

Coverage remained the same at 87.04% when pulling 1565097 on bnavigator:deprecate-matrix into c432fd5 on python-control:master.

Contributor Author

#438 will handle the CI fail: The warning category changed.

murrayrm approved these changes Dec 30, 2020

View reviewed changes

Contributor

Assume it's OK to change tests to use np.array instead of np.matrix then? This caught me out while debugging a test failure in canonical_test.py today.

Member

You might want to have a look at #438 and wait until that is merged before making changes to tests (they are undergoing significant refactoring).

We are waiting on CI tests to run, but merges should happen today.

deprecate np.matrix usage

b83cc15

print pytest summary for all except passing

1565097

Contributor Author

Now that #438 is merged, this passes. But with removing the filter as discussed #438 (comment) it seems that the actual new default is not set properly during the test suite run.

Unit tests do still not use array but matrix by default #488