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

go-e controller support #117

Closed
cfaelled opened this issue Oct 3, 2023 · 15 comments
Closed

go-e controller support #117

cfaelled opened this issue Oct 3, 2023 · 15 comments

Comments

@cfaelled
Copy link

cfaelled commented Oct 3, 2023

Ive tried without success to add the go-e controller to HA using the addon - my go-e charger working flawless..

Is it possible to do so with this addon or does that need further development/coding ?
https://go-e.com/en/products/go-e-controller

@BD992000
Copy link

I think ther has to be done some coding.
Maybe one can just use the code base an "just" implement the definitions according to the go-e controller api (-> https://github.com/goecharger/go-eController-API).

Maybe I will try. But I am no "python guy" so this will be kind of a challenge (because I hate the python syntax ...)

@KJK-78
Copy link

KJK-78 commented Nov 9, 2023

I also just got the controller as my old meter died. Adding mqtt support parameters would be super as then they could be used elsewhere at HA. So basically reading the parameters eGo-Controller provides would be enough. No need to write anything.

If someone has to skills + energy to do this would be super.

@syssi
Copy link
Owner

syssi commented Nov 9, 2023

Do you own the device and could you provide some MQTT message examples + topics?

@karloskar
Copy link
Contributor

I have a controller and fired up a local mqtt broker (Mosquitto). With mqtt-explorer I can see the messages from the controller.

In what format would you prefer the message examples? And how do you prefer to get them?

@syssi
Copy link
Owner

syssi commented Nov 9, 2023

In plaintext as code block here in the issue or as attached txt file if github accepts this file extension.

@karloskar
Copy link
Contributor

karloskar commented Nov 10, 2023

Please see attached file messages.json

It's in JSONL format but GitHub wouldn't let me use the .jsonl extension.
Every line is [{topic w/o prefix}, {dumped payload}]

Let me know if you need more!

@KJK-78
Copy link

KJK-78 commented Dec 12, 2023

Hi, is there any development on this topic? If some more message examples are needed I can also try to provide. I have 3phase setup if that makes any difference.....

@karvanoppa
Copy link

Waiting this too. This add current sensors to my home assistant. Thanks

@karloskar
Copy link
Contributor

I've had busy days workwise but am hoping this holiday season may bring some free time.

Is it desirable to have the controller code as part of this project/repo @syssi? As this project/repo is called homeassistant-goecharger-mqtt I was wondering whether homeassistant-goe-mqtt as a shared place or another project/repo called homeassistant-goecontroller-mqtt would be preferable. I ask this humbly as I don't now what a change of name would entail.

@karloskar
Copy link
Contributor

I created POC here.

As of now only the basic power related things for the six CT sensors have been added.

Please note: there can be rough edges. I haven't tested installing through HACS as an example.

@BD992000
Copy link

@karloskar : I tried your integration. Seems to work really well.
Now I can easily store the data from my go-e controller in my influxdb.

Great work! Many thanks!

@SteffenLav
Copy link

SteffenLav commented Jan 23, 2024

Hey @karloskar - I am using your Controller POC and all the sensors I needed from it you got covered!
Thank you so much. It is so nice finally to have excess PV to charge my Tesla MY fully automated :)
BTW installs nicely through HACS
Steffen

@syssi
Copy link
Owner

syssi commented Feb 3, 2024

I'm happy about your implementation. Feel free to provide a pull request to extend the README of this project to point to your solution. We could also try to merge your device support into this project.

@karloskar
Copy link
Contributor

I've made a pull request for a README change. Feel free to edit if you think it should be positioned elsewhere.

I don't mind at all combining our projects 😄 My experience with HA development is rather limited. Would the questions I raised further up this thread be a good starting point straightening out?

@syssi
Copy link
Owner

syssi commented May 25, 2024

Merged! :-) Let's close this issue.

@syssi syssi closed this as completed May 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

7 participants