sebr / bhyve-home-assistant

Orbit BHyve custom component for Home Assistant
MIT License
252 stars 42 forks source link

Issues with Battery, State and Zone History sensors #192

Closed briodan closed 1 year ago

briodan commented 1 year ago

Describe the bug The Battery Level, State and Zone history sensors are unavailable

Expected behaviour The Battery Level, State and Zone History sensors are available

BHyve devices BH1-0001 WiFi Hub 4x HT25-0000

Device Diagnostics attached config_entry-bhyve-5e8da9c59a6d3cfc9e9124d45a1c9086.json.txt

image image

I did notice this after adding two new HT25-000, one item I noticed is that the two HT25-000 I added don't display battery info in the bhyve app. I did add one of the devices prior to the outage this past Sunday and one after the outage. So that might be part of the cause on those devices, however I'm also not getting battery info from the devices I previously had Old existing devices image New Device 1 added before outage image New Device 2 added after outage no diagnostic entities created

cpritchett commented 1 year ago

Seeing the same with an HT31.

WayneManion commented 1 year ago

I have a pair of HT34 four-port hose timers and they haven't been working for the last few days. I haven't had time to dig too deep, but I suspect I'm having the same issues.

briodan commented 1 year ago

to further add, i'm able to set the programing and rain delays as well as manually trigger the HT25 through HomeAssistant, so all that functionality still works, looks like its just missing the battery, state and history data.

Tried removing and readding the integration but that did not get the sensors back.

Wondering if there was some kind of API change.

briodan commented 1 year ago

quick update, I contacted Orbit about the two new devices that don't pull battery info in the app or web dashboard their instructions were to do an RMA as defective devices. Not sure I buy it but we'll see Sunday when the new HT25's show up.

Did a bit of API exploration and battery info definitely still comes through on an API call for the older, so i dont this its an API change issue

"weather_forecast_location_id": "XXXXX",
"firmware_version": "0041",
"name": "Front Yard",
"type": "sprinkler_timer",
"manual_preset_runtime_sec": 600,
"battery": {
        "percent": 96,
        "charging": false
sebr commented 1 year ago

@briodan the issue is likely the same as #189.

The failure to load the battery caused all of the sensors to fail to load.

I looked at your diagnostics file and you can see that two of the devices have neither battery readings nor zone names. This could have been an issue?

 Sprinkler: Front Yard
   Battery: 96%
     Zones: 1
            - Front Yard

 Sprinkler: Back Yard
   Battery: 96%
     Zones: 1
            - Back Yard

 Sprinkler: Front Sprinkler
   Battery: None%
     Zones: 1
            - Unknown Zone

 Sprinkler: Back Sprinkler
   Battery: None%
     Zones: 1
            - Unknown Zone
briodan commented 1 year ago

Yes it does seems to be a duplicate of 189 and fixed in the latest version

Now regarding the battery etc. on the newer devices I have now received the replacement devices from Amazon for the RMA and unfortunately they behave the same way and they don't report battery etc. status in the app or web dashboard.

Tried contacting Orbit b-hyve about it again but was not able to get anywhere as the chat agent would not talk to me other then to confirm they were still there. I'll try again tomorrow to see if I get anywhere. But at this point it seems to me there is an issue with newly added devices getting properly configured on the Orbit end, so not an issue with the integration.