Added todo

This commit is contained in:
Khalim Conn-Kowlessar 2023-06-22 19:17:50 +01:00
parent c2e67f5e35
commit 7dbeffe8c4

View file

@ -256,6 +256,10 @@ class WallRecommendations(BaseUtility):
if self.property.walls["thermal_transmittance_unit"] != self.U_VALUE_UNIT:
raise NotImplementedError("Haven't handled the case of other u value units yet")
# TODO: It's worth thinking about this logic because depending on when properties were built,
# they're likely to be of a certain standard. E.g. properties built within a certain time
# period are likely to have cavity walls
if (not is_cavity_wall) and (self.year_built >= self.YEAR_WALLS_BUILT_WITH_INSULATION) and (
u_value >= self.BUILDING_REGULATIONS_PART_L_MAX_U_VALUE
):