search

RFD-FHEM / RFFHEM

Counterpart of SIGNALDuino, it's the code for FHEM to work with the data received from the uC
GNU General Public License v3.0
44 stars 33 forks source link

create client list based on whitelistIDs attribute #1205

Closed sidey79 closed 7 months ago

sidey79 commented 7 months ago

Regardles of whielist, we have alway any of yout clinents in our clientlist, which needs to be parsed.

image

no

Improves performance a litte bit

codecov[bot] commented 7 months ago

Codecov Report

Attention: 3 lines in your changes are missing coverage. Please review.

Comparison is base (d3dd2d7) 67.81% compared to head (8471802) 67.98%.

Files Patch % Lines
FHEM/00_SIGNALduino.pm 77.77% 0 Missing and 2 partials :warning:
t/FHEM/00_SIGNALduino/02_SIGNALduino_IdList.t 98.59% 1 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #1205 +/- ## ========================================== + Coverage 67.81% 67.98% +0.17% ========================================== Files 134 138 +4 Lines 9920 10043 +123 Branches 1593 1594 +1 ========================================== + Hits 6727 6828 +101 - Misses 1883 1904 +21 - Partials 1310 1311 +1 ``` | [Flag](https://app.codecov.io/gh/RFD-FHEM/RFFHEM/pull/1205/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=RFD-FHEM) | Coverage Δ | | |---|---|---| | [fhem](https://app.codecov.io/gh/RFD-FHEM/RFFHEM/pull/1205/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=RFD-FHEM) | `57.92% <96.25%> (+0.35%)` | :arrow_up: | | [modules](https://app.codecov.io/gh/RFD-FHEM/RFFHEM/pull/1205/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=RFD-FHEM) | `67.98% <96.25%> (+0.17%)` | :arrow_up: | | [perl](https://app.codecov.io/gh/RFD-FHEM/RFFHEM/pull/1205/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=RFD-FHEM) | `89.80% <ø> (ø)` | | | [unittests](https://app.codecov.io/gh/RFD-FHEM/RFFHEM/pull/1205/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=RFD-FHEM) | `67.98% <96.25%> (+0.17%)` | :arrow_up: | Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=RFD-FHEM#carryforward-flags-in-the-pull-request-comment) to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

sidey79 commented 7 months ago

@HomeAutoUser @elektron-bbs

Also das mit dem Zeilenumbruch, ich habe das eingebaut, aber wenn die Whitelist so lang ist, dann wird da nichts umgebrochen.

Das ist meiner Ansicht nach auch eher ein Thema vom style sheet.

elektron-bbs commented 7 months ago

Ich denke, diese Änderung müssen wir erst noch etwas genauer verfolgen. Bei einem der Neustarts fehlte hier in der Liste der Clients das Modul SD_WS bei 3 SIGNALduinos (allerdings nicht bei allen SIGNALduinos):

   Clients    SD_BELL:SD_WS07:IT:SD_UT:CUL_EM:OREGON:CUL_TX:RFXX10REC:SD_Keeloq:FHT:SOMFY:CUL_WS:Hideki:FLAMINGO:FS10:SD_WS09:CUL_FHTTK:FS20:SD_WS_Maverick:SD_AS:CUL_TCM97001:SD_RSL:Revolt:
   whitelist_IDs 1,2,3,3.1,4,6,7,7.1,7.2,8,9,10,11,12,13,13.1,13.2,14,15,17,17.1,18,24,26,27,32,33,33.1,33.2,33.3,35,37,38,39,41,43,44,44.1,45,46,47,48,50,51,53,54,54.1,55,56,57,58,60,61,62,64,65,66,67,68,70,71,73,74,74.1,76,77,79,80,81,84,85,87,88,91,91.1,94,95,96,97,98,99,105,110,111,113,114,120,121,122,124,124.1,129,130

Das äußerte sich im Log:

2023.12.03 14:51:03 3: sduinoACM: Unknown code W33#01490DC0070, help me!
2023.12.03 14:51:11 3: sduinoESP8266: Unknown code W33#2E419940020, help me!
2023.12.03 14:51:59 3: sduinoUSB0: Unknown code W33#3C43D5C008C, help me!

Nach einem erneuten Neustart von FHEM (jetzt auch mit der letzten Änderung "Added dynamic whitespaces for lf in webview) scheint alles wieder zu laufen.

sidey79 commented 7 months ago

Ok, das ist ein Bug :( An Teil Namensgleichheit habe ich nicht gedacht

SD_WS wird nicht aufgenommen, da SD_WS_Maverick schon aufgenommen ist.

sidey79 commented 7 months ago

@elektron-bbs

So, ich hab das Problem behoben und jetzt 5x den Test laufen lassen, dieser Fehler ist beseitigt.

HomeAutoUser commented 7 months ago

@sidey79, ich habe deinen Branch eingespielt und sehe leidder keinen Zeilenumbruch ;-) ...

grafik

versionmodul | 3.5.6+20230918

Verschiedene FHEM-Restarts habe ich durchgeführt aber leider kam nicht der gewünschte Erfolg. Das Leerzeichen siehst du auch in der Grafik vor CUL_TX.

PS: Vielleicht hängt es damit zusammen? Device global Attribute exclude_from_update | defaultCommon.css ???

sidey79 commented 7 months ago

@HomeAutoUser

Ja das habe ich auch bemerkt. Das liegt daran, dass die Tabelle schon durch WhitelistIDs sehr breit ist.

Das mit dem Leerzeichen ist ja eher so ein : hier könnte ungebrochen werden Ding. Sobald aber eine andere Zelle mehr Breite hat gibt es keinen Grund zu brechen.

Das Phänomen sollte auch ohne die Anpassung schon so sein

HomeAutoUser commented 7 months ago

@sidey79 ich brachte da was durcheinader :-)

Dein PR sollte funktionieren. Ich sehe in der Clientlist nur das benutze Modul der aktivierten ID´s. Einen Schönheitsfehler sehe ich noch, das wäre der letzte Doppelpunkt in der Liste.

Bsp: grafik

sidey79 commented 7 months ago

Ja, der letzte Doppelpunkt. Das ist halt viel einfacher den immer anzuhängen :)

sidey79 commented 7 months ago

@sidey79 ich brachte da was durcheinader :-)

Dein PR sollte funktionieren. Ich sehe in der Clientlist nur das benutze Modul der aktivierten ID´s. Einen Schönheitsfehler sehe ich noch, das wäre der letzte Doppelpunkt in der Liste.

Bsp: grafik

Der letzte Doppelpunkt ist jetzt weg.