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.

Sign up for GitHub

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

Jump to bottom

support for KOSTAL Smart Energy Meter #61

Merged
merged 5 commits into from
Jan 2, 2020
Merged

support for KOSTAL Smart Energy Meter #61

merged 5 commits into from
Jan 2, 2020

Conversation

kartenkarsten
Copy link
Contributor

No description provided.

andig
andig requested changes Dec 26, 2019
View reviewed changes
README.md Outdated
@@ -261,6 +262,7 @@ this as the meter ID.
- **SBC ALE3**: This compact Saia Burgess Controls meter is comparable to the SDM630.
It has two tariffs, both import and export depending on meter version and compact (4TE). It's often used with Viessmann heat pumps.
- **BE MPM3PM**: Compact (4TE) three phase meter.
- **KOSTAL Smart Energy Meter**: Import Export data are currently not implemented.
Copy link
Contributor

Choose a reason for hiding this comment

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

Gibts die bei dem Zähler/ sollten die hinzu gefügt werden?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

siehe auskommentierte Zeilen in models.go ich vermute einige von denen beinhalten die Im- bzw Export daten

README.md Outdated
@@ -236,6 +236,7 @@ manuals for definitive guidance):
| SBC ALE3 | 3 | + | + | + | + | + | + | - | - |
| ABB A/B-Series | 3 | + | + | + | + | + | + | + | + |
| BE MPM3MP | 3 | + | + | + | + | + | + | - | - |
| KOSTAL Smart Ene | 3 | + | + | + | + | - | - | - | - |
Copy link
Contributor

Choose a reason for hiding this comment

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

Bitte ausschreiben

Copy link
Contributor Author

Choose a reason for hiding this comment

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

ich habe dort den Text angegeben den mir ./mbmd inspect -a 192.168.2.101:502 -d SUNS:255 zurück liefert. Kann aber gerne auch den kompletten namen angeben.

@andig
Copy link
Contributor

andig commented Dec 26, 2019 •
edited
Loading

Danke für den PR! Aus Interesse: ist das SEM mit dem WR über RS485 verbunden und wird für Laststeuerung eingesetzt?

@andig andig added the enhancement New feature or request label Dec 26, 2019
@kartenkarsten
Copy link
Contributor Author

kartenkarsten commented Dec 27, 2019 •
edited
Loading

Danke für den PR! Aus Interesse: ist das SEM mit dem WR über RS485 verbunden und wird für Laststeuerung eingesetzt?

Also ja die beiden sind über RS485 verbunden. Den SEM nutze ich für die dynamische Abregelung und halt um den Hausverbrauch zu bestimmen.

@andig
Copy link
Contributor

andig commented Dec 30, 2019

@kartenkarsten würdest Du die angefragten Änderungen noch machen?

@kartenkarsten
Copy link
Contributor Author

um die import export Werte zum Laufen zu bekommen benötige ich Unterstüzung. Mit 8e998b6 habe ich es so umgesetzt wie es meiner Meinung nach sein müsste, aber im Webinterface tauchen die neuen Werte nicht auf.
Auch gibt es vermutlich ein Problem mit den Einheiten. Das Meter gibt die Energie in Whstatt kWh zurück. Gibt es dafür eine einfache Lösung? ansonten würde ich den Letzten Commit aus der History nehmen.

@andig
Copy link
Contributor

andig commented Dec 30, 2019

Wh sollten egtl. zu sehen sein, auch wenn falscher Wert. Für Grösse gibts die DividerMap in der Datei weiter unten. Wenn Du gar nichts siehst ist allerdings etwas seltsam. Probier mal bitte, evtl. würde es helfen wenn ich remote zugreifen kann, für bitte per Mail melden.

@kartenkarsten
Copy link
Contributor Author

Hab den Fehler gefunden und die DividerMap hat auch geholfen. Zwar sieht mir der Wert für TotWhExp fehlerhaft aus z.B. 4294888850 . Wenn man die bits mit Vorzeichen betrachtet passt es -78446. So wie ich die SunSpec verstehe sollte die Zahl zwischen 0 und 4294967295 liegen. Es wäre demnach ein Bug im meter.

@andig
Copy link
Contributor

andig commented Dec 31, 2019

Hab den Fehler gefunden und die DividerMap hat auch geholfen. Zwar sieht mir der Wert für TotWhExp fehlerhaft aus z.B. 4294888850 . Wenn man die bits mit Vorzeichen betrachtet passt es -78446. So wie ich die SunSpec verstehe sollte die Zahl zwischen 0 und 4294967295 liegen. Es wäre demnach ein Bug im meter.

TotWhExp ist laut sunspec ein Acc32, also vorzeichenlos 32 Bit. Wenn der bei Dir falsch ist dann ists entweder ein Fehler in der Spec (siehe https://github.com/sunspec/models/blob/master/smdx/smdx_00203.xml#L41) oder ein Fehler in der Implementierung- hatte ich bei Kostal schonmal.

Zur Sicherheit- was sagt inspect zu dem Model 203?

@kartenkarsten
Copy link
Contributor Author

Zur Sicherheit- was sagt inspect zu dem Model 203?

TotWhExp 4294877599.00 acc32
Kostal hat sich da scheinbar nicht an die Definition von acc32 gehalten.

@andig
Copy link
Contributor

andig commented Jan 2, 2020

Kostal hat sich da scheinbar nicht an die Definition von acc32 gehalten.

Yep, das ist ein Bug. Du könntest Dich an den Support wenden- die nehmen sowas zumindest auf, einen Fix habe ich allerdings noch nicht gesehen.

Wir könnten eine kleine "Kostalweiche" einbauen die das abfängt und für den WR eben das letzte Bit ignoriert. Damit das sauber funktioniert bräuchte ich aber zum testen Zugriff auf den WR. Falls Du mir den über SSH einrichten möchtest bitte per mail melden, ansonsten bliebe es bei der fehlerhaften Anzeige.

@andig andig merged commit 8090e70 into volkszaehler:master Jan 2, 2020
@andig andig added this to the 0.8 milestone Jan 2, 2020
@andig andig mentioned this pull request Jan 6, 2020
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@andig andig andig requested changes

Assignees
No one assigned
Labels
enhancement New feature or request
Projects
None yet
Milestone
0.8
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@kartenkarsten @andig