-
-
Notifications
You must be signed in to change notification settings - Fork 2.1k
raise not implemented error for non refactored distributions and functions #5799
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
Conversation
@ferrine MvStudentTRandomWalk and MvGaussianRandomWalk are also not refactored yet |
Codecov Report
@@ Coverage Diff @@
## main #5799 +/- ##
==========================================
- Coverage 89.40% 88.98% -0.42%
==========================================
Files 74 74
Lines 13772 13793 +21
==========================================
- Hits 12313 12274 -39
- Misses 1459 1519 +60
|
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.
I would phrase the errors differently, but other than that I would say this PR only needs suggested changes to be applied and then we're good to go.
The things left so far as I read are the MvStudentTRandomWalk and MvGaussianRandomWalk |
ded8beb
to
400e205
Compare
Co-authored-by: Thomas Wiecki <[email protected]>
400e205
to
fd31103
Compare
d81e9cb
to
8962d6f
Compare
Closes #5676
NotImplemented
error on attempt to create not yet refactored timeseries distributions or sample_posterior_predctive_w