search

EPAENERGYSTAR / epathermostat

Methods for measuring and reporting connected thermostat savings
16 stars 9 forks source link

Feature/epa2.0 climate zone weights #45

Closed john-zither closed 8 months ago

john-zither commented 8 months ago

Updated code to read these the updated climate zone weighting factors.

I am a little concerned that the tests don't seem to cover the climate zone file at all.

Name                                    Stmts   Miss  Cover   Missing
thermostat/climate_zone.py                 31     16    48%   64-80

What is the best way to test this file? I would like to add a test before merging my changes to it in.

lirainer commented 8 months ago

@john-zither Short answer to the test question is that retrieve_climate_zone() is not used. I have submitted a new issue #47 for cleaning up the zipcode/climate zone/weather code, but I can't assign it.