Skip to content

Commit e217ab7

Browse files
authored
Merge pull request #354 from The-Strategy-Unit/bugfix-pbm-op
Bugfix to allow PBM to run
2 parents d2ccb71 + df11c51 commit e217ab7

File tree

1 file changed

+4
-3
lines changed

1 file changed

+4
-3
lines changed

model/data/databricks.py

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -225,14 +225,15 @@ def get_op(self) -> pd.DataFrame:
225225
.filter(F.col("fyear") == self._year)
226226
.withColumn("dataset", F.lit("NATIONAL"))
227227
.withColumn("sitetret", F.lit("NATIONAL"))
228+
# TODO: temporary fix, see #353
229+
.withColumn("sushrg_trimmed", F.lit("HRG"))
230+
.withColumn("imd_quintile", F.lit(0))
228231
.groupBy(
229232
op.drop(
230233
"index",
231234
"fyear",
232235
"attendances",
233-
"tele_attendances",
234-
"sushrg_trimmed",
235-
"imd_quintile",
236+
"tele_attendances"
236237
).columns
237238
)
238239
.agg(

0 commit comments

Comments
 (0)