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

[New device request] Fairland Full Inverter Pool Heat Pump #222

Closed
VoyteckPL opened this issue Aug 16, 2022 · 4 comments
Closed

[New device request] Fairland Full Inverter Pool Heat Pump #222

VoyteckPL opened this issue Aug 16, 2022 · 4 comments

Comments

@VoyteckPL
Copy link

VoyteckPL commented Aug 16, 2022

Hi,
The heat pump for 15-30m3 pools - Fairland Inverter Plus 6,5kW IPHCR15

https://sklepbasenowy.pl/produkt/fairland-iphcr15/

All options work. Please add this to integration:

https://pastebin.com/kss3N846

@make-all make-all added new device Unsupported device config provided Config included by reporter labels Aug 20, 2022
@make-all
Copy link
Owner

This config is almost identical to the GardenPAC config except hvac_action is changed to hvac_mode. Actually it is a bit strange that the pool heatpumps supported so far have read-only status that can indicate heating or cooling modes, but only an on/off switch, no way to explicitly control the heating or cooling state. I suspect that all the madimack/gardenpac/fairland heatpumps are actually controllable through dp 105, just that users until now have only been interested in using them to heat.

@VoyteckPL
Copy link
Author

It says 105 is read only but you can change it without a problem. This config I provided works 100%.All options are working.

make-all added a commit that referenced this issue Aug 30, 2022
Issue #222

This is almost identical to the GardenPAC pool heatpump, but with the
following differences:
- modes are labelled Silent and Boost rather than Silent and Smart
  (control is a boolean, so this is just a UI difference).
- the hvac mode is controllable between smart, heat and cool.
  GardenPac appears to use the same DP as a read-only indicator of hvac
  action, and leaves the mode in smart always.
@make-all make-all added awaiting confirmation Wating for confirmation the issue is solved unreleased Will be in next release and removed config provided Config included by reporter labels Aug 30, 2022
@make-all make-all removed new device Unsupported device unreleased Will be in next release labels Sep 19, 2022
@make-all
Copy link
Owner

Included in 0.19.0 release

@make-all make-all removed the awaiting confirmation Wating for confirmation the issue is solved label Oct 16, 2022
make-all added a commit that referenced this issue Feb 2, 2023
Add a switch for switching between auto and manual.  Since the effect
attribute is not available when the light is off, it is not so
convenient only having it there, because you can't tell the difference
between off and auto modes when the light is off.

Issue #282

To acheive this, the handling of conditions was modified to not try to
set the constraint dp when it is read-only, and instead require an
exact match.  This allows us to choose between "mode_off" and
"mode_on" when turning the auto mode switch off, to avoid changing the
light state as a side effect.  The previous condition handling was
selecting the first match for off and trying to change both dps to
make that condition match.

- as a side effect of these changes, the Fairland IPHCR15 heatpump also
changed behaviour, as it had a mode dp marked as readonly in the
config, used as conditions for hvac_mode.  A comment on #222 says
that despite being described as readonly in the Tuya portal, it
appeared possible to change it, so to avoid unexpected problems, the
readonly attribute was removed so that current behaviour is retained.
@VoyteckPL
Copy link
Author

Please fix:
- id: 106
name: temperature
type: integer
mapping:
- constraint: temperature_unit
conditions:
- dps_val: false
range:
min: 60
max: 115
range:
min: 12
max: 45

min should be 12 (now it is 16) and it is not possible to set temp in range 12-16.

make-all added a commit that referenced this issue Oct 4, 2023
Based on the history, the temperature range appears to be simply copied from
the GardenPAC device, which was confirmed to have a minimum of 18.
Adjust the F range as well, rounded (previously it matched 16, probably
copied from before the GardenPAC and not updated when the minimum was set to 18)

Issue #222 (comment)
felipejfc pushed a commit to felipejfc/tuya-local that referenced this issue Oct 8, 2023
Based on the history, the temperature range appears to be simply copied from
the GardenPAC device, which was confirmed to have a minimum of 18.
Adjust the F range as well, rounded (previously it matched 16, probably
copied from before the GardenPAC and not updated when the minimum was set to 18)

Issue make-all#222 (comment)
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

No branches or pull requests

2 participants