You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
We should look into removing params as it is causing issues and is not necessary. If one needs the concatenation of the fitted parameters, then that should be a feature and not the default.
Specifically, the ModelContainer has setters for theta_locationtheta_scale that are quite slow because of having to update one .params dask array which is then recreated.
This should allow for theta_scale and theta_location to be optional (i.e for Poisson models)
The text was updated successfully, but these errors were encountered:
We should look into removing
params
as it is causing issues and is not necessary. If one needs the concatenation of the fitted parameters, then that should be a feature and not the default.Specifically, the
ModelContainer
has setters fortheta_location
theta_scale
that are quite slow because of having to update one.params
dask array which is then recreated.This should allow for
theta_scale
andtheta_location
to be optional (i.e forPoisson
models)The text was updated successfully, but these errors were encountered: