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

Preserve Limit accuracy #1121

Merged
merged 5 commits into from
Jul 23, 2024
Merged

Preserve Limit accuracy #1121

merged 5 commits into from
Jul 23, 2024

Conversation

schlimmchen
Copy link
Member

  • when parsing a response from the inverter, preserve the accuracy when diving the integer value by 10.
  • when processing web API limits, process them as float.
  • when processing MQTT limits, process them as float.

@schlimmchen
Copy link
Member Author

@spcqike Would you mind testing to set fractional absolute and relative limits using MQTT?

@spcqike
Copy link

spcqike commented Jul 22, 2024

MQTT Teste ich morgen gerne, wenn ich die Zeit finde :) muss ich erst was einrichten.

auf einen ersten schnellen Blick im WebUI: es geht. Ich kann sowohl 16.5 als auch 16,5 % vorgeben und er stellt 330W ein.

Allerdings zeit er weiterhin 17% an :)

image

@schlimmchen
Copy link
Member Author

Allerdings zeit er weiterhin 17% an :)

Ja, da wird gerundet. Ich fänd das okay so. Denkst du das sollte eine Nachkommastelle haben?

@spcqike
Copy link

spcqike commented Jul 22, 2024

Ich fände es schon verständlicher, ja. Irgendwie verwirrt es mich, wenn von 330W bis 340W alles als 17% angezeigt wird. Auch wenn man 16.5% einstellt und 17% bekommt 😅

In der Konsole würde ich zum besseren Verständnis und zum debuggen vielleicht sogar 2 Nachkommastellen nehmen. Bisher sieht es zwar so aus, als gäbe es maximal 0.1% als Genauigkeit (bei mir eben 2W), aber wer weiß es denn. Vielleicht steckt ja doch mehr drin oder kommt bei ner anderen Firmware Version (:

@schlimmchen
Copy link
Member Author

aber wer weiß es denn.

Ne, mit Zehntel-Prozent ist dann Schluss, das siehst du im Code bzw. im Protokoll. Da steht eine Ganzzahl drin, und zwar Promille oder Watt, je nach Typ. Genauer wirds nicht.

@schlimmchen schlimmchen force-pushed the preserve-limit-accuracy branch from 5117233 to cea1dc4 Compare July 22, 2024 18:44
@schlimmchen
Copy link
Member Author

Auf aktuellen development Branch gesetzt und Präzision der Zahlen im Web UI angepasst:

image

@spcqike
Copy link

spcqike commented Jul 23, 2024

habs getestet. sowohl per webui als auch MQTT kann ich alle möglichen Vorgaben setzen.

Auch absolute Werte können als Dezimalzahl übertragen werden. In meinem Fall wird auch dort immer abgerundet. 1000.8W Vorgabe werden so zu 1000W Limit.

und der HMS-2000 bleibt bei seinen 2W / 0.1%. aus 1111W werden 1110W. Aber das wird am WR bzw. seinem großen Bereich liegen.

@AndreasBoehm
Copy link
Member

Habs auch gerade probiert über MQTT, funktioniert wunderbar.

Ich denke wir sollten die hass integration noch anpassen und dort step: 0.1 mitgeben, aktuell defaulted hass zu step: 1.

min: 0
max: 100
step: 1
mode: auto
unit_of_measurement: "%"
icon: mdi:speedometer
friendly_name: Hoymiles HMS-2000-4T Limit NonPersistent Relative

@schlimmchen
Copy link
Member Author

habs getestet. sowohl per webui als auch MQTT kann ich alle möglichen Vorgaben setzen.

@spcqike Super, danke!

Ich denke wir sollten die hass integration noch anpassen und dort step: 0.1 mitgeben, aktuell defaulted hass zu step: 1.

@AndreasBoehm In der Tat! Danke für diesen Hinweis.

the inverter limit can be adjusted in permille, so the step to adjust
the limit through Home Assistent shall be 0.1 percent.
@schlimmchen schlimmchen merged commit 6ee6eaf into development Jul 23, 2024
10 checks passed
@schlimmchen schlimmchen deleted the preserve-limit-accuracy branch July 23, 2024 19:19
Copy link

This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new discussion or issue for related concerns.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Aug 23, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants