-
Notifications
You must be signed in to change notification settings - Fork 2.8k
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
CLN: Replace #5779 #5781
CLN: Replace #5779 #5781
Conversation
Codecov Report
@@ Coverage Diff @@
## master #5781 +/- ##
==========================================
+ Coverage 82.4% 82.41% +<.01%
==========================================
Files 594 594
Lines 93748 93794 +46
Branches 10357 10352 -5
==========================================
+ Hits 77250 77296 +46
+ Misses 14100 14099 -1
- Partials 2398 2399 +1
Continue to review full report at Codecov.
|
1 similar comment
Codecov Report
@@ Coverage Diff @@
## master #5781 +/- ##
==========================================
+ Coverage 82.4% 82.41% +<.01%
==========================================
Files 594 594
Lines 93748 93794 +46
Branches 10357 10352 -5
==========================================
+ Hits 77250 77296 +46
+ Misses 14100 14099 -1
- Partials 2398 2399 +1
Continue to review full report at Codecov.
|
Random fail. Just merged patch to protect against this |
@@ -266,7 +277,7 @@ def partial_transform_sigma2(sigma2): | |||
approx_fd_centered = ( | |||
mod.score(uparams, transformed=False, approx_complex_step=False, | |||
approx_centered=True)) | |||
assert_allclose(approx_fd, analytic_score, atol=1e-5) | |||
assert_allclose(approx_fd_centered, analytic_score, atol=1e-5) |
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.
Good catch, thanks!
Thanks! |
@ChadFulton the bare minimum to get linter to pass on statespace/tests. There were a few places where changing assert_raises to pytest.raises was the best way to fix a linting complaint, but I didn't make that change in any other places, so the usage is still plenty mixed.