Skip to content
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

internal/design adjustments in discretemod #63

Open
wesm opened this issue Aug 25, 2011 · 2 comments
Open

internal/design adjustments in discretemod #63

wesm opened this issue Aug 25, 2011 · 2 comments

Comments

@wesm
Copy link
Member

wesm commented Aug 25, 2011

Original Launchpad bug 642240: https://bugs.launchpad.net/statsmodels/+bug/642240
Reported by: josef-pktd (joep).

a few notes on discrete models, while looking how nested logit fits in

  • nparams
    aic, bic, ... calculations need the number of params self.df_model+self.model.J-1,
    I need it also for a default starting values and other things in ru2nmnl
    If we set nparams in the model, then bic, aic don't need to be special cased for MNL. We could use df_model as default if nparams doesn't exist in the model
  • in MNL.J and .K are not very informative when reading, nchoices and nexog would be more informative
    in ru2nmnl, I started to use nchoices, I don't think I have any nexog.
  • llnull: I don't think this will work in nested logit, and maybe not in clogit. Unfortunaltely the the arguments to model.class.init are more complicated than the generic (endog, exog)
    --> possibility to override in model or in a Results subclass ?
  • llnull: do we need the null model for anything else ? currently null model is not attached
  • Rsquare: for MNL type models, SAS (proc MDC) has six definitions additional to McFadden, they are all easy to calculate given llnull

extras not checked yet
Confusion/classification matrix ?

(I expect to add more as I work my way through this.)

@josef-pkt
Copy link
Member

the basic class structure of discrete_model has been changed PR #108

But I didn't check yet and work again on extension to conditional or nested logit

@josef-pkt josef-pkt modified the milestones: 0.7, Someday Dec 5, 2014
@josef-pkt
Copy link
Member

I just found this by chance.

review and close, or add to a SUMMary issue together with other refactoring ideas

@josef-pkt josef-pkt modified the milestones: 0.8, 0.7 Jul 19, 2015
@jseabold jseabold removed this from the 0.8 milestone Apr 6, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants