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

Add liquidity constraint to perfect foresight model #35

Closed
wants to merge 2 commits into from

Conversation

mnwhite
Copy link
Collaborator

@mnwhite mnwhite commented Jun 2, 2019

This PR updates model notebooks to account for the existence of BoroCnstArt in the perfect foresight model. It should only be merged after this feature has been included in a public release of HARK, otherwise the notebook will break; likewise, it should be merged immediately otherwise the notebook will break.

mnwhite added 2 commits June 2, 2019 15:53
Perfect foresight model will shortly have BoroCnstArt enabled; this commit fixes up the notebook for PerfForesightConsumerType to account for impending changes.
Changed order of parameters in dictionary to be consistent across notebooks.
@alanlujan91
Copy link
Member

assigned

@MridulS
Copy link
Member

MridulS commented Aug 11, 2020

KinkedRconsumerType.ipynb and PerfForesightConsumerType.ipynb now are part of HARK/examples/ConsIndShockModel so the changes of this PR should be migrated there.

@alanlujan91
Copy link
Member

These changes have been migrated to HARK. See econ-ark/HARK#795

@MridulS MridulS closed this Aug 12, 2020
@llorracc llorracc deleted the AddLiqConstrToPF branch October 20, 2021 19:10
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