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

Add Support for Drexma WiStat thermostat ET7AW #3016

Open
wants to merge 14 commits into
base: main
Choose a base branch
from

Conversation

Kevin-0u
Copy link

Add support for Drexma WiStat thermostat (model: ET-7AW), which is used for floor heating. Product details can be found here. I also added its document and DPs retrieved via the Tuya IoT API in docs folder in my fork so it can be referenced if anyone wants to modify or develop this farther.

@Kevin-0u
Copy link
Author

FYI, this is primarily based on jiahong_et72w_thermostat.yaml with a few additions and modifications of the DPs.

min: -90
max: 90
- id: 107
name: unit
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Since this is a relative temperature, we cannot have HA doing conversions, so it is better to use a hardcoded unit: ° on the dp above and remove the dynamic unit dp.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The dynamic unit dp is retained, with the unit hardcoded to ° and the class: temperature attribute removed. This prevents HA from performing conversions, allowing the calibration range and step to update automatically as needed.

name: preset_mode
mapping:
- dps_val: "Smart"
value: Program
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Use all lower case here to pick up the translations.

name: lock
- entity: select
name: Location
type: string
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Type is not valid here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: 👀 In review
Development

Successfully merging this pull request may close these issues.

2 participants