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

オープン 初心者向け
#211 コメント 1 件 リアクション 0 件 担当者 0 名 GitHub で見る

メンテナーはふだん 9 日以内に返信

まだ誰も着手していません。

評価

難易度
1/5
見積もり時間
1〜3時間
初心者へのやさしさ
76/100
issue の種類
ドキュメント
明瞭さ
明確に書かれている
活発さ
活発
技術スタック
python
領域
documentation

調査の方向性

integration/src/policyengine_macro/mcp_server.pyの372行目付近とeconomy/topics.pyの1959行目付近を読み、続いて参照されているcore.pyとhank_summaryの表現を確認してください。Issueで指定された2か所の金額と生産性の単位説明を更新してください。提案されている年率換算の結果フィールドは任意です。説明がモデルに記載された単位とIssueの例の値に一致すれば完了です。

索引モデルが issue の本文から書いたものです。

説明

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

主要言語
HTML
スター
3
フォーク
0
平均マージ
8日 15時間
マージ済み PR(30日)
1

環境構築

はじめの一歩

  1. issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
  2. 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
  3. リポジトリをフォークし、ブランチを切って変更します。
  4. issue 番号を参照したプルリクエストを送ります。

PolicyEngine/macro のほかの issue

PolicyEngine/macro の issue をすべて見る

似ている issue

Documentation の issue をもっと見る

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。