-
Notifications
You must be signed in to change notification settings - Fork 95
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
Comments
Bedankt voor je melding. Je bent de eerste waarbij ik dit zie of het iemand opvalt. Voor de NL varianten zijn het sowieso integers. De testdata voor Fluvius (die ik ooit van iemand gehad heb) gebruikt ook integers: https://github.com/dsmrreader/dsmr-reader/blob/c037848e0f96028fb500415b9289df40f81bc14f/dsmr_datalogger/tests/datalogger/test_fluvius_polyphase.py#L41. Dus ik ben benieuwd of dit een nieuwe variant is. In principe kan ik het datatype wel een keer aanpassen, maar dat is zeker niet triviaal omdat het alle andere gebruikers dan ook zou raken.
|
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! |
Het raakt daarmee de API, mqtt, influxdb allemaal omdat de serializers kijken naar het type en niet de waarde. En dus exporteren er dan andere types. Binnen dsmrreader zelf raakt het inderdaad relatief weinig.
|
ok, dankjewel voor de reactie! |
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. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Zie
#1394#1641Feature
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:
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
The text was updated successfully, but these errors were encountered: