Verbesserungen aus meiner V3.2.6: Modbus-serial-Handling, JSON-MQTT-Antwort für livedata, Growatt V1.2.4-File und -Sim, Bugfixing #109

StefanNouza · 2024-12-27T20:09:51Z

Verbesserungen aus meiner alten V3.2.6 eingebracht:

Problem beim Laden des Filesystems aus PlatformIO behoben
verbessertes Modbus-serial-Handling
nur eine MQTT-Antwort für alle Daten zusammen (FHEM dankt es)
Growatt V1.2.4-Config-File und -Simulation eingebracht
einige zufällig entdeckte Bugs gefixt

Aenderungen_V3.3.0g.txt

tobiasfaust · 2024-12-28T06:03:58Z

src/main.cpp

-    dbg.begin(115200, SERIAL_8N1, Config->GetSerialRx(), Config->GetSerialTx()); // RX, TX, zb.: 33, 32
-    dbg.println("");
-    dbg.println("ready");
+    //Serial.begin(115200, SERIAL_8N1, Config->GetSerialRx(), Config->GetSerialTx()); // RX, TX, zb.: 33, 32 >>> hier würde "Serial" an "BaseConfig" angepasst werden...


den serialport kann man nach belieben umbiegen.
diese definition ist eine advanced option wenn man zb. eine mittels eines 2. ESP eine TransparentBridge anlegt um das echte SerialLog Remote zu lesen wenn der ESP selbst nicht starten will

ok, dafür also die Konfigurationsmlöglichkeit.

Bleibt noch das Initialisierproblem:

in commonlibs.h definierst Du dbg zu "Serial" oder zu "WebSerial"

in main.cpp / setup() wird zuerst ein Objekt der Klasse "BaseConfig" angelegt

das ist ja auch nötig, um die Seriell-Konfiguration einzulesen

allerdings arbeitet "BaseConfig" selbst schon mit "dbg", d.h. entweder dem default-Serial oder dem noch nicht initialisierten WebSerial

nachdem BaseConfig angelegt ist, wird WebSerial initialisiert (setBuffer() und begin()) bzw. Serial parametriert...

Ich habe mich mit WebSerial nicht auseinandergesetzt, vielleicht darf es ja schon vor begin() verwendet werden - zumindest wollte ich Dich auf diese Stelle aufmerksam machen.
Eine saubere Lösung (aus meiner eingeschränkten Sicht) habe ich leider auch nicht, BaseConfig ohne Debug-Ausgaben ist natürlich nicht gut.

tobiasfaust · 2024-12-28T06:07:33Z

src/mqtt.cpp

@@ -1,4 +1,5 @@
 #include "mqtt.h"
+#include "modbus.h"


die mqtt klasse muss für sich alleine stehen und darf ausser zur config Klasse keine weiteren abhängigkeiten haben. Die mqtt Klasse wird noch in anderen Projekten benötigt. Die EInbindung der modbus klasse ist auch nicht notwendig, zirkelbeziehungen sollten nicht entstehen.

Ja, der include sollte nicht da sein - hatte ich wohl vorübergehend gebraucht und dann vergessen.

tobiasfaust · 2024-12-28T06:10:52Z

src/mqtt.cpp

  dbg.printf("Initializing MQTT (%s:%d)\n", Config->GetMqttServer().c_str(), Config->GetMqttPort());
  espClient = WiFiClient();
-
+
+  #ifdef JSON_MQTT


es darf kein grosser buffer aufgebaut werden, jedes topic muss alleine für sich unabhängig gesendet werden. Bei grossen Definitionen wird einfach das json abgeschnitten und ungültiger code wird gesendet. im schlimmeren fall bootet der esp neu.

Wenn wirklich "eine grosse" Message notwendig ist, so kann man per http://IP/getitems alle daten bekommen. Das benutzen einige imho zb. in homeassist.
Ich nutze auch fhem, und fhem kommt sehr gut mit den einzelnen nachrichten klar

doch, dafür habe ich in denn modbus-Konstruktor oben

this->mqttJson.reserve(JSON_MSG_BUFFER);

aufgenommen, um einen 4,1k-Buffer zu haben. Das beseitigt abgeschnittene JSONs und beseitigt Abstürze (habe ich selbst schmerzhaft erarbeitet... :). Wenn man über meinen Weg gnadenlos alles ausgibt, treten die Probleme natürlich irgendwann wieder auf, wenn 4k Nutzdaten erreicht sind.

Und mein 2019er FHEM auf einem rasPi2 fraß sich mit einzeln gesendeten Topics nach 2-3Tagen zuverlässig fest - ein Verhalten, das mit der großen JSON-Antwort nie auftrat. Aber vielleicht ist das System ja auch etwas aus der Zeit gefallen...

tobiasfaust · 2024-12-28T06:11:56Z

src/modbus.h

-//#define DEBUGMODE
+#define DEBUGMODE
+
+#define JSON_MQTT               // define to get MQTT-data in one big JSON-answer (good for FHEM-based systems)


genau das soll vermieden werden. FHEM kommt mit den einzelnen topics gut klar. So läuft es bei mir auch

tobiasfaust · 2024-12-28T06:14:07Z

src/modbus.cpp

@@ -235,7 +243,7 @@ void modbus::LoadInverterConfigFromJson() {

  File regfile = LittleFS.open("/regs/"+this->InverterType.filename);
  if (!regfile) {
-    dbg.println("failed to open ModbusConfig.json file for writing");
+    dbg.printf("failed to open /regs/%s file\n", this->InverterType.filename.c_str());


ist schon in der aktuellen dev-branch behoben

ok, haben wir zeitgleich gefunden...

tobiasfaust · 2024-12-28T06:20:03Z

src/modbus.cpp

    File regfile = LittleFS.open("/regs/"+this->InverterType.filename);
    if (!regfile) {
-      dbg.println("failed to open ModbusConfig.json file for writing");
+      dbg.printf("failed to open /regs/%s file\n", this->InverterType.filename.c_str());


ist schon in der aktuellen dev-branch behoben

ok, haben wir zeitgleich gefunden...

tobiasfaust · 2024-12-28T06:23:56Z

src/modbus.cpp

@@ -765,21 +799,21 @@ void modbus::ParseData() {
      } else if (datatype == "string") {
        //********** handle Datatype String ***********//
        if (!posArray.isNull()) { 
-          char buffer[posArray.size()];
+          char buf[posArray.size()+1];        // set up c-string to hold all bytes and the trailing 0x00-char


warum buf und nicht buffer?
ist aber schon in der aktuellen dev-branch behoben

ich habe mal gelernt, daß unterschiedliche Variablen auch verschiedene Namen haben sollten, um die Übersicht zu behalten; und je kleiner der Lebensbereich, desto kürzer darf der Name sein - also buf.

Da hat aber jeder seine Philosophie, ich hatte wohl auch hier zeitgleich mit Dir die Lösung eingebracht...

haben wir zeitgleich gefunden...

tobiasfaust · 2024-12-28T06:24:48Z

src/modbus.cpp

@@ -989,7 +1049,7 @@ void modbus::GetRegisterAsJson(AsyncResponseStream *response) {

  File regfile = LittleFS.open("/regs/"+this->InverterType.filename);
  if (!regfile) {
-    dbg.println("failed to open ModbusConfig.json file for writing");
+    dbg.printf("failed to open /regs/%s file\n", this->InverterType.filename.c_str());


ist schon in der aktuellen dev-branch behoben

tobiasfaust · 2024-12-28T06:25:14Z

src/modbus.cpp

@@ -1081,7 +1144,7 @@ void modbus::LoadRegItems(std::vector<reg_t>* vector, String type) {

  File regfile = LittleFS.open("/regs/"+this->InverterType.filename);
  if (!regfile) {
-    dbg.println("failed to open ModbusConfig.json file for writing");
+    dbg.printf("failed to open /regs/%s file\n", this->InverterType.filename.c_str());


ist schon in der aktuellen dev-branch behoben

haben wir zeitgleich gefunden...

tobiasfaust · 2025-01-29T09:18:05Z

hi @StefanNouza
ich habe mal dein json vom Growatt übernommen. Da ich jetzt nicht alles mit dem bestehenden Growatt-SPH file abgleichen konnte, gibt es Register Abweichungen innerhalb der beiden Files? Oder macht es Sinn beide zusammenzuführen?
Das Originale SPH File ist von @thomas-99
Es funktionieren auch "Set" Commands, falls du welche hast könntest du diese hier auch noch integrieren. Eine gute Vorlage gibt es im Solax-X3.json.

Ansonsten habe ich erstmal keine weiteren Anpassungen hier im PR gefunden die zu übernehmen sind.

@thomas-99, könntest du das neue SPH File auch bei dir mal testen?

Stefan Nouza added 2 commits December 27, 2024 20:50

imported changes from my 3.2.6, did some bug-fixing

ad920ce

lost some string-conversions

d765617

tobiasfaust reviewed Dec 28, 2024

View reviewed changes

helper to get WiFi running without IMPROV

b16ed84

tobiasfaust added a commit that referenced this pull request Jan 29, 2025

add Growatt-SPH-V124 register file (thanks to @StefanNouza) (#109)

3ea9a20

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Verbesserungen aus meiner V3.2.6: Modbus-serial-Handling, JSON-MQTT-Antwort für livedata, Growatt V1.2.4-File und -Sim, Bugfixing #109

Verbesserungen aus meiner V3.2.6: Modbus-serial-Handling, JSON-MQTT-Antwort für livedata, Growatt V1.2.4-File und -Sim, Bugfixing #109

StefanNouza commented Dec 27, 2024

tobiasfaust Dec 28, 2024

StefanNouza Dec 30, 2024 •

edited

Loading

tobiasfaust Dec 28, 2024 •

edited

Loading

StefanNouza Dec 30, 2024

tobiasfaust Dec 28, 2024 •

edited

Loading

StefanNouza Dec 30, 2024

tobiasfaust Dec 28, 2024

tobiasfaust Dec 28, 2024

StefanNouza Dec 30, 2024

tobiasfaust Dec 28, 2024

StefanNouza Jan 2, 2025

tobiasfaust Dec 28, 2024

StefanNouza Dec 30, 2024

StefanNouza Jan 2, 2025

tobiasfaust Dec 28, 2024

tobiasfaust Dec 28, 2024

StefanNouza Jan 2, 2025

tobiasfaust commented Jan 29, 2025

Verbesserungen aus meiner V3.2.6: Modbus-serial-Handling, JSON-MQTT-Antwort für livedata, Growatt V1.2.4-File und -Sim, Bugfixing #109

Are you sure you want to change the base?

Verbesserungen aus meiner V3.2.6: Modbus-serial-Handling, JSON-MQTT-Antwort für livedata, Growatt V1.2.4-File und -Sim, Bugfixing #109

Conversation

StefanNouza commented Dec 27, 2024

Choose a reason for hiding this comment

StefanNouza Dec 30, 2024 • edited Loading

Choose a reason for hiding this comment

tobiasfaust Dec 28, 2024 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

tobiasfaust Dec 28, 2024 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

tobiasfaust commented Jan 29, 2025

StefanNouza Dec 30, 2024 •

edited

Loading

tobiasfaust Dec 28, 2024 •

edited

Loading

tobiasfaust Dec 28, 2024 •

edited

Loading