Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

1.2.7 breaks HomeMatic thermostats #112

Closed
agross opened this issue Oct 20, 2020 · 10 comments · Fixed by #116
Closed

1.2.7 breaks HomeMatic thermostats #112

agross opened this issue Oct 20, 2020 · 10 comments · Fixed by #116

Comments

@agross
Copy link
Contributor

agross commented Oct 20, 2020

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:

  • Adapter version: 1.2.7
  • JS-Controller version: 3.1.5
  • Node version: 12
  • Operating system: docker
@Garfonso
Copy link
Collaborator

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
Copy link
Contributor Author

agross commented Oct 20, 2020

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
Copy link
Collaborator

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

@agross
Copy link
Contributor Author

agross commented Oct 20, 2020 via email

@thost96
Copy link
Contributor

thost96 commented Oct 20, 2020

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

@JB1985
Copy link

JB1985 commented Nov 3, 2020

Same here with Bosch Smart Home. I open a bug in simple-thermostat nervetattoo/simple-thermostat#171

@Garfonso
Copy link
Collaborator

Garfonso commented Nov 3, 2020

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
Copy link
Contributor Author

agross commented Nov 4, 2020

Installing 88873e9 fixed my HomeMatic thermostats.

The "Auto" string is missing, though:

1.2.6:
image

master:
image

@Garfonso
Copy link
Collaborator

Garfonso commented Nov 5, 2020

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

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

@Garfonso Garfonso mentioned this issue Nov 6, 2020
@agross
Copy link
Contributor Author

agross commented Nov 6, 2020

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants