Temola TM 40 Demodulation

[sidey79]

Protocol 120 for Temola TM 40 thermometer added

• Please check if the PR fulfills these requirements

• Tests for the changes have been added / modified (needed for for bug fixes / features)
  RFD-FHEM/SIGNALduino_TOOL@608fd6c
• commandref has been added / updated (needed for bug fixes / features)

• What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)

• Bugfix (please link issue)
• Feature enhancement
• 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)

• What is the new behavior (if this is a feature change)?

• Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?)

• Other information:

New bbq sensor
https://forum.fhem.de/index.php?topic=127938.new;topicseen#new

https://www.temola.net/products/view.php?id=10

[codecov]

Codecov Report

Merging #1106 (5f71353) into master (1522212) will increase coverage by 0.09%.
The diff coverage is 51.35%.

@@            Coverage Diff             @@
##           master    #1106      +/-   ##
==========================================
+ Coverage   65.36%   65.46%   +0.09%     
==========================================
  Files         137      139       +2     
  Lines        9811     9893      +82     
  Branches     1552     1572      +20     
==========================================
+ Hits         6413     6476      +63     
+ Misses       2162     2161       -1     
- Partials     1236     1256      +20     

Flag	Coverage Δ
fhem	57.71% <51.35%> (+0.18%)	⬆️
modules	65.46% <51.35%> (+0.09%)	⬆️
perl	90.33% <ø> (ø)
unittests	65.46% <51.35%> (+0.09%)	⬆️

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

Impacted Files	Coverage Δ
FHEM/lib/SD_ProtocolData.pm	100.00% <ø> (ø)
FHEM/14_SD_WS.pm	65.11% <51.35%> (-0.31%)	⬇️
t/FHEM/14_FLAMINGO/00_load.t
t/FHEM/14_SD_AS/00_load.t
t/FHEM/14_SD_BELL/09_parseDatat.t	80.00% <0.00%> (ø)
t/FHEM/14_FLAMINGO/09_parseDatat.t	80.00% <0.00%> (ø)
t/FHEM/10_SD_GT/00_load.t	100.00% <0.00%> (ø)
t/FHEM/14_BresserTemeo/00_load.t	100.00% <0.00%> (ø)
FHEM/10_SD_Rojaflex.pm	70.73% <0.00%> (+2.03%)	⬆️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

[elektron-bbs]

Mit kleinen Änderungen der Definition könnte das schon passen. Ich hatte folgende erstellt (Protokoll-Nr. 120 und 121 habe ich schon für andere Protokolle in Arbeit):

    "122" =>  ## Grill Thermometer Wireless Meat Thermometer, TM40 Roasting Thermometer with 4 Temperature Sensors
              # https://forum.fhem.de/index.php?topic=127938.msg1224516#msg1224516 2022-06-09 @ Prof. Dr. Peter Henning
              # 
              # 
      {
        name            => 'TM40',
        comment         => 'Roasting Thermometer with 4 Temperature Sensors',
        id              => '122',
        one             => [1,-3],           # 520,-1560
        zero            => [1,-1],           # 520,-520
        start           => [2,-1,2,-1,9,-6], # 1040,-520,1040,-520,4680,-3120
        clockabs        => 520,
        format          => 'twostate',
        preamble        => 'u122#',
        # clientmodule    => 'SD_WS',
        modulematch     => '^u122#',
        length_min      => '104',
        length_max      => '108',
      },

Damit bekomme ich aus den Logs folgende Werte:

alle 22 °C
hex   dez
9263
00DC  220
00DC  220
00DC  220
00DC  220
0000
24
8

83 °C und 22 °C
hex   dez
9263
00DC  220
00DC  220
00DC  220
033E  830
0000
DE
8

82 °C und 22 °C
hex   dez
9263
00DC  220
00DC  220
00DC  220
0334  820
0000
5C
8

[sidey79]

Wir können auf 122 anpassen. Du kannst es auch gerne gleich committen.
Bei der Startsequenzen hatte ich das an Aus Signal nicht aufgenommen, da ich annehme es dient nur zum aufwecken des Empfängers.

[elektron-bbs]

Wir können auf 122 anpassen. Du kannst es auch gerne gleich committen. Bei der Startsequenzen hatte ich das an Aus Signal nicht aufgenommen, da ich annehme es dient nur zum aufwecken des Empfängers.

Ich habe die Änderung vorgenommen. Die Startsequenz habe ich erstmal so belassen.

[elektron-bbs]

@sidey79 sorry, das ich ich hier deinen PR gekapert habe, aber ich war gerade so schön dabei :-)

[sidey79]

@elektron-bbs

Alles in Ordnung.

[sidey79]

@HomeAutoUser

Bestätigt obwohl uns hier noch die Tests und die Doku fehlen und der PR draft hat?

[sidey79]

@elektron-bbs

Wenn ich den Stand hier richtig interpretiere, dann fehlten uns noch Testdaten und eine Bestätigung dass alles richtig funktioniert oder?

[elektron-bbs]

Testdaten habe ich eben hochgeladen und auch die README.md noch ergänzt.
Unklar ist noch Byte 12, wahrscheinlich irgend eine Prüfung. Da bin ich aber nicht dahinter gekommen.
Eine Bestätigung im Forum habe ich angefordert.

[elektron-bbs]

Da sich im Forum nach zwei Wochen noch niemand gemeldet hat, würde ich vorschlagen, wir übernehmen das jetzt so wie es ist.
Die Funktion wurde ja bereits bestätigt.

[sidey79]

Sollten wir so machen ja