Since a recent change, where the module 47_OBIS.pm has no device type configured and just echoes the processed input, I have a warning in the log during initialization:
PERL WARNING: Use of uninitialized value in numeric gt (>) at /usr/share/fhem/FHEM/47_OBIS.pm line 643.The anonymized raw definition is:
defmod obis OBIS none
attr obis stateFormat {sprintf("%.0f kWh", (ReadingsVal($name,"total_consumption",0) / 1000) )}
setstate obis 23337 kWh
setstate obis 2023-11-29 11:42:12 ManufID XXX
setstate obis 2023-11-29 11:42:12 PublicKey xxxx-xxxx-xxxx-xxxx-xxxx-xxxx-xxxx-xxxx-xxxx-xxxx-xxxx-xxxx-xxxx-xxxx-xxxx-xxxx-xxxx-xxxx-xxxx-xxxx-xxxx-xxxx-xxxx-xxxx-xx
setstate obis 2023-11-29 11:42:12 Serial xx-xx-xx-xx-xx-xx-xx-xx-xx-xx
setstate obis 2023-11-29 11:42:12 power 1234.5
setstate obis 2023-11-29 11:42:12 total_consumption 123456789.5
setstate obis 2023-11-29 11:42:12 total_consumption_Ch1 123456789.5
setstate obis 2023-11-29 11:42:12 total_consumption_Ch2 0
Attached patch file fixes the warning in my installation. However, as I do not have any other SML devices, I cannot test, if the patch has any side effects for other configurations.
I hope, I have understood the code correctly and have proposed an appropriate fix. If there are any remarks or questions, please ask.
Just wanted to ask again, if this small patch could be incorporated, or if there are any questions or if I should change something?
Thanks and best
It seems, that my first approach was wrong and the message reappeared. Now another try, which seems to work so far - at least the message does not appear (again) after a restart. I uploaded a modified patch file.
So far, after some further restarts, the warning is gone. The module is also working as expected. So, I would like to ask again, if this small patch can be integrated? @gvzdus?
thanks and best
Weiß jemand, was mit gvzdus los ist? Hier reagiert er nicht, auf PMs auch nicht.
Hallo gvzdus,
nochmal die Nachfrage, ob der Patch aufgenommen werden könnte, um die bei mir sonst auftretende Warnung loszuwerden?
Vielen Dank und beste Grüße
dennisk
Hallo gvzdus,
erneut die Frage, ob der kleine Patch aufgenommen werden kann. Bis heute habe ich diese Warnung nicht mehr gesehen und keine anderen Auffälligkeiten bemerkt.
Die Aufnahme oder zumindest eine Rückmeldung wären toll.
Vielen Dank und beste Grüße
dennisk
schon einmal mit einer PM versucht ?
Ja, schon diverse Male, mit dem letzten Post grade auch wieder parallel. Ich hoffe natürlich, dass bei gvzdus alles in Ordnung ist. Aber ohne jede Rückmeldung, keine Ahnung.
Hallo gvzdus, alle,
erneut die Frage, ob der kleine Patch aufgenommen werden kann. Bis heute habe ich diese Warnung wie schon gesagt nicht mehr gesehen und auch keine anderen Auffälligkeiten bemerkt.
Die Aufnahme oder zumindest eine Rückmeldung wären wirklich toll!
@irgendjemand Kann jemand gvzdus erreichen oder weiß jemand, was mit ihm los ist?
Vielen Dank und beste Grüße
dennisk
Da ich gvzdus einfach nicht erreichen kann und er sich wohl auch seit April nicht mehr im Forum angemeldet hat: Gibt es eine andere Möglichkeit, dass der Patch eingespielt wird?
Leider hat sich gvzdus bisher immer noch nicht gemeldet bzw. in irgendeiner Form reagiert. Deswegen hier nochmal die Bitte, den kleinen Fix bitte zu übernehmen. Funktioniert bei mir wie gesagt einwandfrei und ohne Fehlermeldungen im Log. Wäre super, vielen Dank!
Hi, ich habe mir den Patch (schon beim ersten Mal) angesehen, aber ich kann einfach die Auswirkung der Übernahme nicht beurteilen.
Ich übernehme ihn jetzt einfach mal, nachdem ich mir das 10 Minuten angesehen habe.
Zitat von: gvzdus am 20 Oktober 2024, 18:48:39Hi, ich habe mir den Patch (schon beim ersten Mal) angesehen, aber ich kann einfach die Auswirkung der Übernahme nicht beurteilen.
Ich übernehme ihn jetzt einfach mal, nachdem ich mir das 10 Minuten angesehen habe.
Vielen Dank für Deine Rückmeldung und fürs Übernehmen! Leider hast Du den Patch aus dem ersten Post übernommen, der nicht ganz richtig war. Der Patch aus dem Post vom 31.12.2023 ist der richtige! Könntest Du den bitte nehmen!?
Ist angepasst!
Hallo gvzdus,
ich habe seid August 2024 auch einen patch fuer 47_OBIS.pm im Forum plaziert, der leider noch unbeantwortet ist (auch per direkter PM).
https://forum.fhem.de/index.php?topic=139059.msg1319101#msg1319101
[patch] Unterstuetzung WMZ von Siemens UH50/Landis+Gyr T550 in 47_OBIS.pm.
Waere schoen wenn er hinein kommen koennte.
VG Christoph