Accept the withheld-area code lists on UK local-area breakdown results before any executor sends them
Maintainer antworten meist innerhalb von 1 Tag
Bewertung
- Schwierigkeit
- 2/5
- Geschätzter Aufwand
- 1-3 Stunden
- Anfängerfreundlichkeit
- 67/100
Rechercherichtung
Lies den Vertrag und die Gateway-Golden-OpenAPI-Spezifikation, auf die im Issue verwiesen wird, und untersuche anschließend die Serialisierung des Ergebnisses der Aufschlüsselung nach lokalen Gebieten. Füge dem Vertrag und der OpenAPI-Spezifikation die beiden optionalen Felder für zurückgehaltene Codes hinzu. Stelle sicher, dass Ergebnisse ohne Listen beide Schlüssel auslassen, während leere Listen weiterhin enthalten sind. Im Issue werden weder Tests noch Dateipfade genannt. Ermittle daher anhand der Vertragsdefinitionen die relevanten Serialisierungstests.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Beschreibung
Part of #968.
Problem
#968 adds two optional fields to a UK local-area breakdown result (kind: local_areas): constituency_withheld_codes and local_authority_withheld_codes.
A push to main deploys the versioned executor before the stable gateway: in simulation-deploy.reusable.yml, deploy_gateway needs deploy_executor. The deployed gateway's contract rejects unknown keys. If one deploy both added the fields to the contract and had the executor fill them, the gateway would report every breakdown that finished between those two steps as failed.
What should happen
Deploy the change in two steps:
- The contract and the gateway accept the two fields, and no executor sends them. The executor redeployed in this step carries the new contract while the old gateway still serves, so a result that doesn't report the lists must leave both keys out of its JSON rather than write
null. - The executor fills them, in #975, once step 1 is deployed.
Done when
- The contract has both optional fields, and the gateway's golden OpenAPI lists them.
- A breakdown result without the lists dumps with the same keys as before.
- An empty list (
[]) is kept in the dump.
- Vorherrschende Sprache
- Python
- Sterne
- 1
- Forks
- 1
- Ø Merge
- 1 T. 19 Std.
- Gemergte PRs (30 T.)
- 77
Entwicklungsumgebung
- Kein Dockerfile und keine Docker-Compose-Datei
- Hat eine Pull-Request-Vorlage
- Beitragsleitfaden lesen
Erste Schritte
- Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
- Forken Sie das Repository und arbeiten Sie in einem Branch.
- Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.
Mehr aus PolicyEngine/policyengine-sim-api
-
Update policyengine to 6.11.1Evtl. vergeben @policyengine hat das vor 2 Tagen übernommen. Offen
Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 68/100
PolicyEngine/policyengine-sim-api#916 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Update policyengine to 6.7.3Evtl. wieder frei Ein Pull Request dazu wurde ohne Merge geschlossen. Offen
Schwierigkeit 2/5 Ein halber Tag Anfängerfreundlichkeit 70/100
PolicyEngine/policyengine-sim-api#823 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Integration tests fail due to missing UV virtual environment setupEvtl. vergeben Ein verknüpfter Pull Request ist offen oder bereits gemergt. Offen
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 65/100
PolicyEngine/policyengine-sim-api#314 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Move the changelog to Towncrier fragments and build CHANGELOG.md at each deployEvtl. vergeben @anth-volk hat das vor 1 Tag übernommen. Offen
Schwierigkeit 5/5 Über eine Woche Anfängerfreundlichkeit 25/100
PolicyEngine/policyengine-sim-api#978 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Run whole economy reports on v2 in the Simulation EntrypointEvtl. vergeben Ein verknüpfter Pull Request ist offen oder bereits gemergt. Offen
Schwierigkeit 5/5 Über eine Woche Anfängerfreundlichkeit 25/100
PolicyEngine/policyengine-sim-api#973 ·
Maintainer antworten meist innerhalb von 1 Tag
Alle Issues in PolicyEngine/policyengine-sim-api
Ähnliche Issues
-
factory
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 78/100
Arize-ai/phoenix#17420 · 1 Kommentar · 1 Reaktion ·
Maintainer antworten meist innerhalb von 1 Tag
-
bug
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 84/100
FireDynamics/LEDSmokeAnalysis#176 ·
Maintainer antworten meist innerhalb von 1 Tag
-
PC
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 74/100
Maintainer antworten meist innerhalb von 1 Tag
-
Make `ocifs` optionalOffendependencies
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 68/100
dlt-contrib/dlt-filesystem#51 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 68/100
Maintainer antworten meist innerhalb von 1 Tag