Closed sidey79 closed 1 year ago
Merging #1106 (5f71353) into master (1522212) will increase coverage by
0.09%
. The diff coverage is51.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%) |
:arrow_up: |
modules | 65.46% <51.35%> (+0.09%) |
:arrow_up: |
perl | 90.33% <ø> (ø) |
|
unittests | 65.46% <51.35%> (+0.09%) |
:arrow_up: |
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%) |
:arrow_down: |
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%) |
:arrow_up: |
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more
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
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.
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.
@sidey79 sorry, das ich ich hier deinen PR gekapert habe, aber ich war gerade so schön dabei :-)
@elektron-bbs
Alles in Ordnung.
@HomeAutoUser
Bestätigt obwohl uns hier noch die Tests und die Doku fehlen und der PR draft hat?
@elektron-bbs
Wenn ich den Stand hier richtig interpretiere, dann fehlten uns noch Testdaten und eine Bestätigung dass alles richtig funktioniert oder?
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.
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.
Sollten wir so machen ja
Protocol 120 for Temola TM 40 thermometer added
What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
[ ] Bugfix (please link issue)
[x] 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