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

Correct local temperature calibration min and max value on SONOFF TRVZB #8214

Merged
merged 1 commit into from
Oct 29, 2024

Conversation

m-larin
Copy link
Contributor

@m-larin m-larin commented Oct 29, 2024

Experimentally found out the supported calibration range for SNOFF TRVZB. When sending a message to MQTT with subject zigbee2mqtt/0x0cae5ffffebc15dc/set/local_temperature_calibration and transmitting deliberately large values, I received an error explicitly specifying the acceptable range: The value of "value" is out of range. It must be >= -128 and <= 127. Received 160. Send this max 12.7 and minv -12.8 values work correct, error not raise.

@Koenkk Koenkk merged commit db1a214 into Koenkk:master Oct 29, 2024
2 checks passed
@Koenkk
Copy link
Owner

Koenkk commented Oct 29, 2024

Thanks!

@fgsch
Copy link

fgsch commented Nov 22, 2024

Hi. Sorry to write in a closed (and merged) PR. I've opened zigpy/zha#300 to make the exact change in ZHA, and someone suggested changing the step to 0.1. Do you have any thoughts on this?

@m-larin
Copy link
Contributor Author

m-larin commented Nov 22, 2024

Hi. Sorry to write in a closed (and merged) PR. I've opened zigpy/zha#300 to make the exact change in ZHA, and someone suggested changing the step to 0.1. Do you have any thoughts on this?

Hi. The step affects how much the temperature calibration will change when pressing the up or down buttons in the web interface. When sending a number, you can use any value with an accuracy of one tenth.

@fgsch
Copy link

fgsch commented Nov 22, 2024

@m-larin oh right, that's a very good point.

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 this pull request may close these issues.

3 participants