mccall_model_with_sep_markov#
Series: lecture-dp
File:
lectures/mccall_model_with_sep_markov.mdAudit date: 2026-05-28
Categories audited: writing, math, code, jax, figures, references, links, admonitions
Overall score: 8.2 / 10
Priority: LOW
Score breakdown#
Category |
Score |
One-line note |
|---|---|---|
Writing |
8/10 |
Sentence-case headings; one “iid” slip. |
Math |
9/10 |
|
Code |
7/10 |
Unicode Greek; pip install at top; PEP8; figsize 5×. |
JAX |
out of scope |
uses JAX, but JAX rules out of scope. |
Figures |
7/10 |
One |
References |
N/A |
No |
Links |
9/10 |
|
Admonitions |
9/10 |
Exercise + solution-start with dropdown. |
Issues#
Critical#
None found.
High severity#
None found.
Medium severity#
[qe-writing-009 (proposed)] — “iid” used in narrative on line 713. Should be “IID”.
[qe-fig-003] — One f-string
ax.set_title(f'Cross-sectional distribution at t={t_snapshot}, ...')(line 880).[qe-fig-001] —
figsize=set 5 times.
Low severity#
[qe-writing-001] — A few multi-sentence paragraphs (e.g. 43-47, 700-706, 720-726).
Strengths#
Lecture title correctly Title Case.
All H2/H3 headings sentence case.
\mathbb{1}used for the indicator (line 734).No transpose violations; no bold vectors; equation labels and references clean.
“IID” correctly capitalised elsewhere (e.g. line 41, line 100).
pip install at top; Unicode Greek in code.
{doc}used for cross-series.Exercise uses
{exercise}+solution-startwith:class: dropdown.
Recommended actions#
Change “iid” → “IID” on line 713.
Remove the f-string
ax.set_title(...)call.Reduce unnecessary
figsize=usage.Tighten the few multi-sentence paragraphs.