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

AC: Adds switch to jet mode #369

Merged
merged 6 commits into from
Aug 6, 2022
Merged

Conversation

dougiteixeira
Copy link
Contributor

Adds the swtich to toggle jet mode on and off (#175).

Manual information and application behavior observation: Jet mode is compatible with HVAC COOL, HEAT and DRY modes (by my tests when activating in dry mode, cool mode is selected automatically).

Here's an explanation of the jet mode according to the manual, but it may have some variation depending on the model:
image

@dougiteixeira
Copy link
Contributor Author

dougiteixeira commented Aug 5, 2022

An explanation of the operating mode observed in the application:

When activated it has the following behavior:

  • If in HEAT mode, set the temperature to 30°C and increase the speed.
  • If in COOL mode, set the temperature to 18°C and increase the speed.
  • If in DRY mode, change the mode to COOL, set the temperature to 18°C and increase the speed.
  • At the end of time or when the temperature reaches the set, it only changes the speed to the previous value, it does not change the mode or temperature setting.

When disabled it has the following behavior:

  • It only changes the speed to the previous value, it does not change the mode or temperature setting.

@MatiasFernandez
Copy link

Thanks for this work @dougiteixeira. How can I help testing this? I'm not sure what's the best way to get this version of the code running in my HA instance to see how it works. Maybe manually download the files as they are in the latest commit of the PR and then copying them manually to my HA installation?

@dougiteixeira
Copy link
Contributor Author

dougiteixeira commented Aug 5, 2022

Thanks for this work @dougiteixeira. How can I help testing this? I'm not sure what's the best way to get this version of the code running in my HA instance to see how it works. Maybe manually download the files as they are in the latest commit of the PR and then copying them manually to my HA installation?

You need to replace the contents of the files below with the contents of the links below:

Copy link
Owner

@ollo69 ollo69 left a comment

Choose a reason for hiding this comment

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

Thanks for this PR, looks really good👍.
Just a couple of think to change...

custom_components/smartthinq_sensors/wideq/ac.py Outdated Show resolved Hide resolved
custom_components/smartthinq_sensors/wideq/ac.py Outdated Show resolved Hide resolved
@dougiteixeira dougiteixeira requested a review from ollo69 August 6, 2022 05:57
@dougiteixeira dougiteixeira requested a review from ollo69 August 6, 2022 12:08
Copy link
Owner

@ollo69 ollo69 left a comment

Choose a reason for hiding this comment

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

👍

@ollo69 ollo69 merged commit 85d4894 into ollo69:master Aug 6, 2022
@dougiteixeira dougiteixeira deleted the ac-moed-jet branch August 6, 2022 19:52
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.

3 participants