-
-
Notifications
You must be signed in to change notification settings - Fork 199
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
Make non-OO consumption-saving solvers #1394
Merged
Merged
Changes from 1 commit
Commits
Show all changes
58 commits
Select commit
Hold shift + click to select a range
35c63b1
Add simplified PF solver
mnwhite 3ed2e45
Add simplified based ConsIndShockSolver
mnwhite a3b3c78
Add cubic option to simplified ConsIndShock solver
mnwhite a3f3a6c
Add vFuncBool capability to simplified solver
mnwhite 34b7555
Run black on simplified solver
mnwhite 3d92dd1
Add simplified ConsKinkedR solver
mnwhite ab4b086
Run black again
mnwhite 8faf1a1
Fix error in MPCmax in KinkedR
mnwhite 8c4e060
Actually fix error properly
mnwhite f245acb
Discrepancy actually due to indexing offset error
mnwhite 747eeab
Made basic ConsPortfolio simplified solver
mnwhite 45d645c
Add vFunc capability to simplified portfolio solver
mnwhite d9cda03
Run black on ConsPortfolioModel.py
mnwhite 830764b
Add discrete choice capability to simplified portfolio solver
mnwhite 264186a
Run black, move import line
mnwhite 936930a
Merge branch 'master' into SimplifySolvers
mnwhite fb10736
Add simple solver for ConsPrefShock
mnwhite f592a98
More formatting nonsense
mnwhite 0dd9ab3
Make simple KinkyPrefShock solver
mnwhite 5c6a5a5
Actually use new KinkyPref solver
mnwhite 491a6f6
Make simple solver for ConsMarkovModel
mnwhite 8d3cc8d
Fix formatting error and cubic interpolation typo
mnwhite 00861e6
Added vFunc capability to simple ConsMarkov solver
mnwhite c6f8fc1
Resolve vFunc bug by suppressing extrapolation limit
mnwhite ea15b11
Delete two blank lines that upset ruff
mnwhite 056cbd6
Add simple solver for WarmGlowBequest
mnwhite 1568878
Add simple solver for WarmGlowPortfolioModel
mnwhite 753df28
Actually use new solver
mnwhite ccbdd76
Missed one unneeded import
mnwhite f33f16d
Add simple solver for ConsGenIncProcess
mnwhite 9e014f0
Remove unneeded imports
mnwhite 8b759b4
First version of simple ConsRiskyAsset solver
mnwhite 0a94ca7
Value function works for ConsRiskyAsset
mnwhite 28afed2
Add independent distribution capability
mnwhite 1c3b683
Fixed Cubic interpolation issue
mnwhite 7b4da9c
Forgot to re-activate new solver
mnwhite bbdc412
Add simple solver for ConsMedShockModel
mnwhite f275116
Add vFunc and cubic capability to WarmGlowBequest
mnwhite a4105a3
Non-independent distributions for ConsPortfolioModel
mnwhite cf41a10
I guess I mistakenly deleted a carriage return?
mnwhite 260a3f1
Now there was an extra tab
mnwhite ccf9c8a
Update CHANGELOG
mnwhite 561872e
Begin moving OO solvers out of model files
mnwhite 6f9fc02
Move more solvers, try to resolve some testing issues
mnwhite 20352e6
Even more solvers moved, trying to resolve test conflicts
mnwhite 715f845
Move more legacy code, fix more imports
mnwhite c0f76bb
Fix IndShockFast imports
mnwhite e2c38fb
Remove ConsPortfolioJointDistSolver from tests
mnwhite 964894c
Add simple solver for alternate ConsPortfolio
mnwhite eb96ab7
Merge branch 'master' into SimplifySolvers
mnwhite 736b424
Formatting fixes
mnwhite 765be38
Two tiny formatting fixes
mnwhite c1210e2
And again
mnwhite 27c01ab
Restore fixed share type to ConsRiskyAssetModel
mnwhite 4e0436c
Change import location
mnwhite 428ddd5
Even more formatting
mnwhite 18c8f70
Lint loves linebreaks
mnwhite b4583a8
Update example notebooks
mnwhite File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think at some point we were saving some midway calculations for diagnosing issues?
We should probably remove these, unless they are somehow important downstream for @Mv77