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

Abierto Apto para principiantes
  • ¿Algo no está bien?
#211 1 comentario 0 reacciones 0 asignados Ver en GitHub

Los mantenedores suelen responder en 9 días

Nadie ha tomado este issue todavía.

Evaluación

Dificultad
1/5
Tiempo estimado
1-3 horas
Aptitud para principiantes
76/100
Tipo de issue
Documentación
Claridad
Bien especificado
Estado de actividad
Activo
Stack tecnológico
python
Área
documentation

Línea de trabajo

Lee integration/src/policyengine_macro/mcp_server.py alrededor de la línea 372 y economy/topics.py alrededor de la línea 1959; después, comprueba la redacción correspondiente en core.py y hank_summary. Actualiza las descripciones de las unidades monetarias y de productividad en las dos ubicaciones indicadas por el issue; los campos de resultado anualizados sugeridos son opcionales. El trabajo estará hecho cuando las descripciones coincidan con las unidades indicadas por el modelo y con los valores de ejemplo del issue.

Escrito por el modelo de indexación a partir del texto del issue.

Descripción

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

Lenguaje dominante
HTML
Estrellas
3
Forks
0
Merge medio
8 d 15 h
PR fusionados (30 d)
1

Preparar el entorno

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Más de PolicyEngine/macro

Todos los issues de PolicyEngine/macro

Issues similares

Más issues de Documentation

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.