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

Ouverte Adaptée aux débutants
  • Quelque chose ne va pas ?
#211 1 commentaire 0 réactions 0 personnes assignées Voir sur GitHub

Les mainteneurs répondent en général sous 5 jours

Personne n'a encore pris cette issue.

Évaluation

Difficulté
1/5
Temps estimé
1-3 heures
Accessibilité débutants
76/100
Type d'issue
Documentation
Clarté
Clairement spécifiée
Activité
Active
Stack technique
python
Domaine
documentation

Piste de recherche

Lisez integration/src/policyengine_macro/mcp_server.py autour de la ligne 372 et economy/topics.py autour de la ligne 1959, puis vérifiez la formulation correspondante dans core.py et hank_summary. Mettez à jour les descriptions des unités monétaires et de productivité aux deux emplacements indiqués par l’issue ; les champs de résultat annualisés suggérés sont facultatifs. Le travail est terminé lorsque les descriptions correspondent aux unités indiquées par le modèle et aux valeurs d’exemple de l’issue.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Description

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

Langage dominant
HTML
Étoiles
3
Forks
0
Merge moyen
4 j 6 h
PR mergées (30 j)
15

Préparer son environnement

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Autres issues de PolicyEngine/macro

Toutes les issues de PolicyEngine/macro

Issues similaires

Plus d'issues Documentation

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.