14_Hideki.pm - fix #1099

HomeAutoUser · 2022-04-24T12:39:42Z

Please check if the PR fulfills these requirements

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, ...)

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 6f7f3df
- 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 14_Hideki.pm - perlcritic #1086 will remain untouched

codecov · 2022-04-24T12:41:19Z

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%)`	⬆️
modules	`64.75% <67.56%> (+0.24%)`	⬆️
perl	`90.17% <ø> (-0.21%)`	⬇️
unittests	`64.75% <67.56%> (+0.24%)`	⬆️

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%)`	⬆️
FHEM/lib/SD_Protocols.pm	`79.00% <0.00%> (-0.11%)`	⬇️
t/FHEM/14_SD_BELL/00_load.t
t/FHEM/14_FLAMINGO/00_load.t
t/FHEM/14_SD_WS09/00_load.t

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

sidey79 · 2022-04-26T06:23:22Z

@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 · 2022-04-26T07:47:11Z

@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 · 2022-04-26T19:30:22Z

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

commit 4806d2e Author: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Date: Tue Jun 7 22:21:10 2022 +0200 Bump fhem/fhem-controls-actions from 2.0.3 to 2.1.0 (#1105) Bumps [fhem/fhem-controls-actions](https://github.com/fhem/fhem-controls-actions) from 2.0.3 to 2.1.0. - [Release notes](https://github.com/fhem/fhem-controls-actions/releases) - [Commits](fhem/fhem-controls-actions@v2.0.3...v2.1.0) --- updated-dependencies: - dependency-name: fhem/fhem-controls-actions dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> commit 6bb37f9 Author: sidey79 <7968127+sidey79@users.noreply.github.com> Date: Tue May 31 17:12:21 2022 +0200 Basic funkbus support (#1102) * SD_ProtocolData.pm added funkbus protocol 119 from forum https://forum.fhem.de/index.php/topic,19065.msg1217972.html#msg1217972 * SD_Protocols.pm - sub mcBit2Funkbus added - sub mc2dmc added - Tests added

HomeAutoUser · 2022-06-08T11:09:52Z

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 · 2022-06-08T16:00:03Z

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

HomeAutoUser · 2022-06-09T18:27:02Z

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 · 2022-06-09T19:48:23Z

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 · 2022-06-09T21:33:41Z

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

PS: Ich fand einen Namen der Wetterstation.

sidey79 · 2022-06-10T05:05:55Z

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 .

FHEM/14_Hideki.pm

output wind values always as floatingpoint

sidey79 · 2022-06-12T22:07:43Z

@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

HomeAutoUser · 2022-06-13T07:41:08Z

@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,
RFD-FHEM/SIGNALduino_TOOL@cf6dcc3

sidey79 · 2022-06-13T20:03:26Z

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 :)

changed name attr event-min-intervall Co-authored-by: sidey79 <7968127+sidey79@users.noreply.github.com>

changed name Log3 line Hideki_Parse Co-authored-by: sidey79 <7968127+sidey79@users.noreply.github.com>

HomeAutoUser · 2022-06-13T20:37:17Z

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

sidey79 · 2022-06-13T21:03:28Z

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.

commandref bezüglich WindDirCorr und WindSpeedCorr erweitert.

sidey79 · 2022-06-13T21:18:02Z

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

HomeAutoUser · 2022-06-13T21:43:37Z

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

HomeAutoUser and others added 2 commits April 24, 2022 14:27

Update 14_Hideki.pm

e7d4786

Automatic updated controls and CHANGED

a2ccd58

HomeAutoUser added bug Hideki labels Apr 24, 2022

Update Versiondate

aab4a52

HomeAutoUser and others added 4 commits May 30, 2022 09:46

Merge branch 'master' into master_Hideki_wind

f922fa6

Update Versiondate

192fe7b

Update Versiondate

06239aa

HomeAutoUser and others added 2 commits June 9, 2022 20:31

Merge branch 'master' into master_Hideki_wind

07b9012

Update Versiondate

5cb35bc

sidey79 reviewed Jun 12, 2022

View reviewed changes

FHEM/14_Hideki.pm Outdated Show resolved Hide resolved

sidey79 reviewed Jun 12, 2022

View reviewed changes

FHEM/14_Hideki.pm Outdated Show resolved Hide resolved

sidey79 and others added 2 commits June 12, 2022 23:52

14_Hideki.pm

f677c4c

output wind values always as floatingpoint

Update Versiondate

54d9c63

HomeAutoUser and others added 2 commits June 13, 2022 22:34

Update FHEM/14_Hideki.pm

a3013d6

changed name attr event-min-intervall Co-authored-by: sidey79 <7968127+sidey79@users.noreply.github.com>

Update Versiondate

fe67340

Update FHEM/14_Hideki.pm

8057871

changed name Log3 line Hideki_Parse Co-authored-by: sidey79 <7968127+sidey79@users.noreply.github.com>

Update Versiondate

2e64551

14_Hideki.pm

0cf7a01

commandref bezüglich WindDirCorr und WindSpeedCorr erweitert.

Update Versiondate

38f1f37

sidey79 approved these changes Jun 14, 2022

View reviewed changes

HomeAutoUser merged commit 43acfd5 into master Jun 14, 2022

HomeAutoUser mentioned this pull request Jun 14, 2022

14_Hideki.pm: Beim Hideki Windsensor DV928 passt die Zuordnung der Windrichtung nicht #1087

Closed

HomeAutoUser deleted the master_Hideki_wind branch June 14, 2022 06:13

HomeAutoUser restored the master_Hideki_wind branch June 14, 2022 17:48

HomeAutoUser deleted the master_Hideki_wind branch June 14, 2022 17:49

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

14_Hideki.pm - fix #1099

14_Hideki.pm - fix #1099

HomeAutoUser commented Apr 24, 2022 •

edited by sidey79

codecov bot commented Apr 24, 2022 •

edited

sidey79 commented Apr 26, 2022

HomeAutoUser commented Apr 26, 2022

sidey79 commented Apr 26, 2022 •

edited

HomeAutoUser commented Jun 8, 2022

sidey79 commented Jun 8, 2022

HomeAutoUser commented Jun 9, 2022

sidey79 commented Jun 9, 2022

HomeAutoUser commented Jun 9, 2022

sidey79 commented Jun 10, 2022

sidey79 commented Jun 12, 2022

HomeAutoUser commented Jun 13, 2022

sidey79 commented Jun 13, 2022

HomeAutoUser commented Jun 13, 2022

sidey79 commented Jun 13, 2022

sidey79 commented Jun 13, 2022

HomeAutoUser commented Jun 13, 2022

14_Hideki.pm - fix #1099

14_Hideki.pm - fix #1099

Conversation

HomeAutoUser commented Apr 24, 2022 • edited by sidey79

codecov bot commented Apr 24, 2022 • edited

Codecov Report

sidey79 commented Apr 26, 2022

HomeAutoUser commented Apr 26, 2022

sidey79 commented Apr 26, 2022 • edited

HomeAutoUser commented Jun 8, 2022

sidey79 commented Jun 8, 2022

HomeAutoUser commented Jun 9, 2022

sidey79 commented Jun 9, 2022

HomeAutoUser commented Jun 9, 2022

sidey79 commented Jun 10, 2022

sidey79 commented Jun 12, 2022

HomeAutoUser commented Jun 13, 2022

sidey79 commented Jun 13, 2022

HomeAutoUser commented Jun 13, 2022

sidey79 commented Jun 13, 2022

sidey79 commented Jun 13, 2022

HomeAutoUser commented Jun 13, 2022

HomeAutoUser commented Apr 24, 2022 •

edited by sidey79

codecov bot commented Apr 24, 2022 •

edited

sidey79 commented Apr 26, 2022 •

edited