[patch] 13_KS300.pm - zusätzliches Reading für Windstärke (bft)

[betateilchen]

Bezugnehmend auf diesen Thread

http://forum.fhem.de/index.php/topic,19988.0.html

schlage ich diesen Patch vor.

--- edit: Patch entfernt, da fehlerhaft.  Weiterentwicklung im Laufe dieses Threads. ---

Die Doku ist noch nicht für das zusätzliche Reading angepasst und die im Thread gestellte Frage bzgl der stateFormat Aktualisierung ist auch noch offen.

[rudolfkoenig]

stateFormat ist (wie du in dem Thread auch schreibst) Teil von readingFnAttributes, und 13_KS300.pm wird mit diesem Patch weder richtig noch vollstaendig auf readings*Update umgestellt. Ein bisschen umstellen mache ich nicht mit, weil ich dann bei der ersten (berechtigten) Fehlermeldung die Arbeit am Hals habe.

Die Umrechnung mit "$w ~~ [10..59|" schaut zwar elegant aus, erzeugt aber ein Feld mit 50 Elementen, und sucht dann $ w in diese Feld, indem es durchiteriert, ist also deutlich ineffizienter als ein Vergleich. Mit einem altmodischen Vergleich & Doku wuerde ich den Patch nehmen.

Uebrigens liefert ein KS300 laut Doku Werte nur bis 99,9, insofern ist Windstaerke 11 und 12 nur fuer die Code-Leser relevant.

Mich wuerde in diesem Zusammenhang deutlich mehr interessieren, wie man den Windmesser eicht.

[betateilchen]

Hallo Rudi,

die Sache mit dem stateFormat ist - wie erwartet - schon schiefgegangen (siehe den Wunschthread), das werde ich wieder rausnehmen.
Die Sache mit dem Vergleich kann ich gerne umbauen und die Windstärken 11 und 12 auskommentieren .

Danke für Deine Hinweise, ich melde mich hier wieder.

Viele Grüße
Udo

[Michael]

Hallo Rudi, betateilchen

Uebrigens liefert ein KS300 laut Doku Werte nur bis 99,9, insofern ist Windstaerke 11 und 12 nur fuer die Code-Leser relevant.

Mich wuerde in diesem Zusammenhang deutlich mehr interessieren, wie man den Windmesser eicht.

Wenn ich das richtig auf Seite 7 lese können schon Geschwindigkeiten bis 200 km/h gemessen werden.
http://www.elv-downloads.de/service/manuals/KS300/65390_um.pdf

Oder eine Annahme ist total falsch, was gut möglich ist

[det.]

Fein, dann warten wir hier mal in Startposition bis es eine neue Version zum Testen gibt. Jedenfalls ist das prima, wenn dieser Modul auch auf das stateFormat umgestellt wird.
Das Eichen des Windmessers ist mMn weniger das Problem, als ein Wetterwarten adäquater Aufstellungsort des K300. Solange das Ding irgendwo zwischen Haus und Nachbargarage steht (funktechnisch optimal) und nicht auf freiem Feld liefert das eh nur Relativwerte.

[rudolfkoenig]

können schon Geschwindigkeiten bis 200 km/h gemessen werden.

Bei mir stand noch 99.9 in der Anleitung drin. Ich glaube sowohl das als auch das 200 ist irgendetwas, was die Marketing-Leute erfunden haben. Auch deswegen wuerde ich gerne eichen...nein, kalibrieren ist das richtige Wort.

Jedenfalls ist das prima, wenn dieser Modul auch auf das stateFormat umgestellt wird.

Das hat hier keiner behauptet...

[stromer-12]

Bei mir empfange ich einen Windmesser eines Nachbarn. Wenn man die Werte betrachtet, lierfert der m/s und nicht km/h.

[betateilchen]

grnpf... was denn nun? 99.9 oder 200? Ich werde einfach die komplette Liste bis 12 drinlassen.

@Rudi: um auf readingsFnAttr umzustellen, müsste man

• alle $def->{READINGS} (im Coding also $r) auf readings...Update() umstellen
• die Zeilen mit CHANGED alle entfernen

Richtig? Ich frage das, weil diese Logik noch aus meiner "vor-fhem-Developer-Zeit" stammt und ich mich mit dem alten Mechanismus nie beschäftigt habe.

Und an der Doku muss ich gar nix wichtiges ändern, da sind nämlich gar keine Readings beschrieben 

[betateilchen]

Bei mir empfange ich einen Windmesser eines Nachbarn. Wenn man die Werte betrachtet, lierfert der m/s und nicht km/h.

Und wenn Du im define den windfaktor angibst, kannst Du km/h bekommen, siehe commandref.

[stromer-12]

Und wenn Du im define den windfaktor angibst, kannst Du km/h bekommen, siehe commandref.

Den Windfaktor habe ich irgendwie überlesen, Danke für den Tipp.

[betateilchen]

Here we go again...

Alle interessierten Tester mögen bitte diesen Post bis zum Ende lesen!

Ein bisschen umstellen mache ich nicht mit, weil ich dann bei der ersten (berechtigten) Fehlermeldung die Arbeit am Hals habe.

Verlangt auch niemand von Dir. Ich habe Dir die Umstellung komplett abgenommen

Mit einem altmodischen Vergleich & Doku wuerde ich den Patch nehmen.

Geändert.

Uebrigens liefert ein KS300 laut Doku Werte nur bis 99,9, insofern ist Windstaerke 11 und 12 nur fuer die Code-Leser relevant.

Aufgrund der hier aufgekommenen Diskussion habe ich die komplette Zuordnungstabelle bis bft 12 beibehalten.

Mich wuerde in diesem Zusammenhang deutlich mehr interessieren, wie man den Windmesser eicht.

Da kann ich Dir leider nicht weiterhelfen, da ich den Sensor weder selbst besitze noch ihn kenne. Und in meiner Balkonwohnlage würde der auch keinen Sinn machen.



Wichtig für alle Tester!

• In der angehängten Version gab es viele Änderungen zur Einbindung des Moduls in fhem (Umstellung auf readings..Update).
• An der Berechnung der angezeigten Werte wurde grundsätzlich nichts verändert, die zugrundeliegenden Werte mussten allerdings auf ReadingsVal() umgestellt werden und ich hoffe, dass ich dabei keinen Fehler gemacht habe.
• stateFormat sollte damit jetzt auch funktionieren.

Bitte macht Euch einen Screenshot Eurer Readings VOR DEM Update und vergleicht nach dem Modulupdate alle Werte auf Plausibilität!

[det.]

sorry, im LOG steht:

Code Auswählen Erweitern

2014.02.16 14:29:51 1: readingsUpdate(CUL_0,tsecs,1392557391) missed to call readingsBeginUpdate first.
2014.02.16 14:29:51 1: readingsUpdate(CUL_0,rain_raw_adj,939) missed to call readingsBeginUpdate first.
2014.02.16 14:32:23 1: readingsUpdate(CUL_0,tsecs,1392557543) missed to call readingsBeginUpdate first.
2014.02.16 14:32:23 1: readingsUpdate(CUL_0,rain_raw_adj,939) missed to call readingsBeginUpdate first.
2014.02.16 14:34:56 1: readingsUpdate(CUL_0,tsecs,1392557696) missed to call readingsBeginUpdate first.
2014.02.16 14:34:56 1: readingsUpdate(CUL_0,rain_raw_adj,939) missed to call readingsBeginUpdate first.
2014.02.16 14:37:28 1: readingsUpdate(CUL_0,tsecs,1392557848) missed to call readingsBeginUpdate first.
2014.02.16 14:37:28 1: readingsUpdate(CUL_0,rain_raw_adj,939) missed to call readingsBeginUpdate first
und im state defined, sonst passiert nichts

[betateilchen]

Dafür brauchst DU Dich doch nicht entschuldigen. Ich kann das Modul hier mangels Sensor nur auf syntaktische Fehler testen.

War mein Fehler, das readingsBeginUpdate stand an der falschen Stelle

Teste bitte nochmal.

[det.]

irgendwie leitet es state auf CUL_0 um - im KS300 steht:

CODE

1234

[/q][/t]

CUL_0_MSGCNT

2

CUL_0_RAWMSG	K1794000600AB6305
CUL_0_RSSI	-71.5

CUL_0_TIME	2014-02-16 15:23:16
	1234

IODev	CUL_0
LASTInputDev	CUL_0

MSGCNT	2
NAME	KS300

NR	212
RAINUNIT	255

STATE	defined
TYPE	KS300

WINDUNIT

1

Readings

avg_day

T: 7.0 H: 73 W: 0.7 R: 0.0

2014-02-16 14:17:08

[/t]

avg_month

T: 3.6 H: 42 W: 39.8 R: 5.3

2014-02-16 00:03:05

cum_day	2014-02-16 00:03:05 T: 356412.7 H: 3776741 W: 35773.2 R: 239.4	2014-02-16 14:17:08
cum_month	15 T: 54.4 H: 633 W: 597.5 R: 5.3	2014-02-16 00:03:05

humidity	60	2014-02-16 14:17:08
israining	no	2014-02-16 14:17:08

rain	239.4	2014-02-16 14:17:08
rain_raw	939	2014-02-16 14:17:08

rain_raw_adj	939	2014-02-16 14:17:08
state	defined	2014-02-16 15:19:35

temperature	9.7	2014-02-16 14:17:08
tsecs	1392556628	2014-02-16 14:17:08

type_raw	7	2014-02-16 14:17:08
unknown1	2	2014-02-16 14:17:08

unknown3	1	2014-02-16 14:17:08
wind	2.7	2014-02-16 14:17:08

windIndex

1

2014-02-16 14:17:08

und im CUL_0

Code Auswählen Erweitern

CMDS
BCFiAZEGMRTVWXefmltux
CUL_0_MSGCNT
43
CUL_0_TIME
2014-02-16 15:29:39
Clients
:FS20:FHT.*:KS300:USF1000:BS:HMS: :CUL_EM:CUL_WS:CUL_FHTTK:CUL_RFR:CUL_HOERMANN: :ESA2000:CUL_IR:CUL_TX:Revolt:IT:
DEF   
/dev/ttyACM_CUL@9600 1034
DeviceName
/dev/ttyACM_CUL@9600
FD
11
FHTID
1034
NAME
CUL_0
NR
22
PARTIAL
RAWMSG
T7C99688224
RSSI
-56
STATE
T: 9.4 H: 60 W: 0.0 R: 239.4 IR: no Wi: 0
TYPE
CUL
VERSION
V 1.55 CUL868
initString
X21
Readings
avg_day
T: 7.1 H: 72 W: 0.6 R: 0.0
2014-02-16 15:23:16
cmds
B C F i A Z E G M R T V W X e f m l t u x
2014-02-16 15:19:29
cum_day
2014-02-16 00:03:05 T: 393711.9 H: 4014821 W: 35773.2 R: 239.4
2014-02-16 15:23:16
humidity
60
2014-02-16 15:23:16
israining
no
2014-02-16 15:23:16
rain
239.4
2014-02-16 15:23:16
rain_raw
939
2014-02-16 15:23:16
rain_raw_adj
939
2014-02-16 15:23:16
raw
is0F00F0FFFFFF
2014-01-25 16:16:03
state
T: 9.4 H: 60 W: 0.0 R: 239.4 IR: no Wi: 0
2014-02-16 15:29:39
temperature
9.4
2014-02-16 15:23:16
tsecs
1392560596
2014-02-16 15:23:16
type_raw
7
2014-02-16 15:23:16
unknown1
6
2014-02-16 15:23:16
unknown3
1
2014-02-16 15:23:16
wind
0.0
2014-02-16 15:23:16
windIndex
0
2014-02-16 15:23:16


[betateilchen]

das heißt, alle anderen Readings - außer state - funktionieren jetzt?

Doofe Frage, sorry. Sie funktionieren natürlich nicht. Ich hatte nicht weit genug runtergescrollt.