amberelectric / public-api

Discuss the Amber public API
MIT License
46 stars 4 forks source link

5 minute forecasts don't align with transitional TOU periods #212

Open purcell-lab opened 4 days ago

purcell-lab commented 4 days ago

My ToU peak period starts at 1600.

The 30 minute intervals are tagged correctly via the API, however the 5 minute intervals are not tagged correctly and as a consequence the perkWh field is also calculated incorrectly.

I would appear that the last 5 minute interval of the initial 30 minute interval is tagged as peak, whilst I would suggest that all the 5 minute intervals should be tagged as peak and the associated perkWh field requires update.

Screenshot-2024-07-01-16-04-42 (1)

Screenshot_20240701-161614

madpilot commented 4 days ago

Good pickup. Will have a look