search

tomasmcguinness / homeassistant-mixergy

Add support for Mixergy's smart water tank into Home Assistant
MIT License
38 stars 8 forks source link

Switch to push #51

Open mattjgalloway opened 8 months ago

mattjgalloway commented 8 months ago

This migrates the whole integration to use the STOMP over Websocket connection to Mixergy to support pushing of data rather than regular polling.

@tomasmcguinness: I wanted to put this up now, but I don't want to merge it just yet (so putting it up as a draft). I'm going to play around with it and keep testing it out. It all seems to work fine, but it's a huge change so I want to make sure I haven't missed anything. Also if you're keen to test it out, that would be amazing too.

(Fixes #48)

mattjgalloway commented 8 months ago

I've given this more testing and all seems to be well. It's pretty cool seeing the updates happen in realtime for the tank settings. And measurement data comes in straight away when the tank updates. So I would say we're going to be causing less load for Mixergy with this because it sends us data every minute rather than us polling every 30 seconds.

By the way - I lodged a request to Mixergy to support MQTT in the tank. It's probably a long shot but it would be amazing to have local MQTT support. I want it mainly for very fast updating of the clamp and PV divert power data. The minute by minute data is kind of sucky for graphs. But nothing we can do about that with the data we currently get from Mixergy.

tomasmcguinness commented 8 months ago

I've been asking them annually for local control via an API or MQTT. They keep saying "it's something we want to do".

Would you mind if I pre-released this?

mattjgalloway commented 8 months ago

I've been asking them annually for local control via an API or MQTT. They keep saying "it's something we want to do".

How hopeful are you that they'll do it? I really hope they do!

Would you mind if I pre-released this?

Sure thing! How would you do that?

tomasmcguinness commented 8 months ago

Not hopeful. I can see the support issues local control might cause - "I can't connect", "What's MQTT?" etc

Can't prerelease - forgot the branch is your repository, not mine!

mattjgalloway commented 8 months ago

I’m hoping they allow MQTT as a purely developer thing. Not something that’s required to be used. But it probably is a long shot you’re right.

For the branch - you could clone my branch, or give me access to push a branch?

mattjgalloway commented 8 months ago

I've had this running for a week now on my testing instance. All seems to be going well. I'd consider this stable at this point!

mattjgalloway commented 7 months ago

I would hold off on this for now. I've found a slight hiccup where the stomp connection isn't properly reconnecting when the internet goes down. I'm doing some more work on this to see how I can make it resilient to internet outage.

tomasmcguinness commented 4 months ago

I need to get back to this!

Do you kno if it's possible to merge this PR into another branch in this repo?

mattjgalloway commented 4 months ago

Sorry yes, I need to get back to this too! It's close, I just wanted to figure out why it locks up when network disconnects.

I have been quite busy with work recently, but I have a whole load of time off coming up, and am planning to dedicate some to many home automation projects! I'll definitely be trying to get this one over the line!