-
Notifications
You must be signed in to change notification settings - Fork 6
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 our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix scipy's solve_lyapunov and update testing #130
Conversation
036b5ef
to
e09e238
Compare
setup.cfg
Outdated
@@ -3,6 +3,7 @@ description-file = README.rst | |||
|
|||
[flake8] | |||
exclude = _*.py,docs/*.py,build/*.py | |||
ignore = E741 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Suppressing:
nengolib/signal/discrete.py:132:9: E741 ambiguous variable name 'I'
nengolib/signal/lyapunov.py:181:5: E741 ambiguous variable name 'O'
nengolib/signal/lyapunov.py:223:8: E741 ambiguous variable name 'O'
nengolib/signal/realizers.py:87:9: E741 ambiguous variable name 'I'
nengolib/signal/system.py:682:9: E741 ambiguous variable name 'I'
nengolib/signal/tests/test_realizers.py:59:5: E741 ambiguous variable name 'I'
nengolib/signal/tests/test_system.py:208:5: E741 ambiguous variable name 'I'```
Codecov Report
@@ Coverage Diff @@
## master #130 +/- ##
=====================================
Coverage 100% 100%
=====================================
Files 28 28
Lines 1321 1324 +3
Branches 155 155
=====================================
+ Hits 1321 1324 +3
Continue to review full report at Codecov.
|
e09e238
to
f597394
Compare
@@ -3,6 +3,7 @@ description-file = README.rst | |||
|
|||
[flake8] | |||
exclude = _*.py,docs/*.py,build/*.py | |||
ignore = E226,E741 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
E226 is another new error for whitespace around arithmetic operator...
@@ -337,11 +340,13 @@ def _normtail(sig, A, x, C): | |||
x = _state_impulse(Phi, x0=B, k=N, delay=0) # eq (38) | |||
abs_e = np.squeeze(abs(C.dot(x.T))) | |||
x = x[:-1] # compensate for computing where thresholds crossed | |||
abs_y = abs_y[:-1] # compensate for computing where thresholds crossed |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Removing this line gives a warning that I was ignoring before, due to abs_y[cross]
having mismatched shapes between the boolean index cross
and abs_y
(latter had an extra dimension).
f597394
to
4f7f50e
Compare
Fixes #129.