-
Notifications
You must be signed in to change notification settings - Fork 32
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
Rewrite specification helpers #203
Comments
Regarding the kw94 examples, they are hybrids between kw94 and kw97. Some of the covariates are not mentioned in kw94 or can be simplified (e.g. kw94 does not distinguish between college and high school tuition). This makes it hard to compare the implemention with the original.
|
What we essentially need is reworking
Other issues we have on our list:
|
Unfortunately, in German. Can be translated if necessary. Ich habe überlegt, dass
Oder man könnte hier
, was aber schon weniger explizit ist. Die anderen würden dann so aussehen (mache mit dem ersten Design weiter):
Um ein Feature auszuschalten, würde ich einen andere key einfügen. Vielleicht
Dann würde das Man müsste vielleicht ein wenig auf die Reihenfolge achten. ZB erst choices und dann lagged choices. What are your thoughts, @peisenha @mo2561057? Especially the naming should be very intuitive. |
Now that respy is much more flexible, we still want to be able to reproduce the classical Keane Wolpin models easily. Therefore I propose to rewrite the specification helpers. They should now have the arguments:
The helper function should then generate a params DataFrame and an options dict that reproduces that can be used to estimate the corresponding model.
Potentially we could have similar helper functions to get the datasets for those models.
The text was updated successfully, but these errors were encountered: