-
Notifications
You must be signed in to change notification settings - Fork 8
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 custom distribute_params with core distribute_params #3
Comments
If I understand this correctly, they are doing different things, so there's no need to swap in one for the other. |
Interesting ... I haven't looked at this for some time. I think the broader question is how to best define a population of agents in an economy with ex ante heterogeneity. While refactoring part of the REMARK is not particularly urgent, to the extent that we want this to be the example that others look to for how to use HARK, the current custom tooling (which is also of an odd style -- see the use of But maybe your AgentPopulation work on SHARKFin is the best thing to swap in? |
Maybe you would agree that this ticket is superceded by #5? |
Hmmm... well, actually, this error with the array-valued DiscFac is maybe caused by this custom distribute_params code: |
The cstwMPC_MAIN.py code contains an older version of
distribute_params
which is uses quite a bit.https://github.com/econ-ark/DistributionOfWealthMPC/blob/master/Code/cstwMPC_MAIN.py#L763
This functionality is now part of HARK core. The HARK core version should be swapped in.
The text was updated successfully, but these errors were encountered: