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

Feature: Official core integration? #60

Open
AlecRust opened this issue Dec 30, 2020 · 4 comments
Open

Feature: Official core integration? #60

AlecRust opened this issue Dec 30, 2020 · 4 comments

Comments

@AlecRust
Copy link

I'm running Home Assistant on some slow hardware where I'd rather not install HACS.

Just wondered if there were any plans to get this merged in to core, or what remains to be done before this can happen?

I was also curious if the plan would be to replace the existing Heatmiser integration or add an additional one, if your project is specific to the neoStat line of thermostats?

I have a neoStat thermostat and use the Heatmiser Neo iOS app. The current official integration doesn't seem to set up correctly, but looks like your integration would.

@DrTree
Copy link
Contributor

DrTree commented Dec 30, 2020

Hey,
A few of us were looking at what it would take to get this into core. There are a lot of hoops (quite rightly as they don’t want any only junk in core) to jump through so it might take quite a while.

this is for a different line of thermostats (anything that’s controlled via the Neo hub) the existing core integration is for an older (?) stat that communicates directly over a com port I believe. They are complimentary rather than competing if that makes sense

@stikonas
Copy link
Contributor

stikonas commented Jan 3, 2021

Hopefully we are getting there...

First of all, device specific communication needs to happen via external python library, not directly inside integration. @DrTree and I were working on this and this part is close to being finished (#59).

Then I think https://developers.home-assistant.io/docs/device_registry_index/ has to be implemented in this integration, so that it shows devices (e.g. NeoHub, NeoStats and not just entities). That's hopefully fairly easy.

Then most importantly, we need to make sure integration is stable. So it would be good if more people could test it. There might be slightly different devices/configuration than what I have and it would be good if they are all supported (e.g. I only have heating, no cooling), I don't have NeoPlugs...
(By the way, you can run it without HACS too, just copy custom_components/heatmiserneo/ from this repo into config/custom_components/heatmiserneo.)

And then I guess it might be time to try to push this into core.

@stikonas
Copy link
Contributor

stikonas commented Jan 3, 2021

Oh, also one thing I forgot to mention that would be nice is adding support to pairing new devices... Hopefully not too hard, but somebody has to look at Home Assistant documentation and find the right HA API to setup frontent (our backend library has support for pairing).

@cyclops1982
Copy link
Contributor

What's the current status of this item?
I've used the old integration and would really like it if HA had this one out of the box...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants