calvo.md and calvo_machine_learn.md: $L^2$ definition removed but notation still used
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 3/5
- Tempo stimato
- 1-2 giorni
- Idoneità per principianti
- 68/100
- Tipo di issue
- Documentazione
- Chiarezza
- Abbastanza chiara
- Stato di attività
- Tranquilla
- Stack tecnologico
- latex, markdown
- Ambito
- documentation
Direzione di ricerca
Inizia con le sezioni interessate in lectures/calvo.md e lectures/calvo_machine_learn.md, quindi esamina PR #312 per capire perché la definizione di L^2 è stata rimossa. Decidi se l’ipotesi prevista sia una definizione ripristinata, successioni limitate o un collegamento incrociato, e applica tale scelta a tutte le occorrenze elencate. Controlla il testo della lezione renderizzato per confermare che non rimangano riferimenti L^2 non definiti.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
Summary
PR #312 ("[amss] Update Tom's edits to AMSS and Calvo Lectures") removed the local definition of square-summable / $L^2$ from the three Calvo lectures, but several places still use the $L^2$ notation without a definition, leaving readers without context for what space is being referenced.
Locations still using L^2 after the definition was removed
lectures/calvo.md — 7 occurrences
| Line | Context |
|---|---|
| 330 | $\vec \mu =\{\mu_t\}_{t=0}^\infty \in L^2$ into an inflation sequence |
| 331 | $\vec \theta = \{\theta_t\}_{t=0}^\infty \in L^2$. |
| 334 | $\vec v = \{v_t\}_{t=0}^\infty \in L^2$ that satisfies |
| 340 | sequence $\vec \mu \in L^2$. |
| 342 | At this point $\vec \mu \in L^2$ is an arbitrary exogenous policy. |
| 453 | $\{x_{t+1}, \mu_t \}_{t=0}^\infty$ sequences, with each sequence belonging to $L^2$: |
| 460 | where we require that $\{x_{t+1}, \mu_t \}_{t=0}^\infty \in L^2 \times L^2 .$ |
lectures/calvo_machine_learn.md — 1 occurrence
| Line | Context |
|---|---|
| 208 | \vec \theta \in L^2 |
lectures/calvo_abreu.md — clean
calvo_abreu.md had the same definition removed and no remaining $L^2$ references, so it's consistent.
Why it matters
Earlier in each affected lecture, the assumption on $\vec\mu$ was weakened from "square-summable" to "bounded" (a reasonable simplification — the discounted geometric structure of the forward solution is well-defined under boundedness). But the later sections that still use $L^2$ now reference an undefined space. A reader going top-to-bottom encounters $\vec\mu \in L^2$ at line 330+ in calvo.md with no prior definition or external pointer.
Suggested fix
Either:
- Reintroduce a brief $L^2$ definition in each of the two affected lectures (could be a one-liner: "$L^2$ denotes the space of square-summable sequences"), or
- Replace the remaining $L^2$ references with the weaker "bounded" framing if that's now the intended assumption end-to-end, or
- Cross-link to a single canonical definition in one of these lectures and have the others reference it.
Context
Surfaced by GitHub Copilot's PR review during sync of these lectures into lecture-dp (QuantEcon/lecture-dp#11). The PR is byte-identical to this repo's current main, so this issue belongs upstream.
🤖 Generated with Claude Code
- Lingua principale
- TeX
- Stelle
- 37
- Fork
- 26
- Merge medio
- 2g 17h
- PR unite (30g)
- 2
Preparare l'ambiente
Non abbiamo ancora controllato i file di configurazione di questo progetto. Parti dal suo README e consulta la nostra guida al primo contributo per i passaggi generali.
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di QuantEcon/lecture-python-advanced.myst
-
Clean up pre-existing deprecation warnings (matplotlib scatter marker, plotly kaleido engine)Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 84/100
-
Difficoltà 3/5 1-2 giorni Idoneità per principianti 72/100
-
broken-links bug documentation
Difficoltà 4/5 3-5 giorni Idoneità per principianti 42/100
-
Link Checker ReportApertaautomated issue linkchecker report
Difficoltà 3/5 1-2 giorni Idoneità per principianti 52/100
-
Link Checker ReportApertaautomated issue linkchecker report
Difficoltà 3/5 1-2 giorni Idoneità per principianti 55/100
Tutte le issue di QuantEcon/lecture-python-advanced.myst
Issue simili
-
sync-en
Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100
I maintainer di solito rispondono entro 1 giorno
-
community-request
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 95/100
NVIDIA-NeMo/Curator#2464 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
good first issue hacktoberfest
Difficoltà 2/5 1-3 ore Idoneità per principianti 87/100
elnachto/laya-triage#5 ·
-
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 88/100
I maintainer di solito rispondono entro 1 giorno
-
[integration]: InferrailApertaexternal
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
langchain-ai/docs#6328 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno