-
Notifications
You must be signed in to change notification settings - Fork 33
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Support ecowitt wh40 rain gauge #1164
Conversation
- 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 Report
@@ 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
Flags with carried forward coverage won't be shown. Click here to find out more.
... 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
a2ee6db
to
26c8f20
Compare
ddcf811
to
4585853
Compare
Ich habe da aber noch eine Frage zu den Rain Readings. Da gibt es ja durchaus mehrere. Welches Reading soll ich da nehmen und wie rechnen wir dann die anderen readings aus. |
So ist es wohl bei den meisten Regensensoren.
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? |
Hier mal ein paar Daten (ohne Regenfall)
|
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
070bc0f
to
8061e9c
Compare
Eben zufällig entdeckt: "Newer version of the WH40 rain gauge report battery voltage." |
Hmm, da bin ich gespannt, ob ich die neuere habe. |
Also ich habe vermutlich nicht den Mut Batterie Übermittlung. Soll ich den Branch aktualisieren und wir merken mal? |
Willst du damit sagen. das du deinen Sensor mit verschiedenen Betriebsspannungen getestet hast und er aber nicht die Batteriespannung übermittelt hat? |
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. |
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. Die meisten Sensoren senden lt. https://www.wxforum.net/index.php?topic=41764.msg438362#msg438362 wohl keine Batteriespannung. EDIT: |
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.
Was sendet deine WH40 denn als 5. Byte? Ändert sich da was? |
Akku eingebaut , der sollte 1.2V haben. Seit dem ist der Empfang Schrott aber tatsächlich wird an der vermuteten Stelle nun 8D gesendet. 40013E3C8D000FA7FEF1A146B776 Also bauen wir die Batteriespannung ein? |
Das passt doch ausgezeichnet - einbauen! |
- added batteryVoltage reader for wh40 - added batteryState
6b15cfe
to
f0cdd21
Compare
@elektron-bbs vermutlich brauche ich gleich eine neue Bestätigung, weil der Branch nicht aktuell war. |
(You can also link to an open issue here, if this describes the current behavior)
WH40 rain gauge sensor is not decoded
WH40 rain gauge sensor is decoded
no
Test this PR:
update all https://raw.githubusercontent.com/RFD-FHEM/RFFHEM/master-ecowittwh40/controls_signalduino.txt
#1162