Support ecowitt wh40 rain gauge #1164

sidey79 · 2023-04-02T22:23:29Z

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)

WH40 rain gauge sensor is not decoded

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

WH40 rain gauge sensor is decoded

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

no

Other information:

Test this PR:
update all https://raw.githubusercontent.com/RFD-FHEM/RFFHEM/master-ecowittwh40/controls_signalduino.txt

#1162

- Added 124 -> Ecowitt WH 31 14_SD_WS.pm - Added 124 -> Ecowitt WH 31 - increased threshhold wait timer for second UNDEFINED event : protocol 107 & 125 - Added crc & checksum check - updated battery - updated temp calc testData.json - Added tests for ecowitt WH31

- fixed logmessage typos - added some protocol spec - Added dp50 to sensortype - added _TH to model

codecov · 2023-04-02T22:27:17Z

Codecov Report

Merging #1164 (66837d3) into master (08f6446) will increase coverage by 0.00%.
The diff coverage is 70.37%.

@@           Coverage Diff           @@
##           master    #1164   +/-   ##
=======================================
  Coverage   67.71%   67.71%           
=======================================
  Files         133      137    +4     
  Lines        9827     9885   +58     
  Branches     1572     1576    +4     
=======================================
+ Hits         6654     6694   +40     
- Misses       1876     1890   +14     
- Partials     1297     1301    +4

Flag	Coverage Δ
fhem	`57.45% <70.37%> (+0.07%)`	⬆️
modules	`67.71% <70.37%> (+<0.01%)`	⬆️
perl	`90.33% <ø> (+0.16%)`	⬆️
unittests	`67.71% <70.37%> (+<0.01%)`	⬆️

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	`69.14% <69.23%> (+<0.01%)`	⬆️
FHEM/00_SIGNALduino.pm	`64.25% <100.00%> (ø)`

... and 5 files with indirect coverage changes

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

added rfomode to commandref

- Updated length min, max - Added new RF Mode - Updated Testdata - Updated version

FHEM/14_SD_WS.pm

FHEM/lib/SD_ProtocolData.pm

sidey79 · 2023-04-05T19:02:08Z

@elektron-bbs

Ich habe da aber noch eine Frage zu den Rain Readings. Da gibt es ja durchaus mehrere.
So eindeutig habe ich nichts gefunden, wie welches Reading zu verwenden ist.
Bei diesem Sensor ist es so, dass er einen counter Verwendet. Der wird mit jeder Wippe um eins erhöht.

Welches Reading soll ich da nehmen und wie rechnen wir dann die anderen readings aus.
Überlauf vom Zähler und Batteriewechsel wären ja auch noch so zwei Themen die ich gerne betrachten würde.

elektron-bbs · 2023-04-05T19:49:08Z

Ich habe da aber noch eine Frage zu den Rain Readings. Da gibt es ja durchaus mehrere. So eindeutig habe ich nichts gefunden, wie welches Reading zu verwenden ist. Bei diesem Sensor ist es so, dass er einen counter Verwendet. Der wird mit jeder Wippe um eins erhöht.

So ist es wohl bei den meisten Regensensoren.
Ich habe bisher nur das Reading "rain" verwendet, das schon umgerechnet die Regenmenge in l/mm² ausgibt.
"rawRainCounter" sollte dann wohl direkt der Zähler ohne Umrechnung sein - interessiert wahrscheinlich niemanden.
"rain_total" stellt eigentlich das selbe dar wie "rain" - wird nur bei Protokoll 54 verwendet und wurde wohl mal von einem anderen Entwickler verwendet.

Welches Reading soll ich da nehmen und wie rechnen wir dann die anderen readings aus. Überlauf vom Zähler und Batteriewechsel wären ja auch noch so zwei Themen die ich gerne betrachten würde.

Den Überlauf des Zählers bzw. Start bei 0 wenn Sensor neu startet habe ich bisher noch nicht berücksichtigt. Man müsste den aktuellen Zählerstand in einer zweiten Variable speichern und wenn der neue Zählerstand kleiner als der vorhergehende ist, beide Werte addieren. Das könnte dann "rain_total" werden?

sidey79 · 2023-04-05T21:22:43Z

Hier mal ein paar Daten (ohne Regenfall)

	Line 1: 2023.04.05 04:53:35.205 5: signalESP: Parse_MN, Decoded matched MN Protocol id 126 dmsg=W126#40013E3C90004C50E720A4 RSSI = -87.5
	Line 6: 2023.04.05 04:54:24.204 4: signalESP: Read, msg: �MN;D=40013E3C90004C50E720A7;R=229;�
	Line 7: 2023.04.05 04:54:24.205 5: signalESP: Parse_MN, Decoded matched MN Protocol id 126 dmsg=W126#40013E3C90004C50E720A7 RSSI = -87.5
	Line 8: 2023.04.05 04:55:13.205 4: signalESP: Read, msg: �MN;D=40013E3C90004C50E720A0;R=230;�
	Line 9: 2023.04.05 04:55:13.207 5: signalESP: Parse_MN, Decoded matched MN Protocol id 126 dmsg=W126#40013E3C90004C50E720A0 RSSI = -87
	Line 14: 2023.04.05 04:56:02.231 4: signalESP: Read, msg: �MN;D=40013E3C90004C50E720A7;R=228;�
	Line 15: 2023.04.05 04:56:02.232 5: signalESP: Parse_MN, Decoded matched MN Protocol id 126 dmsg=W126#40013E3C90004C50E720A7 RSSI = -88
	Line 20: 2023.04.05 04:56:51.223 4: signalESP: Read, msg: �MN;D=40013E3C90004C50E720A6;R=229;�
	Line 21: 2023.04.05 04:56:51.225 5: signalESP: Parse_MN, Decoded matched MN Protocol id 126 dmsg=W126#40013E3C90004C50E720A6 RSSI = -87.5
	Line 26: 2023.04.05 04:57:40.232 4: signalESP: Read, msg: �MN;D=40013E3C90004C50E720AA;R=229;�
	Line 27: 2023.04.05 04:57:40.234 5: signalESP: Parse_MN, Decoded matched MN Protocol id 126 dmsg=W126#40013E3C90004C50E720AA RSSI = -87.5
	Line 28: 2023.04.05 04:58:29.229 4: signalESP: Read, msg: �MN;D=40013E3C90004C50E720AC;R=228;�
	Line 29: 2023.04.05 04:58:29.236 5: signalESP: Parse_MN, Decoded matched MN Protocol id 126 dmsg=W126#40013E3C90004C50E720AC RSSI = -88
	Line 34: 2023.04.05 04:59:18.206 4: signalESP: Read, msg: �MN;D=40013E3C90004C50E720A6;R=229;�
	Line 35: 2023.04.05 04:59:18.208 5: signalESP: Parse_MN, Decoded matched MN Protocol id 126 dmsg=W126#40013E3C90004C50E720A6 RSSI = -87.5
	Line 40: 2023.04.05 05:00:07.208 4: signalESP: Read, msg: �MN;D=40013E3C90004C50E720AB;R=229;�
	Line 41: 2023.04.05 05:00:07.210 5: signalESP: Parse_MN, Decoded matched MN Protocol id 126 dmsg=W126#40013E3C90004C50E720AB RSSI = -87.5
	Line 46: 2023.04.05 05:00:56.207 4: signalESP: Read, msg: �MN;D=40013E3C90004C50E720A7;R=229;�
	Line 47: 2023.04.05 05:00:56.209 5: signalESP: Parse_MN, Decoded matched MN Protocol id 126 dmsg=W126#40013E3C90004C50E720A7 RSSI = -87.5
	Line 50: 2023.04.05 05:01:45.208 4: signalESP: Read, msg: �MN;D=40013E3C90004C50E720A6;R=228;�
	Line 51: 2023.04.05 05:01:45.210 5: signalESP: Parse_MN, Decoded matched MN Protocol id 126 dmsg=W126#40013E3C90004C50E720A6 RSSI = -88
	Line 52: 2023.04.05 05:02:34.233 4: signalESP: Read, msg: �MN;D=40013E3C90004C50E720A0;R=228;�
	Line 53: 2023.04.05 05:02:34.234 5: signalESP: Parse_MN, Decoded matched MN Protocol id 126 dmsg=W126#40013E3C90004C50E720A0 RSSI = -88
	Line 58: 2023.04.05 05:03:23.232 4: signalESP: Read, msg: �MN;D=40013E3C90004C50E720A6;R=228;�
	Line 59: 2023.04.05 05:03:23.233 5: signalESP: Parse_MN, Decoded matched MN Protocol id 126 dmsg=W126#40013E3C90004C50E720A6 RSSI = -88
	Line 64: 2023.04.05 05:04:12.235 4: signalESP: Read, msg: �MN;D=40013E3C90004C50E720A9;R=229;�
	Line 65: 2023.04.05 05:04:12.236 5: signalESP: Parse_MN, Decoded matched MN Protocol id 126 dmsg=W126#40013E3C90004C50E720A9 RSSI = -87.5
	Line 70: 2023.04.05 05:05:01.233 4: signalESP: Read, msg: �MN;D=40013E3C90004C50E720A0;R=229;�
	Line 71: 2023.04.05 05:05:01.235 5: signalESP: Parse_MN, Decoded matched MN Protocol id 126 dmsg=W126#40013E3C90004C50E720A0 RSSI = -87.5
	Line 74: 2023.04.05 05:05:50.206 4: signalESP: Read, msg: �MN;D=40013E3C90004C50E720A5;R=229;�
	Line 75: 2023.04.05 05:05:50.208 5: signalESP: Parse_MN, Decoded matched MN Protocol id 126 dmsg=W126#40013E3C90004C50E720A5 RSSI = -87.5
	Line 78: 2023.04.05 05:06:39.209 4: signalESP: Read, msg: �MN;D=40013E3C90004C50E720A3;R=228;�
	Line 79: 2023.04.05 05:06:39.211 5: signalESP: Parse_MN, Decoded matched MN Protocol id 126 dmsg=W126#40013E3C90004C50E720A3 RSSI = -88
	Line 84: 2023.04.05 05:07:28.211 4: signalESP: Read, msg: �MN;D=40013E3C90004C50E720A2;R=229;�
	Line 85: 2023.04.05 05:07:28.213 5: signalESP: Parse_MN, Decoded matched MN Protocol id 126 dmsg=W126#40013E3C90004C50E720A2 RSSI = -87.5
	Line 90: 2023.04.05 05:08:17.210 4: signalESP: Read, msg: �MN;D=40013E3C90004C50E720A6;R=228;�
	Line 91: 2023.04.05 05:08:17.212 5: signalESP: Parse_MN, Decoded matched MN Protocol id 126 dmsg=W126#40013E3C90004C50E720A6 RSSI = -88
	Line 96: 2023.04.05 05:09:06.237 4: signalESP: Read, msg: �MN;D=40013E3C90004C50E720AF;R=228;�
	Line 97: 2023.04.05 05:09:06.240 5: signalESP: Parse_MN, Decoded matched MN Protocol id 126 dmsg=W126#40013E3C90004C50E720AF RSSI = -88
	Line 100: 2023.04.05 05:09:55.236 4: signalESP: Read, msg: �MN;D=40013E3C90004C50E720A4;R=228;�
	Line 101: 2023.04.05 05:09:55.243 5: signalESP: Parse_MN, Decoded matched MN Protocol id 126 dmsg=W126#40013E3C90004C50E720A4 RSSI = -88
	Line 104: 2023.04.05 05:10:44.237 4: signalESP: Read, msg: �MN;D=40013E3C90004C50E720A0;R=228;�
	Line 105: 2023.04.05 05:10:44.238 5: signalESP: Parse_MN, Decoded matched MN Protocol id 126 dmsg=W126#40013E3C90004C50E720A0 RSSI = -88

added Testdata for channel 2 and 3 removed max length check

- Added Protocol 126 WH40 rain - wh40 in commandref ergänzt

- added Protocol 126 WH40 rain gauge

- added rfmode wh40 to commandref

- convert rainvalue from hex to dec

- reemoved rain reading - added rain_total and rawrainCounter - added test with rain

elektron-bbs · 2023-04-22T19:22:53Z

Eben zufällig entdeckt: "Newer version of the WH40 rain gauge report battery voltage."
merbanan/rtl_433#2488
merbanan/rtl_433@c86b41a#diff-bf14f50136e7b7d581a086567c85ce451b0a513ef9aed71bd1b0d6049c1d418c

sidey79 · 2023-04-22T19:30:24Z

Hmm, da bin ich gespannt, ob ich die neuere habe.

sidey79 · 2023-05-06T19:41:25Z

@elektron-bbs

Also ich habe vermutlich nicht den Mut Batterie Übermittlung.
So ungetestet den Wert einbauen ist irgendwie unschön, da wir keine Testdaten haben.

Soll ich den Branch aktualisieren und wir merken mal?

elektron-bbs · 2023-05-06T20:35:51Z

Also ich habe vermutlich nicht den Mut Batterie Übermittlung. So ungetestet den Wert einbauen ist irgendwie unschön, da wir keine Testdaten haben.

Willst du damit sagen. das du deinen Sensor mit verschiedenen Betriebsspannungen getestet hast und er aber nicht die Batteriespannung übermittelt hat?
Wenn dem so ist, so findet sich vielleicht in den RAW-Daten ein Unterschied zwischen alter und neuer Version?

sidey79 · 2023-05-06T21:56:50Z

Also ich habe vermutlich nicht den Mut Batterie Übermittlung. So ungetestet den Wert einbauen ist irgendwie unschön, da wir keine Testdaten haben.

Willst du damit sagen. das du deinen Sensor mit verschiedenen Betriebsspannungen getestet hast und er aber nicht die Batteriespannung übermittelt hat? Wenn dem so ist, so findet sich vielleicht in den RAW-Daten ein Unterschied zwischen alter und neuer Version?

Meiner schickt immer 8F in der Anleitung steht auch, er würde die Batteriespannung nicht übermitteln. Hatte gehofft die sei veraltet, aber wohl eher nicht.

elektron-bbs · 2023-05-07T09:59:17Z

Mhmm, ich sehe in den Daten, die du oben gepostet hast zwar kein 0x8F, sondern immer 0x90, dezimal (Bit 0 bis 4) 16 - entspräche 1,6 Volt. In den Beispielen im Quellcode ist FV immer 0x10.
Was auch auffällt, sind die ersten beiden Bytes. Im Quellcode von rtl_433 ist das immer 0x4000 und bei dir 0x4001.

Die meisten Sensoren senden lt. https://www.wxforum.net/index.php?topic=41764.msg438362#msg438362 wohl keine Batteriespannung.

EDIT:
Wenn du da jetzt 0x8F erhältst, entspräche das ja 1,5 Volt - das würde doch passen.

sidey79 · 2023-05-07T18:47:58Z

Was auch auffällt, sind die ersten beiden Bytes. Im Quellcode von rtl_433 ist das immer 0x4000 und bei dir 0x4001.

Da das 4. Nibble gehört zur ID. Mag auch sein, dass es längere oder kürzere IDs gitb, aber da die ID auch neben am Sensor aufgeklebt ist, ist es eindeutig.

Die meisten Sensoren senden lt. https://www.wxforum.net/index.php?topic=41764.msg438362#msg438362 wohl keine Batteriespannung.

EDIT: Wenn du da jetzt 0x8F erhältst, entspräche das ja 1,5 Volt - das würde doch passen.
Hmm, jetzt regnet es gerade, aber wenn ich jetzt Akkus einsetze müsste ein geringerer Wert kommen.

@Antannah

Was sendet deine WH40 denn als 5. Byte? Ändert sich da was?

sidey79 · 2023-05-09T18:38:26Z

Akku eingebaut , der sollte 1.2V haben.

Seit dem ist der Empfang Schrott aber tatsächlich wird an der vermuteten Stelle nun 8D gesendet.
Wenn die Annahmen stimmen, dann wäre das 1,3 V. Das halte ich hinsichtlich Akku auch für realistisch.

40013E3C8D000FA7FEF1A146B776

Also bauen wir die Batteriespannung ein?

elektron-bbs · 2023-05-09T20:14:36Z

Das passt doch ausgezeichnet - einbauen!
Bliebe noch die Frage, was Bit 7 von diesem Byte bedeutet? Das ist bei dir bisher 1, aber bei den Beispielen 0.

- added batteryVoltage reader for wh40 - added batteryState

FHEM/14_SD_WS.pm

sidey79 · 2023-05-16T16:35:36Z

@elektron-bbs vermutlich brauche ich gleich eine neue Bestätigung, weil der Branch nicht aktuell war.

sidey79 added 2 commits April 2, 2023 00:28

SD_ProtocolData.pm

6f8ce1b

- Added 124 -> Ecowitt WH 31 14_SD_WS.pm - Added 124 -> Ecowitt WH 31 - increased threshhold wait timer for second UNDEFINED event : protocol 107 & 125 - Added crc & checksum check - updated battery - updated temp calc testData.json - Added tests for ecowitt WH31

14_SD_WS.pm

2635a81

- fixed logmessage typos - added some protocol spec - Added dp50 to sensortype - added _TH to model

sidey79 added enhancement SD_WS labels Apr 2, 2023

sidey79 self-assigned this Apr 2, 2023

sidey79 linked an issue Apr 2, 2023 that may be closed by this pull request

Wh40 - rain drop sensor #1162

Closed

sidey79 added 2 commits April 3, 2023 00:50

00_SIGNALduino.pm

9a7d108

added rfomode to commandref

SD_ProtocolData.pm

26c8f20

- Updated length min, max - Added new RF Mode - Updated Testdata - Updated version

sidey79 force-pushed the master-ecowittwh31 branch from a2ee6db to 26c8f20 Compare April 2, 2023 22:51

Update Versiondate

f7618e1

sidey79 force-pushed the master-ecowittwh40 branch from ddcf811 to 4585853 Compare April 2, 2023 23:12

elektron-bbs requested changes Apr 3, 2023

View reviewed changes

FHEM/14_SD_WS.pm Outdated Show resolved Hide resolved

FHEM/14_SD_WS.pm Outdated Show resolved Hide resolved

sidey79 and others added 2 commits April 3, 2023 23:18

Merge branch 'master' into master-ecowittwh31

70aeaff

Update Versiondate

74a133b

elektron-bbs approved these changes Apr 5, 2023

View reviewed changes

elektron-bbs requested changes Apr 5, 2023

View reviewed changes

FHEM/lib/SD_ProtocolData.pm Outdated Show resolved Hide resolved

sidey79 and others added 9 commits April 5, 2023 23:26

testData.json

61e9714

added Testdata for channel 2 and 3 removed max length check

Update Versiondate

0e9c89d

14_SD_WS.pm

44573c7

- Added Protocol 126 WH40 rain - wh40 in commandref ergänzt

SD_ProtocolData.pm

fa1d774

- added Protocol 126 WH40 rain gauge

00_SIGNALduino.pm

235f079

- added rfmode wh40 to commandref

Update Versiondate

6622253

14_SD_WS.pm

b8ff3a4

- convert rainvalue from hex to dec

Update Versiondate

608a54f

14_SD_WS

8061e9c

- reemoved rain reading - added rain_total and rawrainCounter - added test with rain

sidey79 force-pushed the master-ecowittwh40 branch from 070bc0f to 8061e9c Compare April 6, 2023 06:30

Update Versiondate

dfa586a

Base automatically changed from master-ecowittwh31 to master May 1, 2023 11:15

sidey79 and others added 2 commits May 6, 2023 23:33

Merge branch 'master' into master-ecowittwh40

e020793

Update Versiondate

7d6dbfa

sidey79 and others added 2 commits May 10, 2023 08:26

14_SD_WS.pm

c168631

- added batteryVoltage reader for wh40 - added batteryState

Update Versiondate

0807594

elektron-bbs requested changes May 10, 2023

View reviewed changes

FHEM/14_SD_WS.pm Outdated Show resolved Hide resolved

14_SD_WS.pm

f0cdd21

sidey79 force-pushed the master-ecowittwh40 branch from 6b15cfe to f0cdd21 Compare May 11, 2023 21:10

Update Versiondate

323fffb

elektron-bbs previously approved these changes May 16, 2023

View reviewed changes

sidey79 and others added 2 commits May 16, 2023 18:33

Merge branch 'master' into master-ecowittwh40

5cd3176

Update Versiondate

66837d3

sidey79 dismissed elektron-bbs’s stale review via 66837d3 May 16, 2023 16:41

elektron-bbs approved these changes May 16, 2023

View reviewed changes

sidey79 merged commit 62ff1d7 into master May 16, 2023
18 checks passed

sidey79 deleted the master-ecowittwh40 branch May 16, 2023 19:51

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Support ecowitt wh40 rain gauge #1164

Support ecowitt wh40 rain gauge #1164

sidey79 commented Apr 2, 2023 •

edited

codecov bot commented Apr 2, 2023 •

edited

sidey79 commented Apr 5, 2023

elektron-bbs commented Apr 5, 2023

sidey79 commented Apr 5, 2023

elektron-bbs commented Apr 22, 2023

sidey79 commented Apr 22, 2023

sidey79 commented May 6, 2023

elektron-bbs commented May 6, 2023

sidey79 commented May 6, 2023

elektron-bbs commented May 7, 2023 •

edited

sidey79 commented May 7, 2023

sidey79 commented May 9, 2023

elektron-bbs commented May 9, 2023

sidey79 commented May 16, 2023

Support ecowitt wh40 rain gauge #1164

Support ecowitt wh40 rain gauge #1164

Conversation

sidey79 commented Apr 2, 2023 • edited

codecov bot commented Apr 2, 2023 • edited

Codecov Report

sidey79 commented Apr 5, 2023

elektron-bbs commented Apr 5, 2023

sidey79 commented Apr 5, 2023

elektron-bbs commented Apr 22, 2023

sidey79 commented Apr 22, 2023

sidey79 commented May 6, 2023

elektron-bbs commented May 6, 2023

sidey79 commented May 6, 2023

elektron-bbs commented May 7, 2023 • edited

sidey79 commented May 7, 2023

sidey79 commented May 9, 2023

elektron-bbs commented May 9, 2023

sidey79 commented May 16, 2023

sidey79 commented Apr 2, 2023 •

edited

codecov bot commented Apr 2, 2023 •

edited

elektron-bbs commented May 7, 2023 •

edited