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] HA Discovery GEN_TOT für Statistik #29

Closed
subivoodoo opened this issue Dec 10, 2023 · 12 comments
Closed

[FEATURE] HA Discovery GEN_TOT für Statistik #29

subivoodoo opened this issue Dec 10, 2023 · 12 comments

Comments

@subivoodoo
Copy link

subivoodoo commented Dec 10, 2023

Is your feature request related to a problem? Please describe

Die Home Assistant Integration mittels MQTT Discovery funktioniert gut und zeigt alle Werte wie gewünscht.

Jedoch kann z.B. der Sensor sensor.epever2mqtt_ep_1_gen_tot in HA nicht als Solarproduktion oder Batteriespeicher ausgewählt werden im Energy Dashboard und es werden auch keine Statistiken gespeichert.

Describe the solution you'd like

Damit die generierte Energie im HA Energy Dashboard einfach genutzt/ausgewählt werden kann, wäre es gut wenn via Discovery die state_class: total_increasing mitgegeben werden würde beim Total Energie Sensor (ich vermute dies wird aktuell nicht gesendet).

Describe alternatives you've considered

Als Workaround kann ein zusätzliches Template angelegt werden z.B. wie folgt:

template:
  - sensor:
    - name: "Solarenergie in Batterie"
      unique_id: epever.charge_total_sensor
      state: >-
          {{ 
            [ states('sensor.epever2mqtt_ep_1_gen_tot'), 
            ] | map('float') | sum
          }}
      unit_of_measurement: kWh
      device_class: energy
      state_class: total_increasing
      attributes:
        last_reset: "1970-01-01T00:00:00+00:00"
      availability: >-
          {{ 
            [ states('sensor.epever2mqtt_ep_1_gen_tot'), 
            ] | map('is_number') | min
          }}

Additional context

No response

@softwarecrash
Copy link
Owner

Hallo,
aktuell wird bei GEN_TOT die device_class mit energy angegeben, das könnte ich zu total_incerasing ändern.
ob das funktioniert kannst du gern mal testen wenn du mit dem mqttexplorer auf deinen mqtt broker schaust dort unter homeassistant/sensors/EP... den GEN_TOT punkt suchst und in den json daten das entsprechend änderst

@subivoodoo
Copy link
Author

Hallo,

Habs gestern im Text verwechselt (habs editiert)... die device_class müsste energy bleiben, es müsste zusätzlich state_class: total_increasing mitgegeben werden damit in HA die Statistik aktiv ist und dieser Sensor damit im Energy Dashboard auswählbar wird.

Habe das wie vorgeschlagen mit dem MQTTExplorer ausprobiert mit zusätzlich "stat_cla": "total_increasing"
image

sieht dann in HA so aus als Sensor in den Entwicklertools:

image

und es funktioniert, man kann es als "Batterie" im Dashboard auswählen:
image

@softwarecrash
Copy link
Owner

danke für die tests.
stat_class ist aktuell garnicht implementiert, da ich HA wenig bis garnicht kenne, bräuchte ich für alle betreffenden datenpunkte die richtigen stat zuweisungen, dann könnte ich mal schauen ob ich das noch unter bekomme

@subivoodoo
Copy link
Author

Ich bin auch kein HA Profi... Statistiken sind mehr oder weniger länger abgelegte Werte z.B. alle 5 Minuten quasi "für immer" abgespeichert damit darauf basierend u.a. das Energy Dashboard dargestellt werden kann und auch die Vorjahreswerte etc. historisiert sind. Sonst sind Werte in HA defaultmässig nur wenige Tage (7 oder 10) gespeichert und fliegen danach raus. Die vorhandenen Sensoren, zu welchen Statistiken gespeichert sind, sind in HA unter Entwicklertools>Statistik aufgelistet.

Es müssten sicher nicht alle Sensoren historisiert werden, die kWh wären jedoch schon sinnvoll.

Bei der AhoyDTU z.B. sind alle Energie-Sensoren (Wh und kWh) definiert als state_class: total_increasing, alle Spannungen/Ströme/Leistungen/Temperatur als state_class: measurement und Texte z.B. haben nix gesetzt... none könnte da gehen?

@Jarnsen
Copy link

Jarnsen commented Dec 11, 2023

ne, none funktioniert nicht... das haben wir schon versucht

@Jarnsen
Copy link

Jarnsen commented Dec 12, 2023

Rest ist mit SoftwareCrash besprochen, er "freut" sich das noch einzubauen ;-)

@subivoodoo
Copy link
Author

Cool danke! Es eilt aber überhaupt nicht, der Workaround funktioniert ja auch.

@thitcher
Copy link

Kann man auch ganz einfach mit einem Riemann-Summenintegral zusammenbauen, ohne die JSON zu editieren.
Falls jemand (wie ich :) ) mit JSON und MQTT auf Kriegsfuß steht

@softwarecrash
Copy link
Owner

@subivoodoo probier die mal, das sollte die entsprechenden einträge anlegen
EPEver2MQTT_d1_mini_2.2.0A-1.zip

@Kloana001
Copy link

Kloana001 commented Dec 13, 2023

Screenshot 2023-12-13 194029
Screenshot 2023-12-13 193704

hallo, gerade ausprobiert - siehe Bilder....
dankeschön....

@subivoodoo
Copy link
Author

Kann ich auch bestätigen, jetzt ist alles ohne Workaround auswählbar, danke!!

IMG_20231213_200942

@softwarecrash
Copy link
Owner

fix with V2.2.1

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

5 participants