Skip to content

Conversation

@jstac
Copy link
Contributor

@jstac jstac commented Dec 2, 2025

Summary

  • Fixed inconsistent code-cell language specifications in optimal savings lectures that caused jupytext UserWarnings
  • Standardized language specs within each notebook for clean conversion

Changes

  • os_egm_jax.md: Changed ipythonpython3 (line 42 import cell had no IPython magics)
  • os_stochastic.md: Changed ipythonpython3 (line 65 import cell had no IPython magics)
  • os_time_iter.md: Changed all cells to ipython for consistency (uses !pip install magic at line 28)

Testing

  • Converted all modified files with jupytext - no UserWarnings
  • Executed all generated .py files - all run successfully
  • Verified no functional changes, only language specification corrections

🤖 Generated with Claude Code

jstac and others added 2 commits December 2, 2025 11:51
… lectures

Standardize code-cell language specifications to eliminate jupytext UserWarnings:
- os_egm_jax.md: Change ipython to python3 (no IPython magics used)
- os_stochastic.md: Change ipython to python3 (no IPython magics used)
- os_time_iter.md: Change all cells to ipython (uses !pip install magic)

These changes ensure consistent language specifications within each notebook,
resolving warnings during jupytext conversion while maintaining functionality.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
@jstac
Copy link
Contributor Author

jstac commented Dec 2, 2025

All files from the last commit (1cb2924) were converted to Python using jupytext and tested successfully:

os_egm.py - runs in ~0.03s
os_egm_jax.py - runs in ~0.009s
os_stochastic.py - runs successfully (takes ~1+ minute, converges in 229 and 237 iterations)
os_time_iter.py - runs successfully (converges in 13-15 iterations)

All UserWarnings from jupytext have been eliminated with these language specification fixes.

@github-actions
Copy link

github-actions bot commented Dec 2, 2025

📖 Netlify Preview Ready!

Preview URL: https://pr-756--sunny-cactus-210e3e.netlify.app (81c01fc)

📚 Changed Lecture Pages: os_egm, os_egm_jax, os_stochastic, os_time_iter

@jstac jstac merged commit 5efda54 into main Dec 2, 2025
1 check passed
@jstac jstac deleted the sfix branch December 2, 2025 03:36
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.

2 participants