Drop the unused x and y columns from API v2's UK area impact tables
Los mantenedores suelen responder en 2 días
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 35/100
Línea de trabajo
Start with the SQLModel tables in policyengine_api/data/v2/models/reports.py and the baseline revision f5ef4347cb2a; check how v2 Alembic revisions are generated and how their downgrade functions are written. Remove x and y from both tables and add an autogenerated revision that drops those columns and restores them on downgrade. Done means the models and revision match the intended schema; deployment is scheduled after the UK Budget on 28 October 2026 and requires Anthony's approval.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
API v2's constituency_impacts and local_authority_impacts tables (the ConstituencyImpact and LocalAuthorityImpact SQLModel tables in policyengine_api/data/v2/models/reports.py) each have required x and y integer columns, created by the v2 baseline revision f5ef4347cb2a. They hold hex-map positions.
Nothing needs these values: the app draws its UK maps from its own layout files, and nothing in the Public API reads or writes either table yet. No managed v2 database role holds any privilege on them (policyengine_api/data/v2/privileges.py).
Intended result
The two tables have no x or y columns, removed by an autogenerated v2 Alembic revision whose downgrade adds them back, and the SQLModel tables match.
Deployment
Deploying the revision changes the staging and production v2 schemas. The merge needs Anthony's approval and waits until after the UK Budget on 28 October 2026.
- Lenguaje dominante
- Python
- Estrellas
- 18
- Forks
- 33
- Merge medio
- 2 d 13 h
- PR fusionados (30 d)
- 56
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-api
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
PolicyEngine/policyengine-api#4037 ·
Los mantenedores suelen responder en 2 días
-
Dificultad 1/5 1-3 horas Aptitud para principiantes 88/100
PolicyEngine/policyengine-api#3911 ·
Los mantenedores suelen responder en 2 días
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 82/100
PolicyEngine/policyengine-api#3910 ·
Los mantenedores suelen responder en 2 días
-
Serve economy reports on the v2 routes and save v2 report and simulation IDsPosiblemente 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-api#4107 ·
Los mantenedores suelen responder en 2 días
-
Relock dependenciesPosiblemente ocupada @policyengine la tomó hace 1 día. Abierto
Dificultad 1/5 Menos de una hora Aptitud para principiantes 1/100
PolicyEngine/policyengine-api#4100 ·
Los mantenedores suelen responder en 2 días
Todos los issues de PolicyEngine/policyengine-api
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
Graphify-Labs/graphify#4860 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
:bug: Bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 84/100
scikit-image/scikit-image#8377 ·
Los mantenedores suelen responder en 1 día
-
Ingredient import validation accepts negative nutrient values and NaNPosiblemente ocupada @LukeSekai la tomó hoy. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
wger-project/wger#2564 ·
Los mantenedores suelen responder en 2 días
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
apache/ponymail-foal#351 ·
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 68/100
Los mantenedores suelen responder en 4 días