-
Notifications
You must be signed in to change notification settings - Fork 762
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
Add Schneider Electric Thermostats #3473
Conversation
413c10d
to
28caae7
Compare
I wasn't sure if I should add it under |
0a3bb0f
to
831eb4e
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## dev #3473 +/- ##
==========================================
+ Coverage 89.44% 89.61% +0.17%
==========================================
Files 311 313 +2
Lines 10031 10218 +187
==========================================
+ Hits 8972 9157 +185
- Misses 1059 1061 +2 ☔ View full report in Codecov by Sentry. 🚨 Try these New Features:
|
d876b1c
to
2a26935
Compare
afa90ac
to
7e23b00
Compare
I went ahead and added the rest of the clusters, custom attributes and commands from the spec file. |
@TheJulianJES I think this one should be ready for review :) I think everything from spec is implemented. The only thing to add I can think of is home assistant numbers, sensors etc. For now, whatever I add gets |
7e23b00
to
cfd7f26
Compare
With HA Core 2024.11, ZHA should fall back to the |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thanks!
Can this be extended to handle the Image CCTFR6700 Schneider Electric's Wiser - Electric heating actuator- 16A - Pilot wire CCTFR6700 too? https://www.zigbee2mqtt.io/devices/CCTFR6700.html The device requires pilot wire setting and method to set temperature to work correctly. Currently it is detected but does not work correctly. |
Proposed change
Adds support to some Electric Thermostats based on the zigbee specs published here: Koenkk/zigbee2mqtt#23739
Specs: RTC-6754012INT-cFMT_16A_F_Zigbee-29.pdf
For now, I added two most interesting attributes for me:
Metering/fixed_load_demand
. It can be set to a nominal consumption of the heating. After setting this value, thermostat starts to output summation delivered estimated based on specified load.TemperatureMeasurement/temperature_sensor_type
. Can be used to modify external (floor) sensor type specified during initial installation.Reference to the original issue #1705
Checklist
pre-commit
checks pass / the code has been formatted using Black