-
-
Notifications
You must be signed in to change notification settings - Fork 55
Refine optimal savings lectures with improved clarity and code organization #754
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
Conversation
…organization - Enhance code formatting and comments for better readability across all OS lectures - Improve mathematical notation and explanations in stochastic optimal savings - Restructure function definitions in os_egm_jax for better logical flow - Simplify utility function and Bellman operator implementations - Add clearer documentation of marginal utility approximations in EGM - Remove redundant code and improve variable naming throughout 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
…ture Rename compute_expectation_k to expected_mu for clarity and brevity. Streamline inline comments to improve code readability. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
|
This PR includes refinements to the optimal savings lecture series to improve code clarity and educational value. The changes focus on:
All lectures build successfully and the mathematical content remains unchanged. These are purely pedagogical improvements to make the code easier to follow for students. |
|
@jstac this seems to be hanging in a queue. You may want to merge the latest main in to trigger a fresh run. |
|
@jstac just merged in |
- Rename 'grid' to 'x_grid' throughout for consistency and clarity - Unify parameter name from 'og' to 'model' in solve_model function - Add type hints and inline comments to solve_model parameters - Fix unpacking error in B function to match Model NamedTuple fields - Resolves notebook execution error in remote build 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
|
Fixed the remote build error in
Changes made:
Testing:
The remote build should now pass. |
|
📖 Netlify Preview Ready! Preview URL: https://pr-754--sunny-cactus-210e3e.netlify.app (222e7c0) 📚 Changed Lecture Pages: ifp_egm_transient_shocks, os_egm, os_egm_jax, os_numerical, os_stochastic |
|
Thanks @mmcky , all help much appreciated |
Summary
Changes
Optimal Savings Lectures (os_*.md)
IFP Transient Shocks Lecture
compute_expectation_ktoexpected_mufor clarity and brevityTest plan
jb build lectures🤖 Generated with Claude Code