lewisbenge / hass-amber-electric

Home Assistant Component to pull the latest energy prices from Amber Electric
MIT License
27 stars 7 forks source link

Sensor is returning the price in the previous period. #7

Closed stuarts0011 closed 4 years ago

stuarts0011 commented 4 years ago

The sensors are displaying the price for the preceding 30 minute period, not the current.

This is also reflected in the fact that there are two attribute readings for the ACTUAL price, but the first is for the preceding period. The second is the correct one and what the sensor should be returning.

stuarts0011 commented 4 years ago

To fix this, in Lines 82 and 87 of sesnor.py change "0" to "1"

adavis59 commented 4 years ago

What about a way to compare if price displayed is accurate with Post code set/time zone?

madpilot commented 4 years ago

PR #12 fixes this.

OmgImAlexis commented 4 years ago

@lewisbenge this can be closed, fixed in #12