hank_shock docstring: -0.0025 is a 100bp-annualised move, not '25bp'; productivity 0.01 is 2.1% TFP, not 1%

Aperta Adatta ai principianti
  • Qualcosa non va?
#211 1 commento 0 reazioni 0 assegnatari Vedi su GitHub

I maintainer di solito rispondono entro 9 giorni

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà
1/5
Tempo stimato
1-3 ore
Idoneità per principianti
76/100
Tipo di issue
Documentazione
Chiarezza
Specificata chiaramente
Stato di attività
Attiva
Stack tecnologico
python
Ambito
documentation

Direzione di ricerca

Leggi integration/src/policyengine_macro/mcp_server.py intorno alla riga 372 e economy/topics.py intorno alla riga 1959, poi controlla la formulazione corrispondente in core.py e hank_summary. Aggiorna le descrizioni delle unità monetarie e di produttività nelle due posizioni indicate dall’issue; i campi di risultato annualizzati suggeriti sono facoltativi. Il lavoro è concluso quando le descrizioni corrispondono alle unità indicate dal modello e ai valori di esempio nell’issue.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

bug documentation

What

The hosted hank_shock docstring gives two unit translations that the model repo contradicts.

Where (main 153cf41) Says Model repo / hank_summary says
integration/src/policyengine_macro/mcp_server.py:372 monetary -0.0025 = a 25bp easing core.py:888 / hank_summary: "25bp-per-quarter". us-hank-model README: 0.01 = +100bp per quarter. So −0.0025 is −100bp annualised
economy/topics.py:1959 --size -0.0025 ... "a 25bp easing" same as above
mcp_server.py:374 productivity 0.01 ~ 1% TFP hank_summary and the README: Z_ss = 0.468, so 0.01 is a 2.14% TFP rise ("wrong by more than a factor of two")

Evidence

hank_shock(kind="monetary", size=0.0025) returns r = +0.34pp per quarter on impact (+1.36pp annualised) and pi = −0.336pp per quarter (−1.34pp annualised). Anyone who reads "25bp" in the usual annualised sense and compares with FRB/US (rffintay_aerr=0.25) or the BoE overstates the HANK shock 4×.

Fix

Change both strings to "−0.0025 = −25bp per quarter (≈ −100bp annualised); for a 25bp annualised move use −0.000625". Change TFP to "0.01 ≈ 2.1% TFP". Consider adding pi_annualised / r_annualised to the result. Docs-only. The repo's own text in core.py and hank_summary is already right.

Found in the cross-model battery (comparison-view issue linked once filed).

🤖 Generated with Claude Code

Lingua principale
HTML
Stelle
3
Fork
0
Merge medio
8g 15h
PR unite (30g)
1

Preparare l'ambiente

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di PolicyEngine/macro

Tutte le issue di PolicyEngine/macro

Issue simili

Altre issue su Documentation

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.