-
-
Notifications
You must be signed in to change notification settings - Fork 12
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
Diagnostic Sensors not created for all equipment: [Discharge Air Temperature is reporting Open instead of the actual temperature] #181
Comments
Thanks for the note, from what I can tell this is just what it reports. Here is the data from my system:
To get the data from your system, follow the instructions for enabling message logging to a file in the readme. Restart HA or Reload the integration, wait a minute or two. Then look in the file for this message and/or post the file here. In this context I believe the word "open" means there is no sensor installed. The other uses of "Open" (note the capitalized O), seem to indicate the switch, etc, is actually open. When you say " as measured from the Discharge Air Sensor, in the Temperature Unit that is set in the S30.", do you see data in the S30 display that shows this temperature? If so it'd be good to know that value, perhaps we can then find it in the message log? |
Thank YOU! |
Super. Can you post your logfile here or email to me I'll try to figure out why it isn't showing up. |
Here goes Thanks |
Very cool. So your system have a zoning controller which is equipment Id 3. Right now the integration is not creating a device for it. That’ll need to be fixed. However, it should be attempting to create a sensor. It’s name should be System_3_Discharge_Air_Temperature See if you can find that. It may also be logging a warning in the home assistant log
But it’s also possible that whole path is broken because it’s never been tested. I’ll load your file into the simulator next day or so and see what comes up. |
Makes sense, the Discharge Air Temperature appears under the Zoning Controller section on the S30 diagnostics screen, that might be why it is reading open now as the zoning controller is taking over it in my configuration, and the device is not created in the integration at this point. in the home assistant log there is the This is cool - thank you :) |
I've made some initial progress. Probably a week or so to get the changes done, maybe less. I'll keep you posted when I have something to share. |
From what I can tell the second sensor should be getting created. if you got into developer tools / states, do you see something similar to this? sensor.System_3_Discharge_Air_Temperature |
The only one I see under developer tools / states is |
Got the changes working, next step is testing and packaging for release. Here is what your system looks like with the changes. Here is the device list. Note the zoning controller is showing up and the integration is pulling in the correct model numbers for the installed equipment. Here is the view of the zoning controller device. The diag sensors are showing up now and the serial number of the zoning controller is showing up under the device Hardware attribute (I changed the digits from what your serial number is) Lastly here's a dashboard to more clearly show the new attributes. |
NICE! |
Merging pull request, auto-closed. Creating release |
Release is here. On HA startup it’ll pull the new version of the lennoxs30api, this takes a few minutes. Let me know how it goes. https://github.com/PeteRager/lennoxs30/releases/tag/0.2.11 |
Installation went smooth, I can now see the Zoning Controler and the associated sensors reporting correctly! |
Integration Version 0.2.10
Within the "Indoor Unit", the Sensor "System_iu_Discharge_Air_Temperature" reports "Open" when it should report a numeric value; Temperature, as measured from the Discharge Air Sensor, in the Temperature Unit that is set in the S30.
The text was updated successfully, but these errors were encountered: