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
numdifftools dependency #45
Comments
[ LP comment 1 by: Skipper Seabold, on 2010-10-04 15:42:44.878854+00:00 ] Should we include numdifftools or go ahead and use the versions that we have in the sandbox? Strictly speaking neither is necessary as the solvers don't yet require the score and hessian methods, but they will if we want to inherit from LikelihoodModel. |
[ LP comment 2 by: joep, on 2010-10-04 16:14:56.316846+00:00 ] The solvers don't need it (not all of them), but the likelihood methods (generic, non-linear or non-normal) need the Jacobian or/and Hessian to calculate the cov_params and bse. Plus there are other places where we need numerical derivatives, e.g. derivatives of moment conditions in gmm and in delta-method to get bse and cov_params for non-linear transformation of estimated parameters. (this function is currently together with generic mle poisson) (current status copied from the mailinglist) There will always be some problems with numerical differentiation, but I wouldn't change around too much with numerical differentiation, |
[ LP comment 3 by: joep, on 2011-01-17 13:45:28.184022+00:00 ] numdifftool (no version information in the installed package) raises exception with numpy 1.5.1 when running the main part in our numdiff" scikits\statsmodels\sandbox\regression\numdiff.py Traceback (most recent call last): |
[ LP comment 4 by: Skipper Seabold, on 2011-02-17 19:01:40.569441+00:00 ] There is still one unprotected import of numdifftools in tsa/mlemodel.py |
ENH: Protect numdifftools import. Closes #45
ENH: Protect numdifftools import. Closes statsmodels#45
Original Launchpad bug 653902: https://bugs.launchpad.net/statsmodels/+bug/653902
Reported by: vincent-vincentdavis (Vincent Davis).
statsmodels/init.py imports tsa
Which then returns an exception from statsmodels/tsa/var.py "raise Warning("You need to install numdifftools to try out the AR model")"
Should numdifftools be a dependency for all of statsmodels ?
The text was updated successfully, but these errors were encountered: