frbus_shock: stacking warning only on monetary levers (trp_aerr periods=4 gives +2.37pp, not +1pp); poil_aerr labelled 'model units'

Abierto
  • ¿Algo no está bien?
#212 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
4/5
Tiempo estimado
3-5 días
Aptitud para principiantes
48/100
Tipo de issue
Error
Claridad
Bastante claro
Estado de actividad
Activo
Stack tecnológico
python
Área
api

Línea de trabajo

Start with integration/src/policyengine_macro/core.py, especially FRBUS_VARIABLES and the fallback near line 730; compare the lever unit strings with the period-stacking guidance in us-frb-model/VALIDATION.md. Check the hosted calls cited in the issue against the documented behavior, then update the affected lever descriptions and poil_aerr units. Done means the exposed levers accurately describe stacking and oil-price units; the optional exogenize mode is a broader follow-up.

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

Descripción

bug

What

frbus_shock warns that add-factors stack over periods only for the two monetary levers. Every other lever's units string is silent, and poil_aerr returns "units": "model units". us-frb-model/VALIDATION.md ("Period stacking") says: "Any wrapper exposing a periods argument must document this on every lever it exposes."

Evidence (hosted, us-frb-model@2508a2d, 6 Nov 2026)

Call Label returned What happened
frbus_shock(var="trp_aerr", shock=0.01, periods=4) "0.01 = a 1 percentage point rise in the effective personal tax rate" trp +6.6% → +15.5% of baseline by q4, i.e. +2.37pp, not +1pp. GDP −0.39% vs −0.115% for periods=1
frbus_shock(var="egfe_aerr", shock=0.01, periods=4) (per VALIDATION.md) "0.01 ~ a 1% higher level" level +2.90% by q4, not +1%
frbus_shock(var="poil_aerr", shock=10, periods=4) "model units" the units are $/bbl: +10 gives poil +15.4%. It does not stack (level equation)

Also: the validated sustained +1%-of-GDP purchases experiment (egfe exogenised, VALIDATION.md Test 4, yr-1 multiplier 0.725) cannot be reached through the hosted tool. The nearest call is a one-quarter impulse.

Fix

In integration/src/policyengine_macro/core.py (FRBUS_VARIABLES, ~l.392–450, fallback at l.730):

  1. Add the stacking sentence to egfe_aerr, egse_aerr, trp_aerr, trci_aerr, ecnia_aerr (and any other _aerr lever whose equation carries persistence). Better still, return the realised lever path (variables=[<lever>] by default), so callers see what they got.
  2. Give poil_aerr a real unit ("$/barrel add-error on the oil price level; +10 ≈ +15% at the 2026 LONGBASE price").
  3. Optional: an exogenize mode for egfe/trp, so the Test 4 experiment is reachable hosted.

🤖 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 Backend & API Design

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.