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

When in away mode on an s30, all zones but the first one are shown unavailable on the dashboard #358

Open
stevesreed opened this issue Feb 5, 2025 · 19 comments · Fixed by #359 or #362
Assignees
Labels
bug Something isn't working

Comments

@stevesreed
Copy link

stevesreed commented Feb 5, 2025

Is there a way to make them show "off" or "away" instead of "unavailable"

Image

@stevesreed stevesreed changed the title When in away mode, all zones but the first one are shown unavailable on the dashboard When in away mode on an s30, all zones but the first one are shown unavailable on the dashboard Feb 5, 2025
@PeteRager
Copy link
Owner

PeteRager commented Feb 5, 2025

That should not be happening.

Do you have zoning disabled? Or disabled when in away mode.

Any errors or warnings in the log?

@stevesreed
Copy link
Author

Zoning it enabled on the S30, And work great when away mode it off.

Is disabling zoning in away mode a setting on the s30 or in HA?

When I disable away mode, the logs say "s30_master bedroom_hvac_mode changed to heat_cool", but when I change to away mode the logs says: "s30_master room_hvac_mode became unknown"

@stevesreed
Copy link
Author

stevesreed commented Feb 5, 2025

I did find these in the system log, but it was when I was editing the dashboard, not when toggling away on or off:

Logger: homeassistant.components.climate
Source: helpers/entity_platform.py:825
integration: Climate (documentation, issues)
First occurred: January 29, 2025 at 8:32:38 AM (6 occurrences)
Last logged: 1:11:17 PM

Error adding entity None for domain climate with platform lennoxs30
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/helpers/entity_platform.py", line 825, in _async_add_entity
capabilities=entity.capability_attributes,
^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/src/homeassistant/homeassistant/components/climate/init.py", line 386, in capability_attributes
if ClimateEntityFeature.TARGET_HUMIDITY in supported_features:
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
TypeError: argument of type 'int' is not iterable

and

Logger: homeassistant.components.websocket_api.http.connection
Source: components/websocket_api/commands.py:287
integration: Home Assistant WebSocket API (documentation, issues)
First occurred: January 29, 2025 at 8:26:23 AM (13 occurrences)
Last logged: 1:09:04 PM

[139977161837584] Entity climate.s30_living_room does not support action climate.set_temperature
[139977255152384] Entity fan.fr_fan_controller does not support action fan.turn_off
[139977071283360] Entity climate.s30_kitchen does not support action climate.turn_off
[139977045539376] Entity fan.fr_fan_controller does not support action fan.turn_off

@PeteRager
Copy link
Owner

PeteRager commented Feb 6, 2025

Please post the home-assistant.log file or the entries from there for those errors, it will have more complete information than in the UI summary.

What version of the integration are you using? What version of HA?

@stevesreed
Copy link
Author

after the update to the 2025.2 core, this log is pretty empty except for a few S30 errors.

home-assistant_2025-02-06T03-14-40.690Z.log

@stevesreed
Copy link
Author

stevesreed commented Feb 6, 2025

Image

After reloading the integration, it seems to have changes behavior. Now it shows "Idle" when I go to away mode, and preset more says "Away". Which is perfect.

It has not done that in months. I wonder if something changed in 2025.2.

I'll watch it over the next few days, to see it it goes unavailable again, and I'll grab the logs if it does.

@PeteRager
Copy link
Owner

That error should not be happening either. Are you running version 2024.11.0 of the integration? The HA code in that error hasn't changed in 2 years, so this does not look like an HA upgrade problem and is probably the result of your S30 sending something the integration is not expecting.

If so, I'd like you to turn on debug logging for the integration and turn on message logging, restart HA, post those two files.

https://github.com/PeteRager/lennoxs30?tab=readme-ov-file#reporting-bugs

@stevesreed
Copy link
Author

@PeteRager
Copy link
Owner

I have been able to duplicate this error and have a fix in PR #359

Error adding entity None for domain climate with platform lennoxs30
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/helpers/entity_platform.py", line 825, in _async_add_entity
capabilities=entity.capability_attributes,
^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/src/homeassistant/homeassistant/components/climate/init.py", line 386, in capability_attributes
if ClimateEntityFeature.TARGET_HUMIDITY in supported_features:

It occurs when the zone becomes disabled. When this occurs at startup the climate entity will not get created - which is what occurred in home-assistant_2025-02-06T03-14-40.690Z.log - and that seems to correspond to the 3 zones being unavailable. That error did not occur in your latest logs.

It could be a startup issue in that if away mode is on during startup this situation happens? If so, we may want to rerun the test / logs with the system in Away Mode to see if it recreates.

This should only occur if zoningMode is not "zoned". This is in the status message:

                    "status": {
                        "diagLevel": 0,
                        "outdoorTemperature": 54,
                        "doNotPersist": true,
                        "diagRuntime": 2033,
                        "configured": false,
                        "diagVentilationRuntime": 0,
                        "diagPoweredHours": 16764,
                        "unknownDeviceFound": false,
                        "ventilationRemainingTime": 0,
                        "wideSetpointRange": false,
                        "outdoorTemperatureStatus": "good",
                        "ventilatingUntilTime": "",
                        "writeAccess": "local",
                        "reminder": false,
                        "configProblem": false,
                        "zoningMode": "zoned",
                        "replacementPart": false,
                        "alert": "moderate",
                        "feelsLikeMode": false,
                        "numberOfZones": 4,
                        "singleSetpointMode": false,
                        "missingDeviceFound": false,
                        "rsbusMode": "normal",
                        "replaced": false,
                        "outdoorTemperatureC": 12,
                        "incompleteSystem": false
                    },

@PeteRager PeteRager linked a pull request Feb 7, 2025 that will close this issue
@PeteRager PeteRager added the bug Something isn't working label Feb 7, 2025
@PeteRager PeteRager self-assigned this Feb 7, 2025
@stevesreed
Copy link
Author

Here are the logs when rebooting HA, while the s30 is in away mode.

home-assistant.log
s30_local.log

@PeteRager
Copy link
Owner

The zoningMode is set to "central" in the logs. So when in away mode the Lennox system must be switching from "zoned" to "central" mode. This means it should be acting as a single zoned system controlled by zone 0 which is your kitchen.

I'll review more tomorrow and work on getting a new release available.

@stevesreed
Copy link
Author

Yes, it does seem to go into non zoned mode when away is on.

It a little weird, but it’s always done that.

I don’t see any settings to keep the zones showing on the S30 when it’s away mode.

Thanks for looking into this, and all your s30/s40/m30 support.

@PeteRager
Copy link
Owner

From the Lennox docs https://www.lennox.com/literature/Lennox_iComfortS30_Homeowner_Manual.pdf

"Away When the away icon is touched, the system will automatically use energy saving settings - heat-to 62 (16.5) and cool-to 85 (29.5). Temperatures can be adjusted by pressing on the available temperature setting (i.e., heat-to or cool-to). To exit away, press the cancel icon. In a zoning system, all zones are set to a single heat-to and cool-too setting. Note that when manually selecting Away from the home screen, the Smart Away feature (if enabled under settings) will be temporally disabled until Away is canceled. See page 10 for additional information."

"Manual Away The user can manually put the system into away mode by pressing the “away” icon. When the system is in away mode, the horseshoe is grayed-out and shows the “away” status text. In order to exit away mode, the user can press the “cancel” icon. In away mode, the away set points are used to control the system. The user can modify the away set points from the default values in menu > settings > away. 55°"

The behavior we should see, once the bug is fixed, is described here

https://github.com/PeteRager/lennoxs30?tab=readme-ov-file#zoning-mode

@PeteRager PeteRager reopened this Feb 8, 2025
@PeteRager
Copy link
Owner

Here is the updated release,

https://github.com/PeteRager/lennoxs30/releases/tag/2025.2.0

I can't directly system test as I don't have a zoned system. Let me know how it works.

@stevesreed
Copy link
Author

Tested the new version:

I don't see any errors when I reboot HA while S30 is in away mode.

The 3 other zone show as "unknown" instead of "unavailable" when in away.

All zone display the correct info as soon as away is disabled.

Thanks for the update!

@PeteRager
Copy link
Owner

The other zones should be displaying the current temperature when in away mode. Sounds like that is not happening?

@stevesreed
Copy link
Author

ah. yes they are showing temperature now.

Image

@PeteRager
Copy link
Owner

I got this working in the simulator and see that behavior. I'd like to get rid of the unknowns - perhaps have it say "away" instead?

@stevesreed
Copy link
Author

Sorry I did not see your last question.

I agree, “away” is a reasonable way to indicate their status in this case.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
2 participants