junte/Model
1
0
Fork 0
mirror of https://github.com/Hestia-Homes/Model.git synced 2026-06-08 11:17:27 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

Merge pull request #536 from Hestia-Homes/eco-eligiblity-bug

Browse source 
handled failure case for descriptions to fuel types and added a note to fix this in the future
This commit is contained in:
KhalimCK 2025-11-07 23:34:36 +00:00 • committed by GitHub
commit 38cef97eda
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
2 changed files with 6 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

1
backend/app/assumptions.py
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

6
recommendations/Recommendations.py
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 [