ndokter / dsmr_parser

Library to parse Dutch Smart Meter Requirements (DSMR) telegrams.
MIT License
110 stars 64 forks source link

Q3D add CURRENT_ELECTRICITY_DELIVERY #149

Closed Aeroid closed 4 months ago

Aeroid commented 4 months ago

to avoid negative CURRENT_ELECTRICITY_USAGE

dupondje commented 4 months ago

How will it change the behaviour of the CURRENT_ELECTRICITY_USAGE value? :) Looks good otherwise

ndokter commented 4 months ago

Looks good to me as well!

Aeroid commented 4 months ago

My EasyMeters/COM-Boxes "misuse" the "USAGE" with negative values for "DELIVERY". This produces strange results in Home Assitant. Moving negative values over to (positive) "DELIVERY" worked for me, but this is probably a very specific problem of my setup. I assume other EasyMeters will expose it properly as "DELIVERY" already, that's why I suggested this NOT to be filtered out by Q3D Spec. ... so no change to the parser other than handing it thru.

dupondje commented 4 months ago

You know if there is somewhere official documentation where it shows all the obis codes used? Cause for example https://www.easymeter.com/downloads/products/zaehler/Q3D/Easymeter_Q3D_DE_2016-06-15.pdf does not mention the DELIVERY field.

Aeroid commented 4 months ago

No, I don't.

ndokter commented 4 months ago

You know if there is somewhere official documentation where it shows all the obis codes used? Cause for example https://www.easymeter.com/downloads/products/zaehler/Q3D/Easymeter_Q3D_DE_2016-06-15.pdf does not mention the DELIVERY field.

These are all the links i have:

ndokter commented 3 months ago

These changes have been put into v1.4.0