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

UI does not show correct thermostat scale, min, max values #169

Closed
silverhack77 opened this Issue Jun 6, 2016 · 10 comments

Comments

Projects
None yet
4 participants
@silverhack77
Copy link

commented Jun 6, 2016

Z-Way server version 2.2.2
UI version 1.4.0

I have modified /opt/z-way-server/config/Defaults.xml to have:

1


1

and restarted the z-way-server service.

The SmartHome UI still shows temperature scale in C, but reports correct device setting (74 deg Fahrenheit) and if I try to change the setting in SmartHome UI, it only allows min 5 through max 40.

image

If I click on the ^ here, it changes 74 to 40 and will not go any higher

image

@silverhack77

This comment has been minimized.

Copy link
Author

commented Jun 6, 2016

@maros

This comment has been minimized.

Copy link
Contributor

commented Jun 6, 2016

IIrc the min/max setting of a thermostat can be changed by setting metrics:min and metrics:max http://raspberrypi:8083/JS/Run/controller.devices.get(%22DeviceId%22).set(%22metrics:max%22,90)

@PoltoS PoltoS unassigned vades Jun 6, 2016

@PoltoS

This comment has been minimized.

Copy link
Member

commented Jun 6, 2016

It looks like engine still remembers old values from old scale. Either delete this device from config.json or re-include it in Z-Wave.

@silverhack77

This comment has been minimized.

Copy link
Author

commented Jun 6, 2016

marcos:
Yes, I can change the min/max metrics using the JS/Run method and now when I use the SmartHome UI I see this:
image
So the circle indicator now shows the correct percentage to maximum, but if I click on the ^ to increase, it still gets set back to 40.

PoltoS:
I did not see the individual devices in
/opt/z-way-server/automation/defaultConfigs/config.json
(attached my config.json)
So I will try to exclude and re-include the device in Z-Wave, and will post the results later tonight.

@silverhack77

This comment has been minimized.

Copy link
Author

commented Jun 6, 2016

config.json

config.json.txt

@silverhack77

This comment has been minimized.

Copy link
Author

commented Jun 6, 2016

A combination of both suggestions was needed.
Here is what I did:

  1. remove device
  2. re-include the device
  3. let interview complete
  4. Set device description (chose the description for my CT30 thermostat)
  5. Change min, max, and scaleTitle using the JS API (see below)
  6. Re-interview device

Here are the commands I used:
(temperature sensor)
http://raspberrypi:8083/JS/Run/controller.devices.get(%22ZWayVDev_zway_16-0-49-1%22).set(%22metrics:scaleTitle%22,%22%C2%B0F%22)

(heating)
http://raspberrypi:8083/JS/Run/controller.devices.get(%22ZWayVDev_zway_16-0-67-1%22).set(%22metrics:scaleTitle%22,%22%C2%B0F%22)
http://raspberrypi:8083/JS/Run/controller.devices.get(%22ZWayVDev_zway_16-0-67-1%22).set(%22metrics:min%22,50)
http://raspberrypi:8083/JS/Run/controller.devices.get(%22ZWayVDev_zway_16-0-67-1%22).set(%22metrics:max%22,90)

(cooling)
http://raspberrypi:8083/JS/Run/controller.devices.get(%22ZWayVDev_zway_16-0-67-2%22).set(%22metrics:scaleTitle%22,%22%C2%B0F%22)
http://raspberrypi:8083/JS/Run/controller.devices.get(%22ZWayVDev_zway_16-0-67-2%22).set(%22metrics:min%22,50)
http://raspberrypi:8083/JS/Run/controller.devices.get(%22ZWayVDev_zway_16-0-67-2%22).set(%22metrics:max%22,90)

Now in the SmartHome UI it is correct:
image

Two followup questions:
A) Should this issue be closed, or be moved to the z-way backend to have these values set correctly after the interview?

B) The Expert UI still shows incorrectly. Is there a separate github repo that I should open a issue for that?
image

@PoltoS

This comment has been minimized.

Copy link
Member

commented Jun 6, 2016

That looks strange. Can you exclude and include and show ZAutomation/api/v1/devices/XXX data for the device? And also ZWaveAPI/Run/zway.devices[XXX].ThermostatSetpoint.data

Let's check this. I believe backend should handle this correctly.

As for Expert UI, this is a bug. Thanks for the report. I've fixed it and it will become part of next release: Z-Wave-Me/ExpertUI@a92263e

@silverhack77

This comment has been minimized.

Copy link
Author

commented Jun 7, 2016

I have excluded and included the device (it is now device 33).

Here is the results from /ZAutomation/api/v1/devices
devices.json.txt

Here is the results from /ZWaveAPI/Run/zway.devices[33]
zway.devices[33].txt

It is still working correctly in the SmartHome UI, and I will follow/update Z-Wave-Me/ExpertUI@a92263e for the Expert UI.

silverhack77 referenced this issue in Z-Wave-Me/ExpertUI Jun 7, 2016

@PoltoS

This comment has been minimized.

Copy link
Member

commented Jun 7, 2016

So we can close it?

@silverhack77

This comment has been minimized.

Copy link
Author

commented Jun 7, 2016

Yes, I think so. Thanks for the help!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.