search

jmcollin78 / solar_optimizer

The Solar Optimizer integration for Home Assistant starts and stops your equipments depending on the Solar net production
MIT License
53 stars 10 forks source link

Sensor domain for energy sell/buy price #15

Closed shiner66 closed 6 months ago

shiner66 commented 6 months ago

Permits the selection of sensors to indicate energy buy and sell prices to use directly prices coming from energy integrations, which are exposed as sensors

jmcollin78 commented 6 months ago

Thank you @shiner66 !

jmcollin78 commented 6 months ago

Unfortunately CI failed. Can you please have a look ?

shiner66 commented 6 months ago

Hello, it seems to be an issue independent from my PR

ERROR: Could not open requirements file: [Errno 2] No such file or directory: 'requirements_test.txt'

I tested the changes on my ha instance and works flawlessly

jmcollin78 commented 6 months ago

Maybe something have change on the HA CI steps. Are you able to have a look on what happens ?

shiner66 commented 6 months ago

Maybe something have change on the HA CI steps. Are you able to have a look on what happens ?

Again, I have no clue, I believe the error is caused by the CI itself, you should investigate your actions code. I don't know how my commit could in any way cause the following command to fail(?), I'm very confused haha

▶ Run python3 -m pip install -r requirements_test.txt

jmcollin78 commented 6 months ago

Will see that later. For now I merge.