search

davidusb-geek / emhass-add-on

The Home Assistant Add-on for EMHASS: Energy Management Optimization for Home Assistant
MIT License
96 stars 29 forks source link

Update config.yml #51

Closed jatty closed 1 year ago

jatty commented 1 year ago

Workaround for Issue #50

davidusb-geek commented 1 year ago

Ok I understand the issue but the code is expecting a 24h hour format. Jsut changing the hours perdios in the configuration doesn"t solve the main issue. How are we supposed to provide hours n the PM side of the day?

jatty commented 1 year ago

This is woraround is meant for the rollout, where the home assistant engine seems to malform the time fields. After the initial config is written, your code is working well. So my workaround is: use values only which are before midday (so its regardless whether it is 12 or 24-hour format).

davidusb-geek commented 1 year ago

Ok, merging...