-
Notifications
You must be signed in to change notification settings - Fork 4
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
Replace scan usage with accumulate_scan #384
Comments
Noted that #388 shifts to using |
A search of the code base suggests that we are down to only |
Agree. We could either keep this for a large refactor of renewal processes (i.e to use #408) or just do a straight swap with fewer changes asap? |
I'm inclined towards a quick swap before the refactor:
|
In #369 we added new functionality for the
AR
latent model that introduce an abstract approach to extendingBase.accumulate
via step functions. This can be used to replace our current usage of thescan
function.This is likely to be attracted as it uses a base function which will likely be a major target of Auto-diff optimisations. It also makes it easier for us to introduce composability to our
scan
functions (i.e #385).The text was updated successfully, but these errors were encountered: