HomeAutoUser commented 2 years ago

Please check if the PR fulfills these requirements
[x] Tests for the changes have been added / modified (needed for for bug fixes / features)
[ ] commandref has been added / updated (needed for bug fixes / features)

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
[x] Bugfix (please link issue)
[ ] Feature enhancement
[x] Documentation update
[ ] Unittest enhancement
[ ] other
What is the current behavior? (You can also link to an open issue here, if this describes the current behavior)
- any wrong values
What is the new behavior (if this is a feature change)?
- takeover of https://github.com/RFD-FHEM/RFFHEM/commit/6f7f3df58821ebd5595b5712472c596348cfb6af
- die Zuordnung der Windrichtung hatte nicht gepasst https://forum.fhem.de/index.php/topic,126712.0.html
- $name durch $ioname ersetzt
- Das Attribut "windDirCorr" hatte keine Funktion
Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?)
Other information:
- until this fix is adopted, PR https://github.com/RFD-FHEM/RFFHEM/pull/1086 will remain untouched

codecov[bot] commented 2 years ago

Codecov Report

Merging #1099 (38f1f37) into master (4806d2e) will increase coverage by 0.24%. The diff coverage is 67.56%.

@@            Coverage Diff             @@
##           master    #1099      +/-   ##
==========================================
+ Coverage   64.50%   64.75%   +0.24%     
==========================================
  Files         136      133       -3     
  Lines        9652     9646       -6     
  Branches     1528     1530       +2     
==========================================
+ Hits         6226     6246      +20     
+ Misses       2237     2207      -30     
- Partials     1189     1193       +4

Flag	Coverage Δ
fhem	`56.65% <67.56%> (+0.29%)`	:arrow_up:
modules	`64.75% <67.56%> (+0.24%)`	:arrow_up:
perl	`90.17% <ø> (-0.21%)`	:arrow_down:
unittests	`64.75% <67.56%> (+0.24%)`	:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files	Coverage Δ
FHEM/14_Hideki.pm	`65.02% <67.56%> (+14.08%)`	:arrow_up:
FHEM/lib/SD_Protocols.pm	`79.00% <0.00%> (-0.11%)`	:arrow_down:
t/FHEM/14_SD_BELL/00_load.t
t/FHEM/14_FLAMINGO/00_load.t
t/FHEM/14_SD_WS09/00_load.t

:mega: Codecov can now indicate which changes are the most critical in Pull Requests. Learn more

sidey79 commented 2 years ago

@HomeAutoUser

Für den Bug haben wir noch keine Tests, also insbesondere einen Test der das Attribut windDirCorr mit abdeckt. Das ließe sich ja in den Testdaten setzen und die Readings verifizieren. Wie siehst Du das?

HomeAutoUser commented 2 years ago

@HomeAutoUser

Für den Bug haben wir noch keine Tests, also insbesondere einen Test der das Attribut windDirCorr mit abdeckt. Das ließe sich ja in den Testdaten setzen und die Readings verifizieren. Wie siehst Du das?

Hallo @sidey79 da müssen wir mal schauen ob wir RAW Nachrichten finden von dem Typ Sensor wo das Attribut windDirCorr zum tragen kommt. Danach können wir diese in die Testdaten einpflegen.

sidey79 commented 2 years ago

Hallo @sidey79 da müssen wir mal schauen ob wir RAW Nachrichten finden von dem Typ Sensor wo das Attribut windDirCorr zum tragen kommt. Danach können wir diese in die Testdaten einpflegen.

DMSGs würden meiner Ansicht nach ausreichen. Das habe ich verlinkten Thread gefunden:

windDirection 10
signalESP_DMSG
P12#7585B2C471BF71BFFDF0029C605C03
P12#7585B284FFBEFFBE0F000260DE7901
P12#7585B2841CBE1CBE1F30FC9CFC3E01

Windfahne mit der Spitze zeigt im Süden also zur Haupteinheit und die Flosse nach Norden und zeigt von der Haupteinheit weg.

Das Display der Wetterstation zeigt S und 180 GRAD
windDirection 12
signalESP_DMSG
P12#7585B244FDBEFDBEF000FE403D6C01
P12#7585B244F8BEF8BE0FE00140DD2D01
P12#7585B2C418BE18BE1EE0FD40B05803

Windfahne mit der Spitze nach Westen und die Flosse zeigt nach Osten

Das Display der Wetterstation zeigt W und 270 GRAD

windDirection 6
signalESP_DMSG
P12#7585B2840EBE0EBEF1F002DC6C1D01
P12#7585B284F3BEF3BEF2D0FC204D3601
P12#7585B2C40DBE0DBE0DE0FC20C20003

HomeAutoUser commented 2 years ago

Hallo @sidey79 da müssen wir mal schauen ob wir RAW Nachrichten finden von dem Typ Sensor wo das Attribut windDirCorr zum tragen kommt. Danach können wir diese in die Testdaten einpflegen.

DMSGs würden meiner Ansicht nach ausreichen. Das habe ich verlinkten Thread gefunden:

windDirection 10
signalESP_DMSG
P12#7585B2C471BF71BFFDF0029C605C03
P12#7585B284FFBEFFBE0F000260DE7901
P12#7585B2841CBE1CBE1F30FC9CFC3E01

Windfahne mit der Spitze zeigt im Süden also zur Haupteinheit und die Flosse nach Norden und zeigt von der Haupteinheit weg.

Das Display der Wetterstation zeigt S und 180 GRAD
windDirection 12
signalESP_DMSG
P12#7585B244FDBEFDBEF000FE403D6C01
P12#7585B244F8BEF8BE0FE00140DD2D01
P12#7585B2C418BE18BE1EE0FD40B05803

Windfahne mit der Spitze nach Westen und die Flosse zeigt nach Osten

Das Display der Wetterstation zeigt W und 270 GRAD

windDirection 6
signalESP_DMSG
P12#7585B2840EBE0EBEF1F002DC6C1D01
P12#7585B284F3BEF3BEF2D0FC204D3601
P12#7585B2C40DBE0DBE0DE0FC20C20003

Hi, wink, die Sommerkühle mal nutzen :-D

Möchtest du die Daten im Tool -> JSON oder als separaten Test haben?

sidey79 commented 2 years ago

Ich denke die Testdaten können in die json Datei vom Tool :)

HomeAutoUser commented 2 years ago

Bitte mal drüber schauen ob das richtig ist? Ich habe mich mal orientiert an der DMSG und was auf dem Display zu sehen war. Das selbe sollte auch im Reading wiedergegeben sein.

--- altes module -----

{"name":"unknown", "id":"12", "module":"Hideki", "data": [
    {
      "dmsg":"P12#7585B2C471BF71BFFDF0029C605C03",  "comment":"no RAWMSG / only DMSG / soll werden E  und 90 Grad, https://forum.fhem.de/index.php/topic,126712.msg1213152.html#msg1213152", "user":"brezel",
      "internals": {"DEF":"Hideki_12_4", "NAME":"Hideki_12_4"},
      "readings": {"state":"T: 19.3 Ws: 7 Wg: 61 Wd: SW", "battery":"ok", "batteryState":"ok", "channel":"4", "package_number":"1", "temperature":"19.3", "windChill":"19.3", "windDirection":"10", "windDirectionDegree":"225", "windDirectionText":"SW", "windGust":"61", "windSpeed":"7"},
      "minProtocolVersion":"unknown", "revision_entry":"2022-06-09 19:59:26",
      "revision_modul":"unknown"
    }
  ]
},

state                 T: 19.3 Ws: 7 Wg: 61 Wd: SW   2022-06-09 19:47:45
battery               ok                            2022-06-09 19:47:45
batteryState          ok                            2022-06-09 19:47:45
channel               4                             2022-06-09 19:47:45
package_number        1                             2022-06-09 19:47:45
temperature           19.3                          2022-06-09 19:47:45
windChill             19.3                          2022-06-09 19:47:45
windDirection         10                            2022-06-09 19:47:45
windDirectionDegree   225                           2022-06-09 19:47:45
windDirectionText     SW                            2022-06-09 19:47:45
windGust              61                            2022-06-09 19:47:45
windSpeed             7                             2022-06-09 19:47:45

--- neues module -----

{"name":"unknown", "id":"12", "module":"Hideki", "data": [
    {
      "dmsg":"P12#7585B2C471BF71BFFDF0029C605C03",  "comment":"no RAWMSG / only DMSG / E  und 90 Grad, https://forum.fhem.de/index.php/topic,126712.msg1213152.html#msg1213152", "user":"brezel",
      "internals": {"DEF":"Hideki_12_4", "NAME":"Hideki_12_4"},
      "readings": {"state":"T: 19.3 Ws: 7 Wg: 61 Wd: SW", "battery":"ok", "batteryState":"ok", "channel":"4", "package_number":"1", "temperature":"19.3", "windChill":"19.3", "windDirection":"4", "windDirectionDegree":"90", "windDirectionText":"E", "windGust":"61", "windSpeed":"7"},
      "minProtocolVersion":"unknown", "revision_entry":"2022-06-09 19:59:26",
      "revision_modul":"unknown"
    }
  ]
},

state                 T: 19.3 Ws: 7 Wg: 61 Wd: E    2022-06-09 20:07:42
battery               ok                            2022-06-09 20:07:42
batteryState          ok                            2022-06-09 20:07:42
channel               4                             2022-06-09 20:07:42
package_number        1                             2022-06-09 20:07:42
temperature           19.3                          2022-06-09 20:07:42
windChill             19.3                          2022-06-09 20:07:42
windDirection         4                             2022-06-09 20:07:42
windDirectionDegree   90                            2022-06-09 20:07:42
windDirectionText     E                             2022-06-09 20:07:42
windGust              61                            2022-06-09 20:07:42
windSpeed             7                             2022-06-09 20:07:42

sidey79 commented 2 years ago

Das sieht doch gut aus, nur was mich ein bisschen irritiert ist der name":"unknown" haben wir da nicht einen Namen von dem Teil?

HomeAutoUser commented 2 years ago

@sidey79 ich habe mal 2 DMSG´s hinzugefügt https://github.com/RFD-FHEM/SIGNALduino_TOOL/commit/f6d215419a9bc20443ad41e8f21e1efb2042982e Mal schauen ob diese Daten nun noch korrekt hier durchlaufen.

PS: Ich fand einen Namen der Wetterstation.

sidey79 commented 2 years ago

Das schaut doch schon gut aus. Jetzt fehlt uns dich nur noch ein Datensatz in dem das Attribut windSpeedCorr gesetzt ist, denn das war doch hier unter anderem die Ergänzung.

Das Attribut kann einfach in einem weiteren Testdatensatz hinterlegt werden, dann wird ja mit dem hinterlegten Wert multipliziert .

sidey79 commented 2 years ago

@HomeAutoUser

Ich habe die Testdaten ergänzt und die Ausgabe auf Nachkommastellen angepasst auch wenn kein Attribut WindDirCorr gesetzt ist. https://github.com/RFD-FHEM/SIGNALduino_TOOL/commit/8b05b90a317f2cfe2c479c2545714e98cb94ab2d

HomeAutoUser commented 2 years ago

@HomeAutoUser

Ich habe die Testdaten ergänzt und die Ausgabe auf Nachkommastellen angepasst auch wenn kein Attribut WindDirCorr gesetzt ist. RFD-FHEM/SIGNALduino_TOOL@8b05b90

Ich habe es gesehen. Daumen hoch

Als einziges Schönheitskorrektur, habe ich aus 2 Datensätzen einen gemacht wie wir es bisher immer pro Sensor hatten, https://github.com/RFD-FHEM/SIGNALduino_TOOL/commit/cf6dcc3ae1e6b31d91fcf6288c92f147bc46dd51

sidey79 commented 2 years ago

Ich habe es gesehen. Daumen hoch

Als einziges Schönheitskorrektur, habe ich aus 2 Datensätzen einen gemacht wie wir es bisher immer pro Sensor hatten, RFD-FHEM/SIGNALduino_TOOL@cf6dcc3

Ja, so geht das auch. Super :)

HomeAutoUser commented 2 years ago

Du kennst Funktionen hier ;-) Ich habe nun die "Commit suggestion" bearbeitet und angenommen.

sidey79 commented 2 years ago

Du kennst Funktionen hier ;-) Ich habe nun die "Commit suggestion" bearbeitet und angenommen.

Gibt es noch nicht so furchtbar lange, aber die Idee finde ich eigentlich gut.

sidey79 commented 2 years ago

@HomeAutoUser Ich musste einfach noch die commandref ergänzen. :) Meinetwegen könnten wir auch mergen. Was meinst Du?

HomeAutoUser commented 2 years ago

Kurz vor Schluss heute ;-) Go von meiner Seite.

RFD-FHEM / RFFHEM

14_Hideki.pm - fix #1099

Codecov Report