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

Income profile calibration tools #941

Merged
merged 70 commits into from
Feb 11, 2021
Merged

Conversation

Mv77
Copy link
Contributor

@Mv77 Mv77 commented Feb 5, 2021

This PR adds utilities to produce life-cycle calibrations for the trajectory of permanent income.

I combined the file with new tools with the one that contained Sabelhaus-Song variance profiles. It contains the exact income calibrations from Cocco, Gomes, Maenhout (2005) and Cagetti (2003). I also added examples:

  • examples/Calibration/Income_calibrations.py plots the means of the income processes.
  • examples/Calibration/Life_Cycle_example.py: uses all the tools that I've added over the last few days to show how to solve and simulate a realistic life-cycle calibration in a few lines.

I'd like to request review.

  • Tests for new functionality/models or Tests to reproduce the bug-fix in code.
  • Updated documentation of features that add new functionality.
  • Update CHANGELOG.md with major/minor changes.

@Mv77
Copy link
Contributor Author

Mv77 commented Feb 8, 2021

It seems like the weird issue from before --the __fast consumer types fails the Windows test and only that test-- happened again.

@sbenthall @MridulS not sure if given the recurrence it's worth making an issue to look into it. Last time it was resolved by simply re-running the checks, but I have no permission to do that. Could any of you re-trigger the checks?

@MridulS
Copy link
Member

MridulS commented Feb 9, 2021

The test seems to be failing for some joblib threading thing on windows, just rerunning them should be okay.

@llorracc
Copy link
Collaborator

llorracc commented Feb 9, 2021

Haven't we had that problem several times before?

@sbenthall sbenthall merged commit a9f10c3 into econ-ark:master Feb 11, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants