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 charger: Simulation Energiezähler während des Ladens #1559

Open
wants to merge 26 commits into
base: master
Choose a base branch
from

Conversation

tensing2
Copy link

@tensing2 tensing2 commented Sep 3, 2021

Wir (tensing2 und philipp123) haben jetzt mehrere enhancements hier eingecheckt:

  • Simulation des Energiezählers vom go-e charger auf Wunsch. Dieses Feature kann in der Konfiguration eingeschaltet werden
    Dadurch springt die Anzeige beim Tagesgraphen nicht mehr bei jeder vollen kWh, sondern es wird der reale Leistungswert des Ladevorganges angezeigt
  • Im Ladelog werden jetzt auch Kommawerte bei der geladenen Menge angezeigt, nicht mehr Ladungen mit 0kWh wenn in der Realität viellecht 0,99kWh geladen wurden.
  • Korrektur der doch sehr ungenauen Messwerte durch einen Korrekturfaktor einstellbar in der Konfiguration. Mein charger ist z.B. ca. 8% ungenau, was einmal ausgemessen werden und mit dem festen Faktor korrigiert werden kann.
  • Beim Strom auf der Status-Seite wird jetzt eine Nachkommastelle angezeigt (kommt von der go-e API, wurde aber bisher abgeschnitten.

Das ganze haben wir beide recht ausführlich getestet.

Diskussion hier: https://openwb.de/forum/viewtopic.php?f=9&t=3937

@tensing2 tensing2 changed the title Patch goe enhancement go-e charger: Simulation Energiezähler während des Ladens Sep 3, 2021
philipp123ow and others added 21 commits September 6, 2021 14:15
Implementierung Korrekturfaktor und Simulation nur wenn Parametrisiert
Implementierung Korrekturfaktor und Simulation nur wenn Parametrisiert
Implementierung Korrekturfaktor und Simulation nur wenn Parametrisiert
Aufnahme goesimulationlp1-3 und
goecorrectionfactorlp1-3
Implementierung Korrekturfaktor und Simulation nur wenn Parametrisiert
Simulation mit go-e Zählerstand initialisiert
-Kleine Umformatierungen
- Verschiebung Simulation in Bereich, dass nur gerechnet wird, wenn Simulation auch gewünscht
- Bisher gab es das Problem, dass bei jedem Neustart der Zähler des go-e gesetzt wurde. Nun wird ramdisk/pluggedladunglp1startkwh nach einem Neustart für die Initialisierung für llkw verendet
Hinweis Tensing2 umgesetzt: Der Startwert der Simulation muss ebenfalls pluggedladunglp1startkwh und nicht go-e-Zählerstand sein.
Und Korrektur Einrückungen
Aufnahme des kWh-Counter bei Reboot in die Radisk, um die SImulation des go-e mit dem letzten simulierten Zählerstand zu versorgen.
Da pluggedladunglp1startkwh der falsche Parameter ist ersetzt mit dem neuen Paramter temp_kWhCounter_lp1
* Korrektur Kommentare

* Erweiterung auf LP2

* Erweiterung Simulation auf LP3

* Umstellung Strom auf rekwh

* Umstellung Strom auf rekwh

* Umstellung Strom auf rekwh

Co-authored-by: philipp123ow <[email protected]>
Merge Branch goe_gorrectionfactor from Philipp
@xerion3800
Copy link
Contributor

Hallo ich wollte den PR mal mit meinen beiden go-E´s testen bekomme aber Syntax Error:

(standard_in) 1: syntax error
modules/goelp2/main.sh: Zeile 14: watt*/100000: Syntax Fehler: Operator erwartet. (Fehlerverursachendes Zeichen ist \"/100000\").
(standard_in) 1: syntax error
modules/goelp1/main.sh: Zeile 14: watt*/100000: Syntax Fehler: Operator erwartet. (Fehlerverursachendes Zeichen ist \"/100000\").
(standard_in) 1: syntax error

xerion3800 added a commit to xerion3800/openWB that referenced this pull request Nov 12, 2021
@tensing2
Copy link
Author

tensing2 commented Nov 12, 2021

Hallo ich wollte den PR mal mit meinen beiden go-E´s testen bekomme aber Syntax Error:

(standard_in) 1: syntax error
modules/goelp2/main.sh: Zeile 14: watt*/100000: Syntax Fehler: Operator erwartet. (Fehlerverursachendes Zeichen ist \"/100000\").
(standard_in) 1: syntax error
modules/goelp1/main.sh: Zeile 14: watt*/100000: Syntax Fehler: Operator erwartet. (Fehlerverursachendes Zeichen ist \"/100000\").
(standard_in) 1: syntax error

Du hast vermutlich das Update manuell gemacht, indem du die Dateien auf deinem PI ersetzt hast. Das Problem ist, dass dann die Konfigurationsdatei nicht geändert wird. Bei einem automatischen Update (wenn der Code in den Trunk gemerged ist) würde die runs/updateConfig.sh ausgeführt werden, die die zusätzlichen Einträge in der openwb.conf automatisch vornimmt.
Du kannst das manuell machen, indem du die Zeilen

goesimulationlp1=0
goecorrectionfactorlp1=1
goesimulationlp2=0
goecorrectionfactorlp2=1
goesimulationlp3=0
goecorrectionfactorlp3=1

an deine openwb.conf anhängst. Dann kannst du in der Konfiguration die Simulation aktivieren.

@xerion3800
Copy link
Contributor

Hallo ich wollte den PR mal mit meinen beiden go-E´s testen bekomme aber Syntax Error:

(standard_in) 1: syntax error
modules/goelp2/main.sh: Zeile 14: watt*/100000: Syntax Fehler: Operator erwartet. (Fehlerverursachendes Zeichen ist \"/100000\").
(standard_in) 1: syntax error
modules/goelp1/main.sh: Zeile 14: watt*/100000: Syntax Fehler: Operator erwartet. (Fehlerverursachendes Zeichen ist \"/100000\").
(standard_in) 1: syntax error

Du hast vermutlich das Update manuell gemacht, indem du die Dateien auf deinem PI ersetzt hast. Das Problem ist, dass dann die Konfigurationsdatei nicht geändert wird. Bei einem automatischen Update (wenn der Code in den Trunk gemerged ist) würde die runs/updateConfig.sh ausgeführt werden, die die zusätzlichen Einträge in der openwb.conf automatisch vornimmt. Du kannst das manuell machen, indem du die Zeilen

goesimulationlp1=0
goecorrectionfactorlp1=1
goesimulationlp2=0
goecorrectionfactorlp2=1
goesimulationlp3=0
goecorrectionfactorlp3=1

an deine openwb.conf anhängst. Dann kannst du in der Konfiguration die Simulation aktivieren.

Genau das war es. 👍 Ich bin noch recht frisch mit openWB unterwegs. Dann teste ich mal. Gibt es schon einen Fahrplan wann der PR in den Master gemerget wird?

@tensing2
Copy link
Author

Genau das war es. 👍 Ich bin noch recht frisch mit openWB unterwegs. Dann teste ich mal. Gibt es schon einen Fahrplan wann der PR in den Master gemerget wird?

Das fragen wir uns auch..

@xerion3800
Copy link
Contributor

xerion3800 commented Nov 12, 2021

Ein Problem habe ich jetzt doch noch. Ich bekomme jetzt keine kWh/km mehr angezeigt. Was könnte ich da prüfen?
image
Im Debug Log tritt das auf, weis aber nicht ob das dazu gehört:

(standard_in) 1: syntax error ValueError: invalid literal for int() with base 10: '' wattposh=int(f.read()) File "/var/www/html/openWB/runs/simcount.py", line 24, in <module> Traceback (most recent call last): ValueError: invalid literal for int() with base 10: '' wattposh=int(f.read()) File "/var/www/html/openWB/runs/simcount.py", line 24, in <module> Traceback (most recent call last):

@tensing2
Copy link
Author

Ein Problem habe ich jetzt doch noch. Ich bekomme jetzt keine kWh/km mehr angezeigt. Was könnte ich da prüfen? image Im Debug Log tritt das auf, weis aber nicht ob das dazu gehört:

(standard_in) 1: syntax error ValueError: invalid literal for int() with base 10: '' wattposh=int(f.read()) File "/var/www/html/openWB/runs/simcount.py", line 24, in <module> Traceback (most recent call last): ValueError: invalid literal for int() with base 10: '' wattposh=int(f.read()) File "/var/www/html/openWB/runs/simcount.py", line 24, in <module> Traceback (most recent call last):

was ist in der Datei /var/www/html/openWB/ramdisk/goewatt0pos und goe2watt0pos?
Da sollte nur eine ganze Zahl drinstehen, sonst nichts.

@xerion3800
Copy link
Contributor

Beide Dateien sind leer.

@tensing2
Copy link
Author

Beide Dateien sind leer.

Das wird das Problem sein. Woran das liegt, müssen wir noch erforschen. Schalte mal bitte die Simulation wieder aus. Dann siehst du im Status unter den Ladepunkten den Zählerstand der go-e charger in kWh. Schreib jetzt bitte mit irgendeinem Editor diesen Wert mulitipliziert mit 3600000 in die Dateien (für ersten und 2. Ladepunkt). Anschließend die Simulation wieder einschalten.
Überprüf dann auch noch ob die anderen Dateien die mit goe bzw. goe2 beginnen evtl. ebenfalls leer sind:
goewatt0neg muss 0 drinstehen
goengkwh ebenfalls 0
goeposkwh der Zählerstand *1000

Eigentlich sollten diese Dateien beim update auch richtig beschrieben werden. Könnte sein, dass es an deinem manuellen update liegt. Das teste ich nochmal.

@xerion3800
Copy link
Contributor

Danke für deine Hilfe. Die letzten beiden Dateien fehlten komplett und bei den anderen habe ich die Werte entsprechend eingetragen.
Fehler tauchen jetzt im Log nicht mehr auf aber in der Anzeige steht nach wie von 0Wh / 0km

@philipp123ow
Copy link
Contributor

Danke für deine Hilfe. Die letzten beiden Dateien fehlten komplett und bei den anderen habe ich die Werte entsprechend eingetragen. Fehler tauchen jetzt im Log nicht mehr auf aber in der Anzeige steht nach wie von 0Wh / 0km

Neu gestartet hast du mal? Welche Version setzt du ein?

@xerion3800
Copy link
Contributor

Danke für deine Hilfe. Die letzten beiden Dateien fehlten komplett und bei den anderen habe ich die Werte entsprechend eingetragen. Fehler tauchen jetzt im Log nicht mehr auf aber in der Anzeige steht nach wie von 0Wh / 0km

Neu gestartet hast du mal? Welche Version setzt du ein?

Nein neu gestartet hatte ich nicht. Kann ich aber gern tun aber warum sollte das denn helfen wenn die Daten da sind? Oder gibt es noch andere Abhängigkeiten?
Was meinst du Mit Version? Ich habe den PR selber auf dem Pi kopiert bzw. Mittlerweile in meinen fork gemergt.

@philipp123ow
Copy link
Contributor

Nein neu gestartet hatte ich nicht. Kann ich aber gern tun aber warum sollte das denn helfen wenn die Daten da sind? Oder gibt es noch andere Abhängigkeiten? Was meinst du Mit Version? Ich habe den PR selber auf dem Pi kopiert bzw. Mittlerweile in meinen fork gemergt.

Beim Restart wird die Ramdisk neu initialisiert. Das kann vielleicht helfen.

Mit Version meine ich die openWB-Version. Also Nightly oder ähnliches.

Ich hatte einmal ein ähnliches Problem. Mir war bei der manuellen Installation ein Fehler unterlaufen. Danach war irgend etwas in der Ramdisk so verbogen, dass auch ein Neustart nicht geholfen hat. Damals hat folgendes geholfen:
Update auf die aktuelle Nightly (einfach nur, damit alles überschrieben wird) und dann nochmals erneut manuelle Nachinstallation mit allen Schritten.

Du kannst alternativ auch den Pfad auf deinen Fork ändern und aus diesem Updatds ziehen, solange der PR nicht gemerged ist.

@xerion3800
Copy link
Contributor

Nein neu gestartet hatte ich nicht. Kann ich aber gern tun aber warum sollte das denn helfen wenn die Daten da sind? Oder gibt es noch andere Abhängigkeiten? Was meinst du Mit Version? Ich habe den PR selber auf dem Pi kopiert bzw. Mittlerweile in meinen fork gemergt.

Beim Restart wird die Ramdisk neu initialisiert. Das kann vielleicht helfen.

Mit Version meine ich die openWB-Version. Also Nightly oder ähnliches.

Ich hatte einmal ein ähnliches Problem. Mir war bei der manuellen Installation ein Fehler unterlaufen. Danach war irgend etwas in der Ramdisk so verbogen, dass auch ein Neustart nicht geholfen hat. Damals hat folgendes geholfen: Update auf die aktuelle Nightly (einfach nur, damit alles überschrieben wird) und dann nochmals erneut manuelle Nachinstallation mit allen Schritten.

Du kannst alternativ auch den Pfad auf deinen Fork ändern und aus diesem Updatds ziehen, solange der PR nicht gemerged ist.

Okay das mit den Neustart und der Ramdisk wusste ich noch nicht . Danke für deine Unterstützung. Weist du wo ich den Pfad anpassen kann, sodas ich von meinen Fork aus updaten kann?

@philipp123ow
Copy link
Contributor

Okay das mit den Neustart und der Ramdisk wusste ich noch nicht . Danke für deine Unterstützung. Weist du wo ich den Pfad anpassen kann, sodas ich von meinen Fork aus updaten kann?

In der Art
$ git remote rm origin
$ git remote add origin git@newserver:project.git
$ git remote show origin
Auf deinem Pi. git@newserver:project.git musst du mit deinem Fork ersetzen. Hat dann den Vorteil, dass du dir nicht bei jedem Nightly-Update alles wieder zerschießt. Wenn der PR gemerged ist, kannst du so auch wieder zurückstellen auf Snaptec.

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.

3 participants