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

💡phase_power_current = integer #1533

Closed
mrombouts opened this issue Dec 8, 2021 · 7 comments
Closed

💡phase_power_current = integer #1533

mrombouts opened this issue Dec 8, 2021 · 7 comments

Comments

@mrombouts
Copy link

mrombouts commented Dec 8, 2021

Zie #1394 #1641


Feature

Hallo,

Is er een specifieke reden waarom phase_power_current door dsmr Reader als iteger wordt behandelt?
Het telegram geeft dit nochtans als een floating point waarde:

/FLU5\253769484_A

1-0:31.7.0(001.06*A)
1-0:51.7.0(002.07*A)
1-0:71.7.0(001.47*A)

Hierdoor wordt deze waarde in mqtt en verdere toepassingen (Home Assistant) ook als integer gezien.

Is het mogelijk dit te veranderen naar floating point?

Bedankt,
Michiel

@mrombouts mrombouts added the review Not sure yet whether to implement this label Dec 8, 2021
@dennissiemensma
Copy link
Member

dennissiemensma commented Dec 8, 2021 via email

@mrombouts
Copy link
Author

Hallo Dennis,

Dit hoeft toch niet iedereen te raken? Als je bv kijkt naar het inkomende telegram, en dan afhankelijk van het formaat interger of FP gebruikt?

Alvast bedankt voor je snelle reactie!

@dennissiemensma
Copy link
Member

dennissiemensma commented Dec 8, 2021 via email

@mrombouts
Copy link
Author

ok, dankjewel voor de reactie!

@dennissiemensma dennissiemensma added this to the 6.0 milestone Dec 9, 2021
@dennissiemensma dennissiemensma removed the review Not sure yet whether to implement this label Jan 17, 2022
@eTh0maz
Copy link

eTh0maz commented Mar 2, 2022

Ik stel hetzelfde vast, de waarden na het decimaal punt gaan verloren.
De Fluvius meter is hier ingebruik genomen voorjaar 2020.
image

@dennissiemensma
Copy link
Member

Het valt me nu pas op dat in dit issue hetzelfde speelt als in #1394.

Ik sluit de huidige, want dan pas ik de andere wel even aan.

@dennissiemensma
Copy link
Member

dennissiemensma commented Mar 3, 2022

Zie #1394 #1641

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

No branches or pull requests

3 participants