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

"Device constant" cannot be changed on HmIP-ESI (10000 imp/kwh -> 1000 imp/kwh) #2937

Closed
oszczech opened this issue Nov 25, 2024 · 8 comments
Labels
🐛 bug-report Something isn't working 🏷️ WebUI This refs the WebUI component

Comments

@oszczech
Copy link

Describe the issue you are experiencing

When opening the device setting page (of HmIP-ESI + "ES-LED") and detecting the sensor a default value of "10000" is set for the "device constant"
This value is not correct as some/my smartmeter only output at a rate of 1000 imp/kwh

When I try to change the value to "1000" and apply, it briefly shows up as "1000.00", but changes quickly back to "10000" after a refresh

could be realted to issue #2712

video:
Screencast_20241125_155923.webm

Describe the behavior you expected

The device constat will be changed to 1000 imp/kwh and show the correct values in homeassistant

Steps to reproduce the issue

  1. go to HmIP-ESI/ES-LED setting
  2. change device constat value to 1000
  3. click apply/ok
  4. reload -> wrong setting

What is the version this bug report is based on?

3.79.6.20241122

Which base platform are you running?

rpi5 (RaspberryPi5, ARM64/aarch64)

Which HomeMatic/homematicIP radio module are you using?

HmIP-RFUSB

Anything in the logs that might be useful for us?

I do not see any related logs in the homematic addon page in homeassistant.

Additional information

No response

@oszczech oszczech added the 🐛 bug-report Something isn't working label Nov 25, 2024
@jens-maus
Copy link
Owner

Bitte mal folgenden Patch lokal anwenden um zu verifizieren das damit das Problem seitigt sein sollte:

--- /www/config/easymodes/etc/uiElements.tcl
+++ /www/config/easymodes/etc/uiElements.tcl
@@ -297,9 +297,12 @@ proc getTextField {param value chn prn {extraparam ""} {superExtra ""}} {
       set maxValue "stringUTF8"
       set maxLength "maxLength=16"
       set sizeTextfield 16
-    } elseif {($param == "METER_CONSTANT_VOLUME") || ($param == "METER_CONSTANT_ENERGY")} {
+    } elseif {($param == "METER_CONSTANT_VOLUME")} {
       set minValue [format {%1.2f} $param_descr(MIN)]
       set maxValue [format {%1.2f} $param_descr(MAX)]
+    } elseif {($param == "METER_CONSTANT_ENERGY")} {
+      set minValue [format {%1.0f} $param_descr(MIN)]
+      set maxValue [format {%1.0f} $param_descr(MAX)]
     }
   }
 

@oszczech
Copy link
Author

Funktioniert jetzt bei mir.

vielen Dank für die sehr schnelle Lösung :)

@jens-maus jens-maus reopened this Nov 25, 2024
@jens-maus
Copy link
Owner

Stopp mal. Das ist hier kein Diskussionsforum wo man den Beitrag einfach schließst wenn man selbst das Problem beseitigt hat. Die Frage war ja eigentlich recht klar: Löst der vorgeschlagene Patch das Problem? Wenn ja muss hier erst das problem auch für den nächsten Release beseitigt werden und DANN schließst man dieses Ticket hier.

Ergo, also bitte beantworten ob du diesen Patch jetzt selbst temporär angewendet hast und wenn ja, dann kann ich das für den nächsten Release auch so einbauen und erst dann wird das permanent beseitigt sein.

@jens-maus jens-maus added the 🏷️ WebUI This refs the WebUI component label Nov 25, 2024
@jens-maus jens-maus added this to the next release milestone Nov 25, 2024
@jens-maus
Copy link
Owner

@Baxxy13 Hast du einen HmIP-ESI und kannst meinen "Fix" von hier vielleicht verifizieren?

@Baxxy13
Copy link
Contributor

Baxxy13 commented Nov 26, 2024

Nein, leider nicht.
Ich nutze inzwischen Shelly Pro 3EM.

@jens-maus
Copy link
Owner

Ok, dann müsstest sich bitte der ursprünglich Autor dieses Reports hier (@oszczech) noch einmal zu Wort melden, ansonsten bleibt es vmtl. so wie es ist und dann wird mit der nächsten offiziellen Version das Problem auch bei ihm wieder auftauchen...

@jens-maus jens-maus added the 🕙 awaiting feedback waiting for feedback from ticket owner label Nov 26, 2024
Copy link
Contributor

@oszczech, the maintainer of this project has requested more detailed information or a specific feedback. Please provide this feedback and report back ASAP or this ticket will be automatically marked as stale after a certain short grace period.

@oszczech
Copy link
Author

Wie gesagt, es funktioniert bei mir mit den vorgeschlagenen Änderungen. Ich habe es manuell im homeassistant addon Container über SSH geändert und konnte nach dem Neustart des Containers den Wert am HmIP-ESI ändern.

Sorry fürs schließen. War nicht absicht, ich nutze github sonst nicht als issue tracker

@jens-maus jens-maus removed the 🕙 awaiting feedback waiting for feedback from ticket owner label Nov 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐛 bug-report Something isn't working 🏷️ WebUI This refs the WebUI component
Projects
None yet
Development

No branches or pull requests

3 participants