MortJC / homebridge-platform-orbit

Orbit Irrigation System platform plugin for HomeBridge
MIT License
34 stars 5 forks source link

B-Hyve Battery and Charge Status #6

Closed mitch7391 closed 4 years ago

mitch7391 commented 4 years ago

@MortJC just want to thank you first for getting this working, I was one of the first testers from Cody’s plug-in and have been dying for him to finish it or get it working. Thanks to you I got my B-Hyve 6 zone WiFi controller to work from HomeKit for the first time :)

I was wondering if you can get other statuses in it? I noticed the Cody’s version had a battery level and charging status (even though the unit is plugged into mains power), it would be nice to have the charging status. Also I wonder if the battery is the internal memory battery? So maybe that is still important.

MortJC commented 4 years ago

Hi Mitch7391,

Thanks for the feedback.

In regards to the battery level, I don't think that applies to the mains versions as I'm not getting any data from the API in regards to battery level. I'm guessing if I had a battery powered device I might get the data.

Cheers James.

On Thu, Apr 9, 2020 at 10:05 PM mitch7391 notifications@github.com wrote:

@MortJC https://github.com/MortJC just want to thank you first for getting this working, I was one of the first testers from Cody’s plug-in and have been dying for him to finish it or get it working. Thanks to you I got my B-Hyve 6 zone WiFi controller to work from HomeKit for the first time :)

I was wondering if you can get other statuses in it? I noticed the Cody’s version had a battery level and charging status (even though the unit is plugged into mains power), it would be nice to have the charging status. Also I wonder if the battery is the internal memory battery? So maybe that is still important.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/MortJC/homebridge-platform-orbit/issues/6, or unsubscribe https://github.com/notifications/unsubscribe-auth/AN5E4TR2T3AMS4ELUWXFIT3RLW2XHANCNFSM4MEVV3AQ .

mitch7391 commented 4 years ago

That is fair, maybe it was a bug in the previous plug-in? It always confused me and that’s why I thought it might have been an internal memory battery or something.

mitch7391 commented 4 years ago

Funny enough @MortJC when I rebooted homebridge this morning after an update, I received the following error which I could swear I had seen once before since installing your plug-in:

[4/15/2020, 6:25:57 AM] [BHyve] Unhandled message received: clear_low_battery

The B-Hyve happened to be running its scheduled run at the time.

MortJC commented 4 years ago

Can't really do much with that unless there's another status.

I think the battery status is really for the battery powered devices.

On Wed, Apr 15, 2020 at 8:31 AM mitch7391 notifications@github.com wrote:

Funny enough @MortJC https://github.com/MortJC when I rebooted homebridge this morning after an update, I received the following error which I could swear I had seen once before since installing your plug-in:

[4/15/2020, 6:25:57 AM] [BHyve] Unhandled message received: clear_low_battery

The B-Hyve happened to be running its scheduled run at the time.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/MortJC/homebridge-platform-orbit/issues/6#issuecomment-613712968, or unsubscribe https://github.com/notifications/unsubscribe-auth/AN5E4TWFFBDAYTPTTEIXEL3RMTP2JANCNFSM4MEVV3AQ .

mitch7391 commented 4 years ago

I will see if I can get that status at all in debug mode if that would help? That appeared in my normal Homebridge logs whilst it was running.

wallacepost commented 4 years ago

27/05/2020, 5:10:30 am [orbit] Unhandled message received: clear_low_battery

MortJC commented 4 years ago

Hi, what issue are you reporting?

On Wed, May 27, 2020 at 5:15 AM wallacepost notifications@github.com wrote:

27/05/2020, 5:10:30 am [orbit] Unhandled message received: clear_low_battery

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/MortJC/homebridge-platform-orbit/issues/6#issuecomment-634223471, or unsubscribe https://github.com/notifications/unsubscribe-auth/AN5E4TRLRHGUFMLGNQPP5PTRTQIOFANCNFSM4MEVV3AQ .

mitch7391 commented 4 years ago

I am assuming they are getting the same message as me in their Homebridge logs? Just as an FYI I still receiver this message with each reboot of Homebridge.

MortJC commented 4 years ago

Thanks for the info.

On Wed, May 27, 2020 at 9:19 AM mitch7391 notifications@github.com wrote:

I am assuming they are getting the same message as me in their Homebridge logs? Just as an FYI I still receiver this message with each reboot of Homebridge.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/MortJC/homebridge-platform-orbit/issues/6#issuecomment-634330611, or unsubscribe https://github.com/notifications/unsubscribe-auth/AN5E4TTI3DBHTE5JSHO33VLRTRFA5ANCNFSM4MEVV3AQ .

genesispc commented 4 years ago

Just to let you know I have also received the "[orbit] Unhandled message received: clear_low_battery" this morning. Will keep an eye out for it happening again. I'm new to the whole homebridge scene. Is there a way to dump the messages being sent just by the orbit? I've edited the platform.js script to print the json messages received into the log file. Hopefully be able to get the battery details from there..

Edit. Updated the platform.js in order to dump the response from the API calls. Created a PR with change to ignore those the response for now...

MortJC commented 4 years ago

Hi

Yes that event can be ignored.

If you run it in Debug you can see all the messages.

Cheers.

On Fri, Sep 4, 2020 at 10:08 AM genesispc notifications@github.com wrote:

Just to let you know I have also received the "[orbit] Unhandled message received: clear_low_battery" this morning. Will keep an eye out for it happening again. I'm new to the whole homebridge scene. Is there a way to dump the messages being sent just by the orbit?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/MortJC/homebridge-platform-orbit/issues/6#issuecomment-686825544, or unsubscribe https://github.com/notifications/unsubscribe-auth/AN5E4TVWQ72RAQDFJ2E2MKTSEAVWHANCNFSM4MEVV3AQ .

mitch7391 commented 4 years ago

@MortJC I’m not sure if I’m missing something here, but is there anyway we can stop the alarm coming up in our logs? Or if it is a real alarm is there anyway to use it like how other accessories can display a low battery status? It’s just that I still have it coming up in my logs with no resolution to what is to happen to it. If I can I reduce one more alarm in my logs that is a plus.

MortJC commented 4 years ago

Hi, it's not an alarm, just an unhandled message that we are not interested in as there is no way to do anything with this event.

On Tue, Sep 8, 2020 at 2:05 PM mitch7391 notifications@github.com wrote:

@MortJC https://github.com/MortJC I’m not sure if I’m missing something here, but is there anyway we can stop the alarm coming up in our logs? Or if it is a real alarm is there anyway to use it like how other accessories can display a low battery status? It’s just that I still have it coming up in my logs with no resolution to what is to happen to it. If I can I reduce one more alarm in my logs that is a plus.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/MortJC/homebridge-platform-orbit/issues/6#issuecomment-688605894, or unsubscribe https://github.com/notifications/unsubscribe-auth/AN5E4TV2LPWUB26NDO5QI3DSEWUQ3ANCNFSM4MEVV3AQ .

mitch7391 commented 4 years ago

Sorry alarm was a poor choice of words. So it is just definitely something we have to ignore? It can’t be removed in any way?

MortJC commented 4 years ago

Yes it can be ignored.

On Tue, Sep 8, 2020 at 2:29 PM mitch7391 notifications@github.com wrote:

Sorry alarm was a poor choice of words. So it is just definitely something we have to ignore? It can’t be removed in any way?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/MortJC/homebridge-platform-orbit/issues/6#issuecomment-688610895, or unsubscribe https://github.com/notifications/unsubscribe-auth/AN5E4TWYLQ2FR24RS3KUBHLSEWXJJANCNFSM4MEVV3AQ .