search

RobertD502 / home-assistant-flair

Custom component for Home Assistant Core for Flair pucks, vents, rooms, structures, and minisplits
MIT License
87 stars 12 forks source link

Feature: Create climate device for main house #36

Closed dkmcgowan closed 1 year ago

dkmcgowan commented 1 year ago

The main house could also have a climate entity, you have the set point and the mode, gives options on ways to display the entities

RobertD502 commented 1 year ago

Bear with me as I'm a bit confused as to what you're asking for here.

Are you asking for the Structure Mode (Heating/cooling) and Home set point entities to be combined into a single Climate entity?

If that is the case,there are some limitations as to what the entity can show/do:

There will be no current temperature reading since this is for the entire structure.

If the structure is set to manual mode: This entity will be unavailable

If the structure is set to auto mode & set point controller is set to thermostat: Only changing heat/cool is possible

If the structure is set to auto mode & set point controller is set to Flair app: Can change heat/cool and target temperature

Edit:

Also, there will be no hvac_action (heating/cooling) attribute available for this entity as we don't know if the structure is being actively heated or cooled. You can only determine if the HVAC mode is set to Heat/Cool/Auto.

dkmcgowan commented 1 year ago

Good points, I was talking about the structure mode and home set point being pushed into a climate entity. There is a current temperature right? The home set point has a current value, so when I display home set point in like an entities card I can see current value and can move it up or down. I get your point on manual mode, I haven't used that one much.

Looks like to complicated, I get it, thanks for responding...

On Tue, Oct 4, 2022 at 3:14 PM Robert Drinovac @.***> wrote:

Bear with me as I'm a bit confused as to what you're asking for here.

Are you asking for the Structure Mode (Heating/cooling) and Home set point entities to be combined into a single Climate entity?

If that is the case there are some limitations as to what the entity can show/do:

There will be no current temperature reading since this is for the entire structure.

If the structure is set to manual mode: This entity will be unavailable

If the structure is set to auto mode & set point controller is set to thermostat: Only changing heating/cooling is possible

If the structure is set to auto mode & set point controller is set to Flair app: Can change heating/cool and target temperature

— Reply to this email directly, view it on GitHub https://github.com/RobertD502/home-assistant-flair/issues/36#issuecomment-1267468205, or unsubscribe https://github.com/notifications/unsubscribe-auth/AJ24VVR54F2NHUHAUVHEQG3WBR6ZHANCNFSM6AAAAAAQ42VVSI . You are receiving this because you authored the thread.Message ID: @.***>

RobertD502 commented 1 year ago

So the Home set point isn't the current temperature. Rather, it is the "target" temperature that you want your system to reach. This is because the current temperature will be different based on what room you're looking at. The structure itself doesn't have a "current" temperature.

dkmcgowan commented 1 year ago

Yep, I get it, I was really just trying to more easily use a better UI control to set the home point and change the mode with mushroom cards and not rely on the entities card to be honest, not really wanting the climate entity really but a means to an end

On Tue, Oct 4, 2022, 5:39 PM Robert Drinovac @.***> wrote:

So the Home set point isn't the current temperature. Rather, it is the "target" temperature that you want your system to reach. This is because the current temperature will be different based on what room you're looking at. The structure itself doesn't have a "current" temperature.

— Reply to this email directly, view it on GitHub https://github.com/RobertD502/home-assistant-flair/issues/36#issuecomment-1267607516, or unsubscribe https://github.com/notifications/unsubscribe-auth/AJ24VVRRM4QSQTGH47ACA3LWBSPX7ANCNFSM6AAAAAAQ42VVSI . You are receiving this because you authored the thread.Message ID: @.***>

RobertD502 commented 1 year ago

I actually think it is a great idea and decreases the number of entities. Just added the climate entity to release 0.1.3 and have removed the entities Structure mode and Home set point since they are combined into that single climate entity.

dkmcgowan commented 1 year ago

That's awesome, just tested and it worked great!

Got me thinking on something I just never did in the Flair app but would be easier in HA integration, if the main structure is say on Cool and you turned a room into Heat what does Flair even do? Seems like that wouldn't do anything right?

On Tue, Oct 4, 2022 at 7:48 PM Robert Drinovac @.***> wrote:

Closed #36 https://github.com/RobertD502/home-assistant-flair/issues/36 as completed.

— Reply to this email directly, view it on GitHub https://github.com/RobertD502/home-assistant-flair/issues/36#event-7520772683, or unsubscribe https://github.com/notifications/unsubscribe-auth/AJ24VVU5JSMRGSLRH4RUMM3WBS65LANCNFSM6AAAAAAQ42VVSI . You are receiving this because you authored the thread.Message ID: @.*** com>

RobertD502 commented 1 year ago

Ah, yeah that's all outlined in the documentation.

You can't change the mode on a room by room basis. However, I still implemented HVAC mode switching in the Room climate entities - what this does is changes the HVAC mode of the structure the Room is associated with and in turn all other Rooms that are associated with said structure.

It was either implement HVAC mode or don't implement a climate entity for Flair Rooms at all - Home Assistant requires all climate entities to be able to change HVAC mode. If changing HVAC mode isn't implemented, users would see a NotImplemented error appear when trying to change HVAC mode. This used to be the case with the OAuth 1 version of the integration. I figured I'd implementing it (and document it) and just have the Room climate entity change the entire structure mode instead of throwing an error for the user. Plus, it may save some time for those that want to change the structure mode from a Room climate entity instead of a Structure climate entity.

dkmcgowan commented 1 year ago

Makes sense and I think actually turns into a feature

On Tue, Oct 4, 2022, 11:22 PM Robert Drinovac @.***> wrote:

Ah, yeah that's all outlined in the documentation.

You can't change the mode on a room by room basis. However, I still implemented HVAC mode switching in the Room climate entities - what this does is changes the HVAC mode of the structure the Room is associated with and in turn all other Rooms that are associated with said structure.

It was either implement HVAC mode or don't implement a climate entity for Flair Rooms at all - Home Assistant requires all climate entities to be able to change HVAC mode. If changing HVAC mode isn't implemented, users would see a NotImplemented error appear when trying to change HVAC mode. This used to be the case with the OAuth 1 version of the integration. I figured I'd implementing it (and document it) and just have the Room climate entity change the entire structure mode instead of throwing an error for the user. Plus, it may save some time for those that want to change the structure mode from a Room climate entity instead of a Structure climate entity.

— Reply to this email directly, view it on GitHub https://github.com/RobertD502/home-assistant-flair/issues/36#issuecomment-1267874105, or unsubscribe https://github.com/notifications/unsubscribe-auth/AJ24VVV3VHAL3QKGM26HJTTWBTX6JANCNFSM6AAAAAAQ42VVSI . You are receiving this because you authored the thread.Message ID: @.***>