search

volkszaehler / mbmd

ModBus Measurement Daemon - simple reading of data from ModBus meters and grid inverters
BSD 3-Clause "New" or "Revised" License
224 stars 81 forks source link

Fix Support for CG EM24 Ethernet Version #317

Open hrpv opened 1 year ago

hrpv commented 1 year ago

The EM24 Ethernet Version has a different register mapping than the EM24 RS485 Version I made a local patch for me to get it to work, but a clean solution would be the definition of a new Counter Type e.g. CGEM24ETH

cg.txt Use type CGEX3X0 for EM24 Ethernet Version, no patch required. Update of documentation would be sufficient

andig commented 1 year ago

Es gibt bereits EM24 (cgem24) und EM/ET 330/340 (cgex3x0). Ist das jetzt nochmal eine weitere Differenzierung? Gibt es eine offizielle Doku dazu?

hrpv commented 1 year ago 
                    Es sieht so aus als ob CGEX3X0 auch für EM24 Ethernet funktioniert, gerade getestet :-)

die meisten Register scheinen identisch zu sein

https://gavazzi.se/app/uploads/2020/11/em24_is_cp.pdf (https://gavazzi.se/app/uploads/2020/11/em24_is_cp.pdf" id="isPasted" rel="noopener noreferrer" target="_blank) https://gavazzi.se/app/uploads/2020/11/em330_em340_et330_et340_cp.pdf (https://gavazzi.se/app/uploads/2020/11/em330_em340_et330_et340_cp.pdf" id="isPasted" rel="noopener noreferrer" target="_blank) also keine Code Änderung, nur Doku Update mbmd_run.md

-----ursprüngliche Nachricht----- Von: @. @." target="_blank) Gesendet: 09.05.2023 17:16 Uhr An: @. @." target="_blank) Kopie: @. @." target="_blank), @. @." target="_blank) Betreff: Re: [volkszaehler/mbmd] Fix Support for CG EM24 Ethernet Version (Issue #317)

Es gibt bereits EM24 (cgem24) und EM/ET 330/340 (cgex3x0). Ist das jetzt nochmal eine weitere Differenzierung?— Reply to this email directly, view it on GitHub (https://github.com/volkszaehler/mbmd/issues/317#issuecomment-1540359277" target="_blank), or unsubscribe (https://github.com/notifications/unsubscribe-auth/AWWMTZU2H4OIYPMAIV46A4TXFJNVFANCNFSM6AAAAAAX3HCJOY" target="_blank). You are receiving this because you authored the thread.Message ID: @.***> -----ursprüngliche Nachricht Ende-----

frosty-geek commented 1 year ago

Scheinbar ist EM340 auch nur teilweise compatible, ich hab hier die offizielle Doku zur EM24 Ethernet variante gefunden: https://www.enika.eu/data/files/produkty/energy%20m/CP/em24%20ethernet%20cp.pdf

hrpv commented 1 year ago

Scheinbar ist EM340 auch nur teilweise compatible, ich hab hier die offizielle Doku zur EM24 Ethernet variante gefunden: https://www.enika.eu/data/files/produkty/energy%20m/CP/em24%20ethernet%20cp.pdf

Mit CGEX3X0 als Typ eingestellt funktioniert der EM24 Ethernet bei mir ohne Probleme, hast du das mal probiert? Der von mir vorgeschlagene Patch ist unnötig, update der Doku sollte reichen

frosty-geek commented 1 year ago

Ja habe ich, die Basis Werte sind da, aber es fehlen diverse Werte die man sonst via Modbus TCP bekommen kann.

Der Tagesverbrauch z.b. kann aktuell nicht ordentlich bestimmt werden. Ich benutze Homeassistent mit nativen Modbus TCP Support parallel und hier fehlen die Werte einfach (siehe communication protocol aus der verlinkten Doku).

hrpv commented 1 year ago

Ja habe ich, die Basis Werte sind da, aber es fehlen diverse Werte die man sonst via Modbus TCP bekommen kann.

Der Tagesverbrauch z.b. kann aktuell nicht ordentlich bestimmt werden. Ich benutze Homeassistent mit nativen Modbus TCP Support parallel und hier fehlen die Werte einfach (siehe communication protocol aus der verlinkten Doku).

O.k. dann musst du wohl selbst Hand anlegen, und den vorgeschlagenen Patch passend um weitere Register erweitern

andig commented 1 year ago

Der Tagesverbrauch z.b. kann aktuell nicht ordentlich bestimmt werden

Den haben wir auch nirgendwo abgefragt. Maximal verfügbar sein könnten die Werte aus dieser Liste: https://github.com/volkszaehler/mbmd/blob/master/meters/measurements.go#L25