Accept the withheld-area code lists on UK local-area breakdown results before any executor sends them
Los mantenedores suelen responder en 1 día
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 67/100
Línea de trabajo
Lee el contrato y la especificación OpenAPI dorada de gateway a los que hace referencia la incidencia y, después, inspecciona la serialización del resultado del desglose por área local. Añade al contrato y a OpenAPI los dos campos opcionales de códigos retenidos. Verifica que los resultados sin listas omitan ambas claves, mientras que las listas vacías sigan presentes. La incidencia no indica pruebas ni rutas de archivos, así que localiza las pruebas de serialización pertinentes a partir de las definiciones del contrato.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
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.
- Lenguaje dominante
- Python
- Estrellas
- 1
- Forks
- 1
- Merge medio
- 1 d 15 h
- PR fusionados (30 d)
- 78
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Tiene una plantilla de pull request
- Leer la guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de PolicyEngine/policyengine-sim-api
-
Update policyengine to 6.7.3Quizá libre de nuevo Un pull request para esta issue se cerró sin fusionarse. Abierto
Dificultad 2/5 Medio día Aptitud para principiantes 70/100
PolicyEngine/policyengine-sim-api#823 ·
Los mantenedores suelen responder en 1 día
-
Integration tests fail due to missing UV virtual environment setupPosiblemente ocupada Un pull request vinculado a esta issue está abierto o ya se fusionó. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
PolicyEngine/policyengine-sim-api#314 ·
Los mantenedores suelen responder en 1 día
-
Calculate budget windows on the v2 economy pathPosiblemente ocupada Un pull request vinculado a esta issue está abierto o ya se fusionó. Abierto
Dificultad 5/5 Más de una semana Aptitud para principiantes 35/100
PolicyEngine/policyengine-sim-api#981 ·
Los mantenedores suelen responder en 1 día
-
Run whole economy reports on v2 in the Simulation EntrypointPosiblemente ocupada Un pull request vinculado a esta issue está abierto o ya se fusionó. Abierto
Dificultad 5/5 Más de una semana Aptitud para principiantes 25/100
PolicyEngine/policyengine-sim-api#973 ·
Los mantenedores suelen responder en 1 día
-
Return a client error for simulation years the model cannot project its dataset toPosiblemente ocupada @anth-volk la tomó hace 3 días. Abierto
Dificultad 3/5 1-2 días Aptitud para principiantes 68/100
PolicyEngine/policyengine-sim-api#969 ·
Los mantenedores suelen responder en 1 día
Todos los issues de PolicyEngine/policyengine-sim-api
Issues similares
-
[oblt-aw][autodoc] Document new required `shared-org-key` input for `obs-aw-dependency-review`Posiblemente ocupada @github-actions la tomó hoy. Abierto
Dificultad 1/5 Menos de una hora Aptitud para principiantes 90/100
Los mantenedores suelen responder en 1 día
-
getchaintips leaves out the active tip once a header is staged on it, where Core always lists itAbierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 88/100
btclib-org/btclib-node#2158 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 85/100
dbt-msft/dbt-sqlserver#897 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
Gathin23/cleanyourtv#2770 · 2 comentarios ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
Dao-AILab/flash-attention#3097 ·
Los mantenedores suelen responder en 3 días