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 request: add pull of "Einheiten" and old data #1

Closed
AleXSR700 opened this issue May 3, 2022 · 27 comments
Closed

Feature request: add pull of "Einheiten" and old data #1

AleXSR700 opened this issue May 3, 2022 · 27 comments
Labels
enhancement New feature or request feature request Request a new ista EcoTrend feature outdated performance Performance stale

Comments

@AleXSR700
Copy link

AleXSR700 commented May 3, 2022

Hello @Ludy87

thank you for working on an ista Ecotrend integration for Home Assistant.

I wanted to give it a try and added your repository to HACS, installed and then added a package with the config.

The first data has been pulled. Excellent!

Would it be possible to pull all data available, i.e. old data and also the "Einheiten" that were consumed?
This would allow to also track the conversion being performed from "Einheiten" to kWh as one is the "Verbrauchseinheiten" which can be read locally from the sensors.

Maybe, if you are a member and feel like it, we can also discuss in a dedicated thread in Home Assistant Community Forum? I could imagine more people being interested in this :)

Thank you and looking forward to the official HACS release
Alex

@AleXSR700 AleXSR700 reopened this May 3, 2022
@Ludy87 Ludy87 added the feature request Request a new ista EcoTrend feature label May 4, 2022
@AleXSR700 AleXSR700 changed the title Integration not being loaded/no sensor added Feature request: add pull of "Einheiten" and old data May 4, 2022
@Ludy87 Ludy87 added enhancement New feature or request performance Performance wait in examination labels May 7, 2022
Ludy87 added a commit that referenced this issue May 7, 2022
Feature request: add pull of "Einheiten" and old data #1
Feature request: Set pull intervall #2
correction of value seperator  #3

Signed-off-by: Ludy87 <[email protected]>
@Ludy87 Ludy87 closed this as completed May 7, 2022
@AleXSR700
Copy link
Author

AleXSR700 commented May 8, 2022

Thank you for taking my requests into account so quickly!

Testing right now :)

About "Einheiten" , I think there may have been a misunderstanding. I meant the "Verbrauchseinheiten" which are simply listed as "Einheiten" (see picture). The unit is correctly pulled (in my case kWh).

Screenshot_20220508-090310_ista EcoTrend.jpg

186 Verbrauchseinheiten = 202,7 kWh

@Ludy87
Copy link
Owner

Ludy87 commented May 8, 2022

@AleXSR700 du kannst jetzt zwischen Einheiten und kwh wählen, um kwh angezeigt zu bekommen muss unit: "kwh" gesetzt werden sonst sind es nur Einheiten

@AleXSR700
Copy link
Author

AleXSR700 commented May 8, 2022

Ah, verstehe.
Und wenn ich beides möchte (ursprüngliches Ziel)?
Wird dann ein zweiter Sensor erzeugt?

@Ludy87
Copy link
Owner

Ludy87 commented May 8, 2022

Nein, das wird noch kommen;)

@Ludy87
Copy link
Owner

Ludy87 commented May 8, 2022

@AleXSR700 du kannst aber auch jetzt schon beide Werte auslesen, da beides übertragen wird. Du müsstest dir ein Template o.ä. anlegen.

@Ludy87 Ludy87 reopened this May 8, 2022
@AleXSR700
Copy link
Author

AleXSR700 commented May 9, 2022

Hallo @Ludy87
Ich hatte mich die ganze Zeit gewundert, wieso keine neue Daten geholt wurden. Jetzt habe ich festgestellt, dass sie geholt werden, aber für jeden Monat wurde eine neue Entity erzeugt und die ursprüngliche wird nicht mehr ergänzt.

Ich habe also aktuell mehrere entities heating_2022_*

ecotrend_ista:
  email: !secret ista_ecotrend_user
  password: !secret ista_ecotrend_password
  year:
    - 2022
  scan_interval: 43199 #11:59:59

Zudem wird nur die Verbrauchseinheit abgerufen, nicht aber die kWh, wenn man keine Angabe in der config macht.

@Ludy87
Copy link
Owner

Ludy87 commented May 9, 2022

Zudem wird nur die Verbrauchseinheit abgerufen, nicht aber die kWh, wenn man keine Angabe in der config macht.

So ist es, füge unit: "kwh" hinzu https://github.com/Ludy87/ecotrend-ista#optional

Die Entity warmwater_XXXXXXXXX und heating_XXXXXXXXX zeigt immer die aktuelle Auswertung vom Vormonat an.

@AleXSR700
Copy link
Author

AleXSR700 commented May 9, 2022

Aber wenn man pro Monat eine entity hat, dann lassen sich die Daten nicht mehr auswerten. Die Idee wäre ja, dass man eine Entity für "Einheiten" und eine für kWh hat und in diesen beiden Entities alle Daten bereitliegen.
Sonst müsste man seine Plots ja jeden Monat um eine weitere Entity ergänzen, nur um einen neuen Datenpunkt zu haben.

Außerdem spamt einem dann ista-ecotrend die Entities voll ;-)

Zur Config:
Und wenn man beides möchte? Gibt man dann

unit:
 - kWh
 - Einheiten

an?

@Ludy87
Copy link
Owner

Ludy87 commented May 9, 2022

Du beschreibst gerade zwei verschiede Sachen.

  1. Das/Die Entity ohne Jahres und Monats sind die Aktuellen, bei Monatswechsel, wird diese entsprechend aktuallisiert.
  2. In jeder Entity sind immer beide Werte enthalten, daher ist es eigentlich nicht sinnig, dass kwh und Einheiten bzw. m³ angelegt werden.

Außerdem spamt einem dann ista-ecotrend die Entities voll ;-)
pro Jahr 12 :D

Wenn du

  year:
    - 2022

weglässt, dann hast du nur die zwei

@AleXSR700
Copy link
Author

AleXSR700 commented May 9, 2022

Nunja, das Problem ist, dass der "normale" Sensor nur aktuelle und zukünftige Daten holt. Da die Daten aber auf dem Server zur Verfügung stehen, wäre es ja am besten, wenn er alle Daten vom Server abrufen würde. So hätte man im Falle eines Datenbankcrashes o.ä. keine Probleme.
Man hätte immer alle Daten, die auf dem Server liegen. Egal ob alt oder neu.

Für die erzeugten, monatlichen Sensoren sehe ich spontan keinen Anwendungsfall. Aber das ist nur persönliche Überlegung. Andere möchten vielleicht einen Sensor pro Datenpunkt :-)
Mit dem aktuell Ansatz müsste man nämlich sämtliche Monatssensoren mit dem "normalen" mergen, wenn man die alten Daten ebenfalls haben möchte.

Deshalb, aus meiner ganz persönlichen Sicht, bräuchte es nur einen Sensor pro Einheit (sprich einen für Einheiten und einen für kWh). Und dieser Sensor ruft alle alten sowie künftigen Daten automatisch ab. So hat man immer alle vergangenen und künftigen Daten zu einem Parameter in nur einem Sensor und kann diesen wunderbar plotten.
Und wenn man Verbrauchseinheiten und kWh in je einem Sensor hat, kann man die korrelation beider mit nur zwei Sensoren plotten.

@Ludy87
Copy link
Owner

Ludy87 commented May 9, 2022

Mir ist nicht bekannt, dass man alte Daten restoren kann, ohne das man aktiv in die DB diese einpflegen kann.

@AleXSR700
Copy link
Author

Du meinst, dass man keinen Datumswert schreiben kann?

@Ludy87
Copy link
Owner

Ludy87 commented May 9, 2022

Ist mir nicht bekannt

@Ludy87
Copy link
Owner

Ludy87 commented May 9, 2022

Zur Config: Und wenn man beides möchte? Gibt man dann

unit:
 - kWh
 - Einheiten

an?

Dazu ein Template erstellen: https://github.com/Ludy87/ecotrend-ista#template und schon kann es los gehen

@AleXSR700
Copy link
Author

Ist mir nicht bekannt

So wie es aussieht wäre es möglich, wenn die Integration die Werte direkt in die DB schreibt. Dann könnten auch alte Werte abgerufen und gespeichert werden.
Das wäre vor allem bei neuen Usern und auch bei Wiederherstellungen o.ä. interessant.

Da sich auch recht oft Dinge im HA core und der Datenbank ändern, wäre das sehr nützlich.

@Ludy87
Copy link
Owner

Ludy87 commented Jun 2, 2022

Ich werde mich mit der Datenbank beschäftigen, wird aber nicht sofort passieren, da meine andere Integration Momentan mehr Zeit beansprucht, als mir lieb ist.

@AleXSR700
Copy link
Author

Ich habe die Tage noch ein wenig probiert und man kann den Datenbankwert nachträglich ändern. Es müsste also möglich sein den Bezugszeitpunkt in die Datenbank zu schreiben anstatt des Datums der Erzeugung des Datensatzes.

Leider konnte ich noch nicht in Erfahrung bringen wie man das dann bei einer Integration macht.

@Ludy87
Copy link
Owner

Ludy87 commented Nov 5, 2022

Wie hast du es denn angestellt?

@ekeih
Copy link

ekeih commented Nov 19, 2022

Thanks for this integration, really appreciate it 🙏

If I understand it correctly https://community.home-assistant.io/t/import-old-energy-readings-for-use-in-energy-dashboard/341406/9 shows how integrations can add data with a different timestamp than now.

@Firehome2022
Copy link

Hallo Ludy87.
erst einmal vielen dank für die Integration. Ich habe sie gestern entdeckt und bin froh dass es eine Integration für ISTA gibt.
Ich habe den Eintrag in die configuration.yaml übernommen. Das template liegt in der template.yaml.
Mein Problem ist das anscheinend keine Daten im Home Assistant ankommen. Die Anmeldung mit dem ISTA User/Passwort über die Ecotrend App funktioniert also müssen die User Daten passen. Wenn ich es richtig verstehe müssten doch bei den Entitäten neue Entitäten zu finden sein oder sehe ich das falsch. Ich bin noch nicht so lange mit HA dran daher hoffe ich das die Frage / Problem nicht zu blöd ist.
Gruß Firehome

@AleXSR700
Copy link
Author

@Ludy87 : do you think you will be able to pull and process old data?

@Ludy87
Copy link
Owner

Ludy87 commented Jun 18, 2023

@AleXSR700 Getting the old data isn't the problem, but integrating it is. I've been trying to integrate this for a long time, and the database keeps breaking. As soon as I have a solution I will incorporate it.

@AleXSR700
Copy link
Author

Hello,
any luck getting the old data written to the database?

@Ludy87
Copy link
Owner

Ludy87 commented May 21, 2024

Hello, any luck getting the old data written to the database?

I think I won't try it until HA provides it.

@AleXSR700
Copy link
Author

Is there a FR or PR for that?

@Ludy87 Ludy87 added stale and removed wait in examination labels Jun 23, 2024
Copy link
Contributor

github-actions bot commented Jul 1, 2024

This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@github-actions github-actions bot closed this as not planned Won't fix, can't repro, duplicate, stale Jul 1, 2024
Copy link
Contributor

This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jul 31, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request feature request Request a new ista EcoTrend feature outdated performance Performance stale
Projects
None yet
Development

No branches or pull requests

4 participants