-
Notifications
You must be signed in to change notification settings - Fork 10
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
Missing schedule and away modes and fan settings #29
Comments
Agreed it would. be good to support all these functions. To break this down I think there are three distinct things here:
I question whether using presets is the best way to surface these. For example, with the support for emergency heat mode just added as a preset (which I think makes sense since that is the closest things to a mode), choosing a different preset would seem to imply turning off emergency heat. Not to mention the issues you are also bringing up with the preset approach. I don't see a way to make that make sense. Another option I can think of is start adding other types of entities on the thermostat. So for example we could have a What do you think? |
Emergency heat as an HVAC mode would be preferable. If it can't be done (I don't know enough about the coding side to assess feasibility) then moving it out to a service call or a switch makes sense. HASS has a climate.set_aux_heat service call (per https://www.home-assistant.io/integrations/climate/), so that seems to be the way they want you to do it. For air circulation, I think climate.set_fan_mode with 7 options: Away and Schedule switches is in line with the physical thermostat UI, but the two settings interact since Away overides schedule (i.e., you can't have Schedule and Away both On together.). Also shecdule functions actually like 3 options: on, off and temp overriden. I think the way the daikinskyport integration handles this part makes sense using the preset. As for setting schedule parameters, I (personally) think it's lower priority as users can use automations within Home Assistant. |
Looks like the daikinskyport intergation has a bunch of this stuff in [services]https://github.com/apetrycki/daikinskyport/blob/master/custom_components/daikinskyport/services.yaml) |
Going back and forth between this and the spyport integration. I'm getting a lot of functionality from controlling the away and schedule modes on the skyport integration. I feel some of these parameters would be better suited for a select entity anyways. Temporary Hold duration could be a good candidate. |
I just added support in the latest release for controlling the fan mode (off, on, scheduled) and speed. Fan mode is on the climate entity and there is a new select for the speed. Open to feedback on the controls but hopefully that is a step in the right direction |
I was looking for a way to pause and resume scheduling, but it seems to be missing. Looking over at the daikinskyport integration, they have it under Preset:
daikinskyport presets:
Temp hold: pauses the schedule until the next change
Manual: turns off schedule
Schedule: turns on schedule
Away: turns off schedule and enables Away mode
They also have fan speed control which would also be nice to have in this integration:
It sets the circulate air functions on the thermostat:
Auto: turns off circulate air
On: circulate air always on
Schedule: circulate air on a schedule
Low/Medium/High: sets the speed for On and Schedule
It's a bit awkward how it works and needs a two-sequence tap to get everything to work.
If it is currently set to schedule/medium you want fan mode to On/low, you need to first tap Low. The UI shows Low for a few seconds and then reverts back to Schedule. Then Tap On.
The low/medium/high labels are only shown temporarily then revert back Auto/On/Schedule.
I guess this follows the physical thermostat UI where the off/on/schedule is shown at the upper level. What's awkward is that your can't actually see the current fan speed anywhere.
If doable I think it would make more sense to have 7 options instead:
Auto: turns off circulate air
On L/M/H: circulate air always on at set speed
Schedule L/M/H: circulate air on a schedule at set speed
The text was updated successfully, but these errors were encountered: