home-assistant / core

:house_with_garden: Open source home automation that puts local control and privacy first.
https://www.home-assistant.io
Apache License 2.0
69.94k stars 29.02k forks source link

Gree AC Swing modes not fully supported #88368

Open ic-dev21 opened 1 year ago

ic-dev21 commented 1 year ago

The problem

It is not possible to choose a fixed vertical or horizontale positions, only swing modes.

What version of Home Assistant Core has the issue?

Home Assistant 2023.2.3

What was the last working version of Home Assistant Core?

No response

What type of installation are you running?

Home Assistant Container

Integration causing the issue

Gree

Link to integration documentation on our website

https://www.home-assistant.io/integrations/gree/

Diagnostics information

No response

Example YAML snippet

No response

Anything in the logs that might be useful for us?

No response

Additional information

I have not tried this custom integration but this seems like it would work;

Replacing this:

https://github.com/home-assistant/core/blob/1e056970800d236a84535d817e01e2a93796f3c1/homeassistant/components/gree/climate.py#L86

With this:

https://github.com/RobHofmann/HomeAssistant-GreeClimateComponent/blob/7947b7fe129c7caee0089aa676b273eafd0b6d2f/custom_components/gree/climate.py#L69

home-assistant[bot] commented 1 year ago

Hey there @cmroche, mind taking a look at this issue as it has been labeled with an integration (gree) you are listed as a code owner for? Thanks!

Code owner commands Code owners of `gree` can trigger bot actions by commenting: - `@home-assistant close` Closes the issue. - `@home-assistant rename Awesome new title` Change the title of the issue. - `@home-assistant reopen` Reopen the issue. - `@home-assistant unassign gree` Removes the current integration label and assignees on the issue, add the integration domain after the command.

(message by CodeOwnersMention)


gree documentation gree source (message by IssueLinks)

issue-triage-workflows[bot] commented 1 year ago

There hasn't been any activity on this issue recently. Due to the high number of incoming GitHub notifications, we have to clean some of the old issues, as many of them have already been resolved with the latest updates. Please make sure to update to the latest Home Assistant version and check if that solves the issue. Let us know if that works for you by adding a comment πŸ‘ This issue has now been marked as stale and will be closed if no further activity occurs. Thank you for your contributions.

ic-dev21 commented 1 year ago

Still an issue.

Le ven. 19 mai 2023 Γ  00:06, issue-triage-workflows[bot] < @.***> a Γ©crit :

There hasn't been any activity on this issue recently. Due to the high number of incoming GitHub notifications, we have to clean some of the old issues, as many of them have already been resolved with the latest updates. Please make sure to update to the latest Home Assistant version and check if that solves the issue. Let us know if that works for you by adding a comment πŸ‘ This issue has now been marked as stale and will be closed if no further activity occurs. Thank you for your contributions.

β€” Reply to this email directly, view it on GitHub https://github.com/home-assistant/core/issues/88368#issuecomment-1553966261, or unsubscribe https://github.com/notifications/unsubscribe-auth/AZZGCFNG3U4V5UUVPBFWZGDXG3WULANCNFSM6AAAAAAVADA4TQ . You are receiving this because you authored the thread.Message ID: @.***>

geraldnolan commented 1 year ago

+1

I would like to be able to control the direction of swing like you can in the app.

bradleewright commented 1 year ago

I found a way to get it working - Gree Extension for Home-Assistant built in integration installed via HACS.

It lets you move the swing position via a service call - Works really well

ic-dev21 commented 1 year ago

That is a nice workaround but I think it should be fixed in the native integration, thanks for letting me know.

GBCRAS1 commented 11 months ago

I do agree that the GREE Native integration should offer these capabilities.

issue-triage-workflows[bot] commented 8 months ago

There hasn't been any activity on this issue recently. Due to the high number of incoming GitHub notifications, we have to clean some of the old issues, as many of them have already been resolved with the latest updates. Please make sure to update to the latest Home Assistant version and check if that solves the issue. Let us know if that works for you by adding a comment πŸ‘ This issue has now been marked as stale and will be closed if no further activity occurs. Thank you for your contributions.

darmach commented 8 months ago

Still persists

ic-dev21 commented 7 months ago

Using node-red to do this right now but native gree should work

almostlunatic commented 7 months ago

I would be very happy with that fix for swing angles, too. That is very important to me, living far away from my home for a long time. So, would you please fix tis issue, it shouldn't be so hard to do?

From the divice's manual:

opera_ufbFqYRcyX
issue-triage-workflows[bot] commented 3 months ago

There hasn't been any activity on this issue recently. Due to the high number of incoming GitHub notifications, we have to clean some of the old issues, as many of them have already been resolved with the latest updates. Please make sure to update to the latest Home Assistant version and check if that solves the issue. Let us know if that works for you by adding a comment πŸ‘ This issue has now been marked as stale and will be closed if no further activity occurs. Thank you for your contributions.

ic-dev21 commented 3 months ago

Still an issue. Using node red to compensate.

orzechszek commented 3 months ago

Hi,

Any updates on this feature? Summer is comming... πŸ˜„ I think should be added possiblity to change horizontal and vertical swing separately.

geraldnolan commented 3 weeks ago

Any updates on this? Summer is here and I'd love to be able to change the swing to one direction and not only "swing horizontally"

@ic-dev21 How are you using Node Red to compensate?

ic-dev21 commented 3 weeks ago

Any updates on this? Summer is here and I'd love to be able to change the swing to one direction and not only "swing horizontally"

@ic-dev21 How are you using Node Red to compensate?

I found this Node-Red component: https://flows.nodered.org/node/node-red-contrib-gree-hvac

I suck at node-red as I can't really wrap my mind around this but I came up with this flow, probably isn't the best or most efficient but it works:

image

My mini split is in the dining room, open floor space with living room and kitchen. It's near the dining table. So when someone sits at the table I flip a boolean ON using a motion snesor in Home Assistant and pass it on to Node Red.

If I'm in heating mode, vertical swing is fully down, as heat rises and I want my heating to be efficient. That means that someone at the table would get blasted with heat. So when the sensor is ON, swing mode moves up and away from the dining table until nobody is around the table for like 15 minutes.

geraldnolan commented 3 weeks ago

@ic-dev21 - Thanks for sharing! Your post gave me a few ideas. I decided to install the HACS component from this link: HomeAssistant-GreeClimateComponent.

Not sure why I waited so long to do this. Now, I have full control over the swing mode, and the Fan Mode "Quiet" works perfectly every time, unlike the default one worked sometimes.

janmashat commented 3 weeks ago

I have not tried this custom integration but this seems like it would work;

Replacing this:

https://github.com/home-assistant/core/blob/1e056970800d236a84535d817e01e2a93796f3c1/homeassistant/components/gree/climate.py#L86

With this:

https://github.com/RobHofmann/HomeAssistant-GreeClimateComponent/blob/7947b7fe129c7caee0089aa676b273eafd0b6d2f/custom_components/gree/climate.py#L69

The implementation should ideally utilize the dependent library's range of swing modes: https://github.com/cmroche/greeclimate/blob/master/greeclimate/device.py#L67-L91

The problem is that those are separated into VerticalSwing and HorizontalSwing, while the HA climate entity only supports a single swing_mode property: https://developers.home-assistant.io/docs/core/entity/climate/#swing-modes

There's a discussion to add horizontal swing, so hopefully that will get some traction: https://github.com/home-assistant/architecture/discussions/553