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

Werte vom Datenpunkt todayRainCounter #508

Open
lolmeister77 opened this issue Jan 8, 2024 · 9 comments
Open

Werte vom Datenpunkt todayRainCounter #508

lolmeister77 opened this issue Jan 8, 2024 · 9 comments
Labels
enhancement New feature or request

Comments

@lolmeister77
Copy link

Hier, wie im Forum gewünscht, der Issue:

Danke vorab für die Unterstützung

Describe the bug
Werte vom Datenpunkt todayRainCounter werden beispielsweise so ausgegeben:
0.3000000000001819 oder 0.6000000000003638
In der offiziellen App werden die Werte als 0,3 oder 0,6 angezeigt.

Versions:

  • Adapter version: v1.20.0)
  • JS-Controller version: v5.0.17
  • Node version: v16.20.2
  • Operating system: raspian
@RP70DP
Copy link

RP70DP commented Jan 15, 2024

Kann ich nicht nachvollziehen, daher hier mal ein paar Screenshots:
hmipws

hmipwsidb

hmipwsakf

Wie man aber am Wert vaporAmount sieht, kommen da von der API halt Werte mit vielen Nachkommastellen rein. Es ist nicht Aufgabe des Adapters, die Werte zu verfälschen. Nahezu jedes Widget bietet die Möglichkeit, die Anzahl der angezeigten Nachkommastellen zu konfigurieren. Da man aber sowieso niemals die Datenpunkte der Instanz selbst nutzen sollte, sondern immer einen Alias erstellen sollte, kann man auch die Konvertierungsfunktion des Alias nutzen.

@lolmeister77
Copy link
Author

Danke für die Rückmeldung.
Die Alias und Konvertierungsfunktion kannte ich bisher noch nicht. Nur aus Interesse, warum sollte man immer Alias verwenden?
Ich denke die Konvertierungsfunktion wird mein Problem schon lösen.

@RP70DP
Copy link

RP70DP commented Jan 15, 2024

Stell dir vor, du hast den Datenpunkt von todayRainCounter in einem Skript, einer Visualisierung und sonst wo vielleicht auch gleich mehrfach genutzt. Jetzt geht dein Wettersensor kaputt, der neue Wettersensor hat aber eine andere SGTIN. Viel Spaß jetzt beim Ändern des Datenpunkts an allen diesen Stellen. Mit einem Alias änderst du nur das Ziel in deinem Alias. Fertig!

@lolmeister77
Copy link
Author

OK, das macht dann natürlich Sinn. Vielen Dank

@mcm1957
Copy link
Member

mcm1957 commented Jan 15, 2024

Ich kann hier auch keinen Fehler erkennen. Wie RP70DP richtig angemerkt hat kommen werden vom API (anscheinend) Daten mit mehr Nachkommastellen geliefert als in der App angezeigt. Der Adapter sollte aber keine Datenmanipulation vornehmen.

Auf Grund der sehr kleinen Abweichung nach vielen "0"en sollte trotzdem im Code geprüft werden, ob hier ev. Berechnungen im Adapter mitspielen oder ob diese Werte wirklich vom Api geliefert werden.

@mcm1957 mcm1957 closed this as completed Jan 15, 2024
@mcm1957 mcm1957 added the wontfix This will not be worked on label Jan 15, 2024
@mcm1957 mcm1957 reopened this Jan 15, 2024
@mcm1957 mcm1957 removed the wontfix This will not be worked on label Jan 15, 2024
@RP70DP
Copy link

RP70DP commented Jan 15, 2024

Von der REST-API:

3014FXXX HmIP-SWO-PR Wettersensor lowBat(False) unreach(False) rssiDeviceValue(128) rssiPeerValue(None) configPending(False) dutyCycle(False) temperatureOutOfRange(False) actualTemperature(0.7) humidity(85) vaporAmount(4.32401565560426) illumination(0.0) illuminationThresholdSunshine(3500.0) raining(False) storm(False) sunshine(False) todayRainCounter(0.6000000000003638) todaySunshineDuration(83) totalRainCounter(4656.1) totalSunshineDuration(808302) weathervaneAlignmentNeeded(False) windDirection(230.0) windDirectionVariation(33.75) windSpeed(11.3) windValueType(MAX_VALUE) yesterdayRainCounter(1.800000000000182) yesterdaySunshineDuration(49)

@mcm1957
Copy link
Member

mcm1957 commented Dec 22, 2024

OK, offensichtlich liefert das Rest Api die Werte mit vielen Nachkomastellen.
Deher kein BUG

@mcm1957 mcm1957 closed this as not planned Won't fix, can't repro, duplicate, stale Dec 22, 2024
@Apollon77 Apollon77 reopened this Jan 2, 2025
@Apollon77
Copy link
Member

We could still round it :-) more as 2 numbers after digit makes no sense

@mcm1957 mcm1957 added the enhancement New feature or request label Jan 2, 2025
@mcm1957
Copy link
Member

mcm1957 commented Jan 2, 2025

Normally we provide data as it is delivered by api.
But OK - lets keep open as enhancemengt until someone has time to implement and test.

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

No branches or pull requests

4 participants