britkat1980 / giv_tcp

TCP connection (from inverter) and MQTT implementation
78 stars 37 forks source link

Request: AIO - Add battery temperature and voltage #161

Closed blastehh closed 4 months ago

blastehh commented 9 months ago

I had a look at the REST output and under raw.inverter there is temp_battery and v_battery available, can these be added so HA can see them? Would it also be possible to send the inverter temp, battery temp, and battery voltage to influxdb?

blastehh commented 9 months ago

Attaching my output for reference aio-rest.json

7Van737 commented 8 months ago

Here is the code I found on a GE facebook thread that creates the Battery Temperature as an entity. Works for me.

sensor:

Created a new file called mqtt.yaml in the config folder with 'mqtt: !include mqtt.yaml' added to the configeration.yaml

7Van737 commented 8 months ago

EDIT - for some reason the dash is being replaced with a bullet point in the code, please use a Dash "-"

And following the same logic here is the code for the battery voltage which appears to work.

sensor:

PianSom commented 8 months ago

Thanks for the hints. For me to get this to work I have to have the capital E in GivEnergy

PS include code by starting the block with three backticks

like this ```
7Van737 commented 8 months ago
like this ```

Thanks for this. I'm actually working my way down the issues and a better example of the voltage (and battery current) is given in post #4 of this thread: https://github.com/britkat1980/giv_tcp/issues/130 I'm just copying/pasting what I can find (and adapting as necessary) so can't claim any the credit for these solutions.

Wil have a look at the battery card link above now. Thanks

PianSom commented 8 months ago

Deleted the comment, because it doesn't have either voltage or temperature! Oops

blastehh commented 8 months ago

Here is the code I found on a GE facebook thread that creates the Battery Temperature as an entity. Works for me.

sensor:

* name: "AIO BMS Temp"
  unique_id: aio-bms-temp
  state_topic: "Givenergy/CHXXXXXXXX/raw/invertor/temp_battery"
  device_class: Temperature
  unit_of_measurement: "°C"
  state_class: measurement
  icon: "mdi:thermometer"

Created a new file called mqtt.yaml in the config folder with 'mqtt: !include mqtt.yaml' added to the configeration.yaml

this is perfect, thanks.

for those who want to copy:

sensor:
  - name: "AIO BMS Temp"
    unique_id: aio-bms-temp
    state_topic: "Givenergy/CHXXXXXX/raw/invertor/temp_battery"
    device_class: Temperature
    unit_of_measurement: "°C"
    state_class: measurement
    icon: "mdi:thermometer"

  - name: "AIO Voltage"
    unique_id: aio-voltage
    state_topic: "Givenergy/CHXXXXXX/raw/invertor/v_battery"
    device_class: Voltage
    unit_of_measurement: "V"
    state_class: measurement
7Van737 commented 8 months ago

Thanks for the hints. For me to get this to work I have to have the capital E in GivEnergy

I think your capital E comes from how you will have typed "GivEnergy" as the 'MQTT topic ' in the GivTCP configuration.

blastehh commented 8 months ago

yep, it refers to this image Mine is set to GEAIO

PhillWalker commented 5 months ago

this would be great to be incorporated, very useful data to have

britkat1980 commented 4 months ago

New (currently in beta) version includes much more AIO battery data inc BCU data and voltage/temps

PianSom commented 4 months ago

Hey @britkat1980 - thanks for the continuing work.

I don't suppose you are feeling charitable enough to build a docker container version for those of us who use HA Container and can't use Add-ons?

Pretty please?

britkat1980 commented 4 months ago

It’s already there, checkout the beta branch on dockerhub

PianSom   @. @.)>   14 May 2024 at 12:59:01   wrote:

Hey @britkat1980 (https://github.com/britkat1980) - thanks for the continuing work.

I don't suppose you are feeling charitable enough to build a docker container version for those of us who use HA Container and can't use Add-ons?

Pretty please?

Reply to this email directly, view it on GitHub (https://github.com/britkat1980/giv_tcp/issues/161#issuecomment-2110018731), or unsubscribe (https://github.com/notifications/unsubscribe-auth/AQPLJBRKVLFZCRJ7KZOFI63ZCH4AFAVCNFSM6AAAAABA7ZDKNWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCMJQGAYTQNZTGE).

You are receiving this because you were mentioned.Message ID: @.***>

PianSom commented 4 months ago

Blow me - so it is.

I did check before posting - but looked for a beta tag in the main gif_tcp-ma, rather than looking for a beta branch.

Apologies!