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

Z-Wave JS - MH9-CO2 - Temperature Reporting Threshold - Only allows Integer #106807

Closed
damige opened this issue Dec 31, 2023 · 6 comments
Closed

Comments

@damige
Copy link

damige commented Dec 31, 2023

Describe the issue you are experiencing

When adjusting Z-Wave JS - MH9-CO2 - Temperature Reporting Threshold, i am only able to set the value of 0 or 1, but not 0.5 or 0.1.
The example shows 0.5 as a possible value, but when i apply this error appears: "'An error occurred.expected int for dictionary value @ data['value']. Got 0.5"

A temperature sensor with a resolution of full degrees celcius is not ideal.

Z-Wave JS version: 0.4.3
Home Assistant Yellow
core-2023.12.4

What type of installation are you running?

Home Assistant OS

Which operating system are you running on?

Other (e.g., Raspbian/Raspberry Pi OS/Fedora)

Which add-on are you reporting an issue with?

Z-Wave JS

What is the version of the add-on?

0.4.3

Steps to reproduce the issue

  1. Settings, Device and services, Zwave
  2. Devices , CO2 Monitor Air Quality Detector
  3. Configure, Parameter 3 - Temperature Reporting Threshold
  4. Set float value like 0.1 or 0.5
  5. Error appears
    ...

System Health information

System Information

version core-2023.12.4
installation_type Home Assistant OS
dev false
hassio true
docker true
user root
virtualenv false
python_version 3.11.6
os_name Linux
os_version 6.1.58-haos-raspi
arch aarch64
timezone Europe/Amsterdam
config_dir /config
Home Assistant Community Store
GitHub API ok
GitHub Content ok
GitHub Web ok
GitHub API Calls Remaining 5000
Installed Version 1.33.0
Stage running
Available Repositories 1427
Downloaded Repositories 2
Home Assistant Cloud
logged_in false
can_reach_cert_server ok
can_reach_cloud_auth ok
can_reach_cloud ok
Home Assistant Supervisor
host_os Home Assistant OS 11.2
update_channel stable
supervisor_version supervisor-2023.12.0
agent_version 1.6.0
docker_version 24.0.7
disk_total 916.2 GB
disk_used 8.8 GB
healthy true
supported true
board yellow
supervisor_api ok
version_api ok
installed_addons Silicon Labs Multiprotocol (2.3.2), InfluxDB (4.8.0), Advanced SSH & Web Terminal (17.0.1), Grafana (9.1.1), Z-Wave JS (0.4.3)
Dashboards
dashboards 1
resources 0
views 6
mode storage
Recorder
oldest_recorder_run December 21, 2023 at 15:30
current_recorder_run December 27, 2023 at 18:18
estimated_db_size 247.49 MiB
database_engine sqlite
database_version 3.41.2

Anything in the Supervisor logs that might be useful for us?

No response

Anything in the add-on logs that might be useful for us?

No response

Additional information

No response

@damige
Copy link
Author

damige commented Dec 31, 2023

image

@MartinHjelmare MartinHjelmare transferred this issue from home-assistant/addons Jan 1, 2024
@home-assistant
Copy link

home-assistant bot commented Jan 1, 2024

Hey there @home-assistant/z-wave, mind taking a look at this issue as it has been labeled with an integration (zwave_js) you are listed as a code owner for? Thanks!

Code owner commands

Code owners of zwave_js can trigger bot actions by commenting:

  • @home-assistant close Closes the issue.
  • @home-assistant rename Awesome new title Renames the issue.
  • @home-assistant reopen Reopen the issue.
  • @home-assistant unassign zwave_js Removes the current integration label and assignees on the issue, add the integration domain after the command.
  • @home-assistant add-label needs-more-information Add a label (needs-more-information, problem in dependency, problem in custom component) to the issue.
  • @home-assistant remove-label needs-more-information Remove a label (needs-more-information, problem in dependency, problem in custom component) on the issue.

(message by CodeOwnersMention)


zwave_js documentation
zwave_js source
(message by IssueLinks)

@MartinHjelmare
Copy link
Member

The text to the right of the input box in the configuration panel isn't an example. It's the unit. So the value you enter should represent the desired amount in the shown unit.

In this case the unit is 0.5 degrees Celsius. So if you want to change to eg 0.5 degrees Celsius, you'd enter 1. If you want to change to 2.5 degrees Celsius you'd enter 5.

@damige
Copy link
Author

damige commented Jan 1, 2024

Thanks for the clarification.
Would it be possible to have the unit changed to units of 0.1? This was the case when i used openhab. and as i use this sensor as input for a thermostat, the added resolution is required.

@MartinHjelmare
Copy link
Member

The unit comes from the driver configuration of the device.

If you have proof that the driver configuration of the device is incorrect, ie not matching the manufacturer specifications for the device, you can submit an issue to the driver project issue tracker or even better, a pull request where you update the device configuration to the correct values.

Here's the configuration of the device in question:

https://github.com/zwave-js/node-zwave-js/blob/9549a040477364b54a2df617a42fd30639e67e14/packages/config/config/devices/0x015f/mh9-co2.json#L78-L94

@damige
Copy link
Author

damige commented Jan 1, 2024

First time for everything. zwave-js/node-zwave-js#6581

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

No branches or pull requests

2 participants