Skip to content

SYNC: add inventory_q.md from lecture-python.myst - #19

Merged
mmcky merged 2 commits into
mainfrom
sync-inventory-q-from-lecture-python
May 28, 2026
Merged

mmcky merged 2 commits into
mainfrom
sync-inventory-q-from-lecture-python

Conversation

@mmcky

@mmcky mmcky commented May 28, 2026

Copy link
Copy Markdown
Contributor

Summary

Follow-ups

Test plan

  • CI passes (file is outside the TOC, so build should be unaffected).
  • Diff against upstream is a clean copy with no unintended modifications.

🤖 Generated with Claude Code

Mirrors lectures/inventory_q.md from QuantEcon/lecture-python.myst at upstream commit d825b9236 (#830, optimistic initialization update; originally added in #827). File is not yet wired into _toc.yml, so it will not build until the TOC PR for #4 lands. Preparation step for the new "Markov Decision Processes" section proposed in #4.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings May 28, 2026 07:08

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR adds a new lecture page, lectures/inventory_q.md, introducing an inventory management model solved via dynamic programming and then via Q-learning, as part of the planned new Markov Decision Processes section.

Changes:

  • Adds the new MyST/Jupytext-synced lecture inventory_q.md.
  • Implements VFI and Q-learning code examples (Numba-accelerated) and accompanying figures/text.
  • Introduces new cross-references/citations that will need to resolve cleanly once the page is included in the TOC.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread lectures/inventory_q.md
Comment thread lectures/inventory_q.md
Comment thread lectures/inventory_q.md
Comment thread lectures/inventory_q.md
Adds the missing BibTeX entry for the {cite} reference on line 180 of inventory_q.md (flagged by Copilot review on this PR). Entry copied verbatim from QuantEcon/lecture-python.myst/lectures/_static/quant-econ.bib so the citation resolves once the file enters the build via the upcoming TOC PR for #4.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
mmcky added a commit that referenced this pull request May 28, 2026
Mirrors lectures/rs_inventory_q.md from QuantEcon/lecture-python.myst at upstream commit 78030a3a2 (latest commit touching this file, via PR QuantEcon/lecture-python.myst#859). File contains 10 {doc} references to inventory_q (added in #19) and no {cite} keys. Not yet wired into _toc.yml; activated by the TOC PR for #4.

Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@mmcky
mmcky merged commit 6a7bc1c into main May 28, 2026
1 check passed
@mmcky
mmcky deleted the sync-inventory-q-from-lecture-python branch May 28, 2026 07:28
mmcky added a commit that referenced this pull request Jun 21, 2026
Implements the lecture reorder agreed in #4:

- Removes discrete_dp from Introduction (Introduction now contains only short_path).
- Removes mccall_q from Job Search (now 8 lectures).
- Adds a new "Markov Decision Processes" section between Income Fluctuation Problems and LQ Control, containing discrete_dp, inventory_q, rs_inventory_q, mccall_q.

discrete_dp is the theoretical foundation (MDP setup) that the Q-learning lectures build on, so they sit together; this also fixes the difficulty curve in Introduction (per @jstac's note that discrete_dp was much more advanced than the lectures that followed). Files inventory_q.md and rs_inventory_q.md were added to the repo in #19 and #20 respectively but were not yet wired into the build (only_build_toc_files: true); this commit activates them.

Closes #4

Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
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.

TASK: reorder lectures

2 participants