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.

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

Q3D add CURRENT_ELECTRICITY_DELIVERY #149

Merged
merged 1 commit into from
Feb 19, 2024
Merged

Conversation

Aeroid
Copy link
Contributor

@Aeroid Aeroid commented Feb 9, 2024

to avoid negative CURRENT_ELECTRICITY_USAGE

to avoid negative CURRENT_ELECTRICITY_USAGE
@dupondje
Copy link
Collaborator

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

@ndokter
Copy link
Owner

ndokter commented Feb 15, 2024

Looks good to me as well!

@Aeroid
Copy link
Contributor Author

Aeroid commented Feb 15, 2024

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
Copy link
Collaborator

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
Copy link
Contributor Author

Aeroid commented Feb 15, 2024

No, I don't.

@ndokter
Copy link
Owner

ndokter commented Feb 15, 2024

@dupondje dupondje merged commit fcbcbb3 into ndokter:master Feb 19, 2024
6 checks passed
@ndokter
Copy link
Owner

ndokter commented Mar 12, 2024

These changes have been put into v1.4.0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants