ioBroker / ioBroker.lovelace

Visualization with Lovelace-UI
Apache License 2.0
58 stars 27 forks source link

1.2.7 breaks HomeMatic thermostats #112

Closed agross closed 3 years ago

agross commented 3 years ago

Describe the bug

Since 1.2.7 HomeMatic thermostats display N/A temperature and "Auto".

To Reproduce

  1. Upgrade to 1.2.7
2020-10-20_15-14-50

The upper card is a simple-thermostat.

Expected behavior

With 1.2.6:

2020-10-20_16-49-39

Versions:

Garfonso commented 3 years ago

Thanks for the report and sorry for breaking it.

Could you post a Screenshot of the entity (with attributes) and the IoBroker objects?

Background: there are repeatedly reports from people that can not set a temperature using lovelace. So I'd be interested to find out where the difference is. Also I see that it says "auto" for you in 1.2.6, too. Does it always display auto there?

agross commented 3 years ago

Could you post a Screenshot of the entity

climate.HmIP_BWTH_000C9A498DC865_1

(nothing in the States column)

current_temperature: hm-rpc.1.000C9A498DC865.1.ACTUAL_TEMPERATURE
friendly_name: HmIP-BWTH 000C9A498DC865_1
icon: mdi:thermostat
max_temp: 30.5
min_temp: 4.5
temperature: hm-rpc.1.000C9A498DC865.1.SET_POINT_TEMPERATURE
unit_of_measurement: °C

and the IoBroker objects

image

image

Also I see that it says "auto" for you in 1.2.6, too. Does it always display auto there?

Not sure, I mostly use simple-thermostat because it uses up less screen real estate. I only added the normal thermostat for the screenshots above. Right now the "Auto" is missing, but I don't know why.

Garfonso commented 3 years ago

I will try to reproduce the problem with your information. But it will take a few days.

agross commented 3 years ago

Sure, in the meantime I downgraded to 1.2.6. Let me know if you need further assistance.

Beste Grüße,

Alex

+49 175 4107268 Tiny phone, tiny mail

On Tue, Oct 20, 2020, 17:23 Achim Königs notifications@github.com wrote:

I will try to reproduce the problem with your information. But it will take a few days.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/ioBroker/ioBroker.lovelace/issues/112#issuecomment-712932762, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAAD65IABZQ3R7SBRF5JLEDSLWTQRANCNFSM4SYOY7AQ .

thost96 commented 3 years ago

Same for me with EQ-3 Max thermostat. Same JS Controller, NodeJS version also on docker.

JB1985 commented 3 years ago

Same here with Bosch Smart Home. I open a bug in simple-thermostat https://github.com/nervetattoo/simple-thermostat/issues/171

Garfonso commented 3 years ago

They won't be able to help you. It is a bug in 1.2.7. I already fixed it (if someone wants to try, you can install lovelace from github. Version is not jet bumped, though). I am working on a few more things and then will release a new beta to latest (not sure when, though)

agross commented 3 years ago

Installing https://github.com/ioBroker/ioBroker.lovelace/commit/88873e9696a15276d364d37ddbe8dcf165fa9e83 fixed my HomeMatic thermostats.

The "Auto" string is missing, though:

1.2.6: image

master: image

Garfonso commented 3 years ago

@agross honestly I do not really know where the 'auto' string in 1.2.6 comes from. Is it important? :smile:

This was the commit, that broke thermostats. I just reverted it in the current dev version.

agross commented 3 years ago

Since I use simple-thermostat it doesn't really matter. Reverting is fine with me!