-
Notifications
You must be signed in to change notification settings - Fork 526
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 device Pool Heatpump Fairland X20 series #2191
Add device Pool Heatpump Fairland X20 series #2191
Conversation
Hi I wanted to buy the Fairland X20 pump but I can't understand if it is compatible with tuya and home assistant. Do you know? thanks |
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.
This seems to match the poolsystem_ips100max that was recently added, only the "Show ..." binary sensors and error_2 attributes are missing from that config, but could easily be added as optional if needed.
I don't really understand the "Show ..." sensors - are they user config options for what is shown on the device's built in display, or are they informing the app which optional features are supported by the specific device?
I don't think either of those cases should use binary_sensor for these, but what exactly to do depends on the details.
- dps_val: 4 | ||
value: true | ||
- value: false | ||
- entity: binary_sensor |
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.
Why would this be a binary_sensor, not a switch? Same for the other two "Show..." items.
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.
Yes, it does look like it matches the poolsystem_ips100max
, when I started working on this it didn't exist yet.
After some testing, it appears that the Show ...
are just to inform the info to display on the app. I initially though they would be something we could act on but apparently not.
Shall I close this PR since the poolsystem_ips100max
has already been added?
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.
OK, I will add the additional dps as optional attributes on the relevant entities in case they are useful to someone.
Hi! I have it integrated with tuya-local, using the configurations in this PR but apparently the support for it was already added under the name Fairland produces a lot of equipment that is rebranded and sold under other names. You should be OK with the current version of tuya-local |
Add some optional dps from PR #2191
Poolsystems IPS100Max is also a recent addition in PR #2171, and is still unreleased. |
What do you think of renaming the |
I'm not sure which is the main brand. Probably both are selling a white labelled product from China. |
From what I could find Fairland is the "white label" brand from China but I can't say for sure |
Add support for the Fairland X20 series of Pool Heatpumps
https://www.fairland.com.cn/product/inverx20-inverter-pool-heat-pump.html
Supported functions/sensors:
Fault error 1: ["E1", "E2", "E3", "E4", "E5", "E6", "E7", "E8", "E9", "EA", "EB", "ED", "P0", "P1", "P2", "P3", "P4", "P5", "P6", "P7", "P8", "P9", "PA", "F1", "F2", "F3", "F4", "F5", "F6", "F7"]
Fault error 2: ["F8", "F9", "Fb", "Fa"]
I could only identify "E3" as "No water flow" and "E5" as "abnormal power supply"
fairland-x20-data-model.json
fairland-x20-dps-sample.json