Skip to content

Commit

Permalink
0.7.22
Browse files Browse the repository at this point in the history
* attempt to fix homeassistant auto discovery #1066
  • Loading branch information
lumapu committed Aug 4, 2023
1 parent 7afa9d1 commit 47c4ca6
Show file tree
Hide file tree
Showing 3 changed files with 6 additions and 3 deletions.
3 changes: 3 additions & 0 deletions src/CHANGES.md
Original file line number Diff line number Diff line change
@@ -1,5 +1,8 @@
# Development Changes

## 0.7.22 - 2023-08-04
* attempt to fix homeassistant auto discovery #1066

## 0.7.21 - 2023-07-30
* fix MqTT YieldDay Total goes to 0 serveral times #1016

Expand Down
2 changes: 1 addition & 1 deletion src/defines.h
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@
//-------------------------------------
#define VERSION_MAJOR 0
#define VERSION_MINOR 7
#define VERSION_PATCH 21
#define VERSION_PATCH 22

//-------------------------------------
typedef struct {
Expand Down
4 changes: 2 additions & 2 deletions src/publisher/pubMqtt.h
Original file line number Diff line number Diff line change
Expand Up @@ -391,9 +391,9 @@ class PubMqtt {
const char *devCls, *stateCls;
if (!total) {
if (rec->assign[mDiscovery.sub].ch == CH0)
snprintf(name, 32, "%s %s", iv->config->name, iv->getFieldName(mDiscovery.sub, rec));
snprintf(name, 32, "%s %s", iv->getFieldName(mDiscovery.sub, rec));
else
snprintf(name, 32, "%s CH%d %s", iv->config->name, rec->assign[mDiscovery.sub].ch, iv->getFieldName(mDiscovery.sub, rec));
snprintf(name, 32, "CH%d_%s", rec->assign[mDiscovery.sub].ch, iv->getFieldName(mDiscovery.sub, rec));
snprintf(topic, 64, "/ch%d/%s", rec->assign[mDiscovery.sub].ch, iv->getFieldName(mDiscovery.sub, rec));
snprintf(uniq_id, 32, "ch%d_%s", rec->assign[mDiscovery.sub].ch, iv->getFieldName(mDiscovery.sub, rec));

Expand Down

1 comment on commit 47c4ca6

@beegee3
Copy link
Contributor

@beegee3 beegee3 commented on 47c4ca6 Aug 4, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@lumapu pubMqtt.h Zeile 394: snprintf erwartet 2 Parameter (%s %s), bekommt aber nur einen.
Meintest du vielleicht:
snprintf(name, 32, "%s", iv->getFieldName(mDiscovery.sub, rec));

Hab' leider nicht mehr viel Zeit, konstruktive Vorschläge hier zu posten, auch wenn ich noch einiges mitlese.
Aber als ich den Beitrag #1063 und die Idee mit der SD Karte gelesen habe (gefällt mr 👍), dachte ich mir, vielleicht hilft auf die Schnelle auch meine (ewig alte) FTP Erweiterung. Die war mir ganz entfallen, zu der Zeit zuviele Mqtt Issues.
Hab' sie ausgegraben und an die dev. 0.7.22 angepasst. Nicht wichtig genug für einen pull request oder als Issue, aber einfach zu integrieren. Aber erst mal für dich zum Testen, für Produktivbetrieb braucht man konfigurierbare Settings (s.u). Auch, wenn's eigentlich nicht hierher gehört, daher hier die geänderten und die neuen Dateien als zip ftp Dateien.zip

Damit werden die Inverter Daten (keine Totals) jede Minute in eine CSV Datei auf einen FTP Server geschrieben.
In den geänderten Dateien (defines.h, app.h, app.cpp und settings.h) nach 'ftp' suchen, dann findest du alle Ergänzungen.
Es sind nur Ergänzungen, keine Änderungen an vorhandenem Code! Dazu noch der FTP publisher und der eigentliche FTP Client (nur die benötigten Funktionen, um Code und Speicherplatz klein zu halten).

In app::setup sind für's Testen statische FTP Vorgaben. Die müssten eigentlich über's Web Interface konfigurierbar gemacht werden: FTP 'server', 'user', 'passwort', 'port', 'subdirectory' (in das geschrieben wird) und für CSV noch ein 'locale', da z.B. im englischen ',' das Spaltentrennzeichen und '.' der Dezimalpunkt ist, im deutschen (Excel) aber ';' und ','.

Entweder du schmeißt es in die Tonne, oder ich wünsch' dir viel Spaß beim Testen. Wie's dir passt 😄

Please sign in to comment.