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

SMaBiT AV2010/32 does support cool when configured via ctrlSeqOfOper #5452

Merged
merged 10 commits into from
Feb 8, 2023

Conversation

sjorge
Copy link
Contributor

@sjorge sjorge commented Feb 7, 2023

#5399 (comment)

This device will support cool or heat depending on the ctrlSeqOfOper value. The manual it has logic for heating_only and cooling_only.

image

The device now has a 'dynamic expose' function that returns the correct supported modes based on ctrlSeqOfOper. I also added batteryVoltage reading, fixed the battery calculation. I also extended Climate expose to add a withCtrlSeweofOper methode.

We should update the notes for this device that a restart of z2m is needed after changing ctrlSeqOfOper for the expose data to be updated.

```
power supply: 2x1,5V AA Size Battery
```

The voltage alarm is set at 25, so the 25-30 range of `3V_2500` is the correct one.
- read ctrlSeqeOfOper, we need this for the dynamic expose data as this dictates if we can be off/heat or off/coolo
- read localTemperatureCalibration instead of report, limited reporting slots and you cannot change this on the device itself via input
@sjorge
Copy link
Contributor Author

sjorge commented Feb 7, 2023

ctrlSeqeOfOper set to 0 (cooling_only) and restarted z2m
Screenshot 2023-02-07 at 19 14 17

Yes, system_mode is still set to heat as the thermostat hasn't reported the new mode yet. But hitting cool or off does not result in a INVALID_VALUE.

After setting ctrlSeqeOfOper to 2 (heating_only) and restarting z2m:
image

I can now set system_mode to off or heat without an INVALID_VALUE message.

@sjorge
Copy link
Contributor Author

sjorge commented Feb 8, 2023

@Koenkk I'll do some more testing during my lunch break and let you know if everything still looks good.

@sjorge
Copy link
Contributor Author

sjorge commented Feb 8, 2023

All good, still works as expected. After changing ctrlSeqeOfOpr and restarting.

Koenkk added a commit to Koenkk/zigbee2mqtt.io that referenced this pull request Feb 8, 2023
@Koenkk
Copy link
Owner

Koenkk commented Feb 8, 2023

Thanks!

@Koenkk Koenkk merged commit f7e27c2 into Koenkk:master Feb 8, 2023
@sjorge sjorge deleted the av2010_32 branch February 11, 2023 22:04
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.

2 participants