feat(pipeline): include 'Open, but proposed to close' schools in dims
These schools are still operating and publish results; they drop out automatically when GIAS flips them to Closed since marts fully rebuild each run. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
This commit is contained in:
@@ -30,7 +30,7 @@ models:
|
|||||||
- name: status
|
- name: status
|
||||||
tests:
|
tests:
|
||||||
- accepted_values:
|
- accepted_values:
|
||||||
values: ["Open"]
|
values: ["Open", "Open, but proposed to close"]
|
||||||
|
|
||||||
- name: dim_location
|
- name: dim_location
|
||||||
description: School location dimension with PostGIS geometry
|
description: School location dimension with PostGIS geometry
|
||||||
|
|||||||
@@ -31,4 +31,5 @@ select
|
|||||||
else null
|
else null
|
||||||
end as longitude
|
end as longitude
|
||||||
from {{ ref('stg_gias_establishments') }} s
|
from {{ ref('stg_gias_establishments') }} s
|
||||||
where s.status = 'Open'
|
-- Must match dim_school's status filter exactly (the API inner-joins the two).
|
||||||
|
where s.status in ('Open', 'Open, but proposed to close')
|
||||||
|
|||||||
@@ -91,4 +91,7 @@ from schools s
|
|||||||
{% if ofsted_relation is not none %}
|
{% if ofsted_relation is not none %}
|
||||||
left join {{ ref('int_ofsted_latest') }} o on s.urn = o.urn
|
left join {{ ref('int_ofsted_latest') }} o on s.urn = o.urn
|
||||||
{% endif %}
|
{% endif %}
|
||||||
where s.status = 'Open'
|
-- "Open, but proposed to close" schools are still operating (pupils enrolled,
|
||||||
|
-- results published) — include them; they drop out automatically once GIAS
|
||||||
|
-- flips them to "Closed" (marts are fully rebuilt each run).
|
||||||
|
where s.status in ('Open', 'Open, but proposed to close')
|
||||||
|
|||||||
Reference in New Issue
Block a user