Remove household_options from the country metadata

Geschlossen
  • Stimmt etwas nicht?
#4,209 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Maintainer antworten meist innerhalb von 2 Tagen

Ein zugehöriger Pull Request wurde bereits gemerged.

  • #4210 von @anth-volk — gemerged

Bewertung

Schwierigkeit
3/5
Geschätzter Aufwand
1-2 Tage
Anfängerfreundlichkeit
35/100
Issue-Typ
Refactoring
Klarheit
Klar beschrieben
Aktivitätsstatus
Veraltet
Tech-Stack
python
Bereich
api

Rechercherichtung

Start with policyengine_api/country.py, policyengine_api/household_years.py, and policyengine_api/spm.py, then inspect the tests for build_household_options and the metadata response. Remove the listed household-specific options and their tests; done means /{country}/metadata no longer includes household_options, while the follow-up change handles the shared country years. Pull request #4210 is already open for this issue.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

Problem

#4159 added household_options.time_period to the /{country}/metadata response: a list of years for household calculations, separate from economy_options.time_period. No client reads it.

The maintainer decided on 4–5 November 2026 that household and society-wide calculations share one list of years per country. A separate household list is therefore no longer needed.

Change

Remove what #4159 added:

  • household_options in the /{country}/metadata response, built by build_household_options in policyengine_api/country.py;
  • policyengine_api/household_years.py;
  • certified_spm_forecast_years in policyengine_api/spm.py, which only build_household_options used;
  • their tests.

A follow-up change defines each country's years once and uses them for economy_options.time_period.

Vorherrschende Sprache
Python
Sterne
18
Forks
33
Ø Merge
2 T. 15 Std.
Gemergte PRs (30 T.)
110

Entwicklungsumgebung

Erste Schritte

  1. Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
  3. Forken Sie das Repository und arbeiten Sie in einem Branch.
  4. Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.

Mehr aus PolicyEngine/policyengine-api

Alle Issues in PolicyEngine/policyengine-api

Ähnliche Issues

Weitere Issues zu Python

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.