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

Add Support for Home Assistant running in docker #39

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

MKSG-MugunthKumar
Copy link

Hey,
I run docker in HA and added this to support another method for installation.

@vk2him
Copy link
Owner

vk2him commented Oct 16, 2024

Thanks - this is awesome. Do you think we should update the readme with instructions on how to implement? Maybe you can write a few lines that I can add? I assume you've only added the two files, and that no other changes are needed in my original python script?

@MKSG-MugunthKumar
Copy link
Author

MKSG-MugunthKumar commented Oct 16, 2024 via email

@MKSG-MugunthKumar
Copy link
Author

Hey, updated the README file.
It works on my end with one caveat.

My sensors for FW 7.6 looks like this (Similar to FW8). I'm in Australia.
I think we will leave this up to the person who uses this?

- sensor:
      name: envoy mqtt consumption
      state_topic: "/envoy/json"
      value_template: '{{ (value_json.meters.load.agg_p_mw| float / 1000) | round(0) | int(0)}}'
      unique_id: envoy_mqtt_consumption
      qos: 0
      unit_of_measurement: "W"
      state_class: measurement
      device_class: power

- sensor:
      name: envoy mqtt grid
      state_topic: "/envoy/json"
      value_template: '{{ (value_json.meters.grid.agg_p_mw| float / 1000) | round(0) | int(0)}}'
      unique_id: envoy_mqtt_grid
      qos: 0
      unit_of_measurement: "W"
      state_class: measurement
      device_class: power

- sensor:
      name: envoy mqtt solar production
      state_topic: "/envoy/json"
      value_template: '{{ (value_json.meters.pv.agg_p_mw | float / 1000) | round(2) | int(0)}}'
      unique_id: envoy_mqtt_solar_production
      qos: 0
      unit_of_measurement: "W"
      state_class: measurement
      device_class: power

@MKSG-MugunthKumar
Copy link
Author

One more thing:
The data/options.json has two variables ENVOY_PASSWORD and ENVOY_USER_PASS.
The python code uses only ENVOY_USER_PASS. I believe that could be removed.

@vk2him
Copy link
Owner

vk2him commented Oct 17, 2024

Hey,
Thanks for this and the additional comments - I'm a bit busy with my real job ATM so hope to review over weekend ...

I'm in Australia too, just south of Sydney. Yeah your idea about the sensors makes sense to leave it to the users - to my way of thinking if someone is clever enough to user github then the examples we provide should be sufficient for them to build their own. And for those that can't, they raise issues that I (might) be able to assist with - I have a Firmware V5 system so I can't test V6 or higher.

And thanks for the feedback re ENVOY_USER_PASS - I'll remove after double checking ... cheers

@MKSG-MugunthKumar
Copy link
Author

In that case, I've updated the example values for FW7 and FW8 with actual responses from my system. (And updated the sensors as well)

@vk2him vk2him mentioned this pull request Feb 16, 2025
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

Successfully merging this pull request may close these issues.

2 participants