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

Aberta Para iniciantes
  • Algo errado?
#211 1 comentário 0 reações 0 responsáveis Ver no GitHub

Mantenedores costumam responder em até 5 dias

Ninguém assumiu esta issue ainda.

Avaliação

Dificuldade
1/5
Tempo estimado
1-3 horas
Facilidade para iniciantes
76/100
Tipo de issue
Documentação
Clareza
Claramente especificada
Status de atividade
Ativa
Stack de tecnologia
python
Domínio
documentation

Direção de pesquisa

Leia integration/src/policyengine_macro/mcp_server.py por volta da linha 372 e economy/topics.py por volta da linha 1959; em seguida, confira a redação correspondente em core.py e hank_summary. Atualize as descrições das unidades monetárias e de produtividade nos dois locais indicados pela issue; os campos de resultado anualizados sugeridos são opcionais. O trabalho estará concluído quando as descrições corresponderem às unidades declaradas pelo modelo e aos valores de exemplo da issue.

Escrita pelo modelo de indexação a partir do texto da issue.

Descrição

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

Linguagem predominante
HTML
Estrelas
3
Forks
0
Merge médio
4d 6h
PRs com merge (30d)
15

Preparar o ambiente

Primeiros passos

  1. Leia a issue inteira e depois o guia de contribuição do projeto.
  2. Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
  3. Faça um fork do repositório e trabalhe em uma branch.
  4. Abra um pull request que referencie o número da issue.

Mais de PolicyEngine/macro

Todas as issues de PolicyEngine/macro

Issues semelhantes

Mais issues de Documentation

Receba novas issues na sua caixa de entrada

Um resumo curto de issues do GitHub para quem está começando.