handled failture case for descriptions to fuel types and added a note to fix this in the future

This commit is contained in:
Khalim Conn-Kowlessar 2025-11-07 23:33:30 +00:00
parent 19a766f442
commit fa30ecf1f5
2 changed files with 6 additions and 1 deletions

View file

@ -79,6 +79,7 @@ DESCRIPTIONS_TO_FUEL_TYPES = {
"fuel": "Electricity", "cop": AVERAGE_ASHP_EFFICIENCY / 100
},
"Electric heat pump for water heating only": {"fuel": "Electricity", "cop": 1},
"Ground source heat pump, warm air, electric": {"fueld": "Electricity", "cop": AVERAGE_ASHP_EFFICIENCY / 100},
}
# These are the measure types where if there is a ventilation recommendation, we force the inclusion of it

View file

@ -688,7 +688,11 @@ class Recommendations:
}
raise NotImplementedError("Handle this case")
mapped = descriptions_to_fuel_types[heating_description]
mapped = descriptions_to_fuel_types.get(heating_description, None)
if mapped is None:
# TODO: This is a non-ideal placeholder but we put something in place for a process that falls over
# fairly regularly. A task has been added to planner to refactor this
mapped = {"fuel": 'Unmapped', "cop": 0.9}
heating_fuel = mapped["fuel"]
if hotwater_description in [