Wetterstation Binding TE923 Berechtigung

[labm]

Hallo,

ich habe dieses Binding für meine Wetterstation Installiert.
Bekomme die Berechtigungen für mein Openhab2 nicht so gesetzt das das funktioniert.

Wenn ich in Putty Commando sudo te923con aufrufe bekomme ich die Daten angezeigt.
Jetzt muss es nur noch Berechtigt werden(gehe ich stark davon aus..).

In derm Thing in der Paper UI bekomme ich fogender fehler

Unexpected answer from cmd command line: Error while setting configuration (-1). Error while setting configuration (-1). - Size is 1 and should be 22

Bekomme bei
[22:02:42] openhabian@RaspberryPi:~$ sudo -u openhab /usr/bin/te923con
Error while setting configuration (-1).



Bitte um Hilfe.

Danke

[udo1toni]

Wie funktioniert die Kommunikation? zwischen

Code: Alles auswählen

sudo te923con

und

Code: Alles auswählen

sudo -u openhab /usr/bin/te923con

besteht ein himmelweiter Unterschied. Der erste Befehl führt das Kommando als Superuser root aus, der zweite Befehl führt das Kommando als User openhab aus. Im Zweifel möchtest Du, dass der User openhab den Befehl als Superuser root abschickt.
Dazu musst Du dem User openhab erlauben, diesen Befehl (und genau diesen einen!) mittels sudo auszuführen, und zwar ohne die Notwendigeit, ein Password einzugeben.
Der notwendige Beehl lautet

Code: Alles auswählen

sudo visudo

um die entsprechende Konfigurationsdatei zu bearbeiten. visudo verhindert, dass Du die Datei kaputt konfigurierst und Dich damit eventuell aus dem System aussperrst.
Innerhalb der Datei legst Du eine Zeile an:

Code: Alles auswählen

openhab    ALL=(ALL:ALL) NOPASSWD: /usr/bin/te923con

Und zwar unter der Zeile, die so ähnlich aussieht und mit root beginnt.

Anschließend musst Du den Befehl einmalig über die Konsole aufrufen, und zwar als User openhab:

Code: Alles auswählen

sudo -u openhab sudo /usr/bin/te923con

Dabei wirst Du dann nicht nur nach dem Passwort gefragt, sondern bekommst einmalig einen Warnhinweis, dass mit großer Macht große Verantwortung einhergeht. Diesen musst Du bestätigen, woraufhin der eigentliche Befehl ausgeführt werden müsste.
Danach sollte der Befehl einfach mittels

Code: Alles auswählen

/sbin/sudo /usr/bin/te923con

über das exec Binding ausgeführt werden können.

[labm]

Hallo Udo1Toni,
das ist supper beschrieben 5 Sterne:)
Ich habe das bereits vor paar Stunden hinbekommen, woran das lag kann ich nicht genau sagen habe zu schluß reboot gamacht danach hat das funktioniert.

Dieser Hinweise hier sind trotz dem sehr hilfreich.
Vielen Dank

Die Manuelle Things, Items, funktionieren aber immernoch nicht...
Ist hier was falsch es soll auf das gleicher pfad gehen und die Daten rausholen

Thing exec:command:te923cmd [command="/usr/bin/te923con", interval=60, timeout=5, autorun=false]

String te923_raw "[%s]" {channel="exec:command:te923cmd:output"}

Number te923_t0 "Temperature 0 [%.1f °C]" <temperature> (gTe923)
Number te923_h0 "Humidity 0 [%.0f %%]" <humidity> (gTe923)
Number te923_t1 "Temperature 1 [%.1f °C]" <temperature> (gTe923)
Number te923_h1 "Humidity 1 [%.0f %%]" <humidity> (gTe923)
Number te923_t2 "Temperature 2 [%.1f °C]" <temperature> (gTe923)
Number te923_h2 "Humidity 2 [%.0f %%]" <humidity> (gTe923)
Number te923_t3 "Temperature 3 [%.1f °C]" <temperature> (gTe923)
Number te923_h3 "Humidity 3 [%.0f %%]" <humidity> (gTe923)
Number te923_t4 "Temperature 4 [%.1f °C]" <temperature> (gTe923)
Number te923_h4 "Humidity 4 [%.0f %%]" <humidity> (gTe923)
Number te923_t5 "Temperature 5 [%.1f °C]" <temperature> (gTe923)
Number te923_h5 "Humidity 5 [%.0f %%]" <humidity> (gTe923)
Number te923_press "Pressure [%.1f mBar]" <pressure> (gTe923)
Number te923_uv "UV [%.0f]" <sun> (gTe923)
// The external part of my weather station is HS, don't hesitate to uncomment...
// Number te923_fc "Forecast [%.0f]" <clouds> (gTe923)
// Number te923_storm "Storm Warning [%.0f]" <wind> (gTe923)
// Number te923_wd "Wind Direction [%.1f]" <sun> (gTe923)
// Number te923_ws "Wind Speed [%.1f] m/s" <sun> (gTe923)
// Number te923_wg "Wind Gust speed [%.1f] m/s" <sun> (gTe923)
// Number te923_wc "Windchill temperature [%.1f] °c" <sun> (gTe923)
// Number te923_rc "Rain Counter [%.0f]" <sun> (gTe923)

Group gTe923 "Temperature from TE923 Weather Station" <temperature>


Rules
rule "Process data from TE923"
when
Item te923_raw received update
then

var e = te923_raw.state.toString();
var s = e.split(":");

if (s.length<22 || s.length>25) {
logInfo("TE923 Rule", "te923_raw is not of the expected format: "+e);
return;
}

// T0:H0:T1:H1:T2:H2:T3:H3:T4:H4:T5:H5:PRESS:UV:FC:STORM:WD:WS:WG:WC:RC
// T0 - temperature from internal sensor in °C
// H0 - humidity from internal sensor in % rel
// T1..5 - temperature from external sensor 1..4 in °C
// H1..5 - humidity from external sensor 1...4 in % rel
// PRESS - air pressure in mBar
// UV - UV index from UV sensor
// FC - station forecast, see below for more details
// STORM - stormwarning; 0 - no warning, 1 - fix your dog
// WD - wind direction in n x 22.5°; 0 -> north
// WS - wind speed in m/s
// WG - wind gust speed in m/s
// WC - windchill temperature in °C
// RC - rain counter (maybe since station starts measurement) as value


//var unixdate = s.get(0);

var t0 = s.get(1);
if (!t0.equals("i")) {
sendCommand(te923_t0, t0);
}

var h0 = s.get(2);
if (!h0.equals("i")) {
sendCommand(te923_h0, h0);
}

var t1 = s.get(3);
if (!t1.equals("i")) {
sendCommand(te923_t1, t1);
}
usw..ist hier nicht alles


Gruß

LABM

[labm]

Thing habe ich gerade so geändert, funktioniert leider nicht.

Thing exec:command:te923cmd [command="/sbin/sudo /usr/bin/te923con", interval=60, timeout=5, autorun=false]

[udo1toni]

Die Frage ist: funktioniert

Code: Alles auswählen

sudo -u openhab sudo /usr/bin/te923con

wie gewünscht?
Deine Rule sieht schon mal nicht schlecht aus, allerdings gibt es da doch Verbesserungspotiential
Zum einen gibt es in der Rules DSL exakt einen Befehl, bei dem man ein Semikolon dazu schreiben sollte, das ist return; Bei allen anderen Befehlen hat das Semikolon nichts zu suchen.
Das Andere ist Dir sicher schon aufgefallen, die Rule hat ihre Längen
Dazu habe ich aber zuerst noch eine Frage, nämlich: ist die Anzahl der übergebenen Werte tatsächlich variabel? und werden dann tatsächlich die letzten Werte nicht gebraucht?
Sinnvoller wäre es, die Items in diesem Fall durchzunumerieren (das geht auch zusätzlich zum eigentlichen Namen):

Code: Alles auswählen

String te923_raw "[%s]" {channel="exec:command:te923cmd:output"} 

Number te923_t0_01 "Temperature 0 [%.1f °C]"         <temperature> (gTe923)
Number te923_h0_02 "Humidity 0 [%.0f %%]"            <humidity>    (gTe923)
Number te923_t1_03 "Temperature 1 [%.1f °C]"         <temperature> (gTe923)
Number te923_h1_04 "Humidity 1 [%.0f %%]"            <humidity>    (gTe923)
Number te923_t2_05 "Temperature 2 [%.1f °C]"         <temperature> (gTe923)
Number te923_h2_06 "Humidity 2 [%.0f %%]"            <humidity>    (gTe923)
Number te923_t3_07 "Temperature 3 [%.1f °C]"         <temperature> (gTe923)
Number te923_h3_08 "Humidity 3 [%.0f %%]"            <humidity>    (gTe923)
Number te923_t4_09 "Temperature 4 [%.1f °C]"         <temperature> (gTe923)
Number te923_h4_10 "Humidity 4 [%.0f %%]"            <humidity>    (gTe923)
Number te923_t5_11 "Temperature 5 [%.1f °C]"         <temperature> (gTe923)
Number te923_h5_12 "Humidity 5 [%.0f %%]"            <humidity>    (gTe923)
Number te923_press_13 "Pressure [%.1f mBar]"         <pressure>    (gTe923)
Number te923_uv_14 "UV [%.0f]"                       <sun>         (gTe923)
// The external part of my weather station is HS, don't hesitate to uncomment...
Number te923_fc_15 "Forecast [%.0f]"                 <clouds>      (gTe923)
Number te923_storm_16 "Storm Warning [%.0f]"         <wind>        (gTe923)
Number te923_wd_17 "Wind Direction [%.1f]"           <sun>         (gTe923)
Number te923_ws_18 "Wind Speed [%.1f] m/s"           <sun>         (gTe923)
Number te923_wg_19 "Wind Gust speed [%.1f] m/s"      <sun>         (gTe923)
Number te923_wc_20 "Windchill temperature [%.1f] °c" <sun>         (gTe923)
Number te923_rc_21 "Rain Counter [%.0f]"             <sun>         (gTe923)

Nun wird der Code "etwas" kürzer:

Code: Alles auswählen

// globale Variablen werden zu Beginn der Datei definiert!
var s = ["0","0"]

        // unixdate:T0:H0:T1:H1:T2:H2:T3:H3:T4:H4:T5:H5:PRESS:UV:FC:STORM:WD:WS:WG:WC:RC
        // T0 - temperature from internal sensor in °C
        // H0 - humidity from internal sensor in % rel
        // T1..5 - temperature from external sensor 1..5 in °C
        // H1..5 - humidity from external sensor 1...5 in % rel
        // PRESS - air pressure in mBar
        // UV - UV index from UV sensor
        // FC - station forecast, see below for more details
        // STORM - stormwarning; 0 - no warning, 1 - fix your dog 
        // WD - wind direction in n x 22.5°; 0 -> north
        // WS - wind speed in m/s
        // WG - wind gust speed in m/s
        // WC - windchill temperature in °C
        // RC - rain counter (maybe since station starts measurement) as value

rule "Process data from TE923"
when
    Item te923_raw received update
then
    s = te923_raw.state.toString.split(":")
    if (s.size<22 || s.size>25) {                                                     // .size sollte die Anzahl der Felder sein.
        logWarn("te923", "te923_raw is not of the expected format: {}",te923_raw.state)
        return;
    }
    gTe923.members.forEach[ i |
        val myNum = Integer::parseInt(i.name.split("_").get(2))
        val t = s.get(myNum)
        if(!t.equals("i"))
            i.postUpdate(t)
    ]
end

Der Block mit der Erklärung, wie das Datentelegramm aufgebaut ist, ist schön für die Dokumentation, allerdings könnte man das vielleicht woanders notieren
Der Code splittet wie gehabt den Raw Code und durchläuft anschließend die Gruppe der Items, die zu der Wetterstation gehören. Da im Namen des Items die Nummer der Wertes angegeben ist, muss nur der entsprechende Wert übertragen werden.
Ein wichtiges Detail ist hierbei, dass ich nicht sendCommand() verwende, sondern postUpdate(). postUpdate() setzt den Status, sendCommand() sendet einen Befehl. Solange ein Item nicht mit einem Binding gekoppelt ist, gibt es keinen Empfänger für einen solchen Befehl. openHAB führt automatisch ein postUpdate() aus, sobald es ein sendCommand() empfängt, aber warum über Bande, wenn es auch direkt geht?
Die Methode ist grundsätzlich der Action vorzuziehen, da sie weniger kritisch auf die verschiedenen Datentypen reagiert (das hängt damit zusammen, dass die Methode Kenntnis vom Datentyp des Items hat, was der Action verwehrt bleibt).
Es müssen auch nicht alle Items angelegt werden, nur die Nummer muss natürlich stimmen. Wenn Du also z.B. nur einen exernen Sensor hast, lässt Du die anderen Temperatur/Luftfeuchte Items einfach weg, das Item für Pressure behält aber die 13 am Ende. Die Gruppe der Items ist dann halt kleiner und die Rule schreibt entsprechend weniger Werte in die Items.

[labm]

Hallo

vielen Dank,

habe sendcommand auf postUpdate geändert es geht aber trotz dem nicht.

In der Logdatei steht folgendes kann ich auch nicht nachvollzieen.

2019-06-25 22:43:37.368 [WARN ] [sitemap.internal.SitemapProviderImpl] - Filename `te923.sitemap` does not match the name `temperature` of the sitemap - please fix this as you might see unexpected behavior otherwise.

2019-06-25 22:43:37.372 [WARN ] [sitemap.internal.SitemapProviderImpl] - Filename `te923.sitemap` does not match the name `temperature` of the sitemap - please fix this as you might see unexpected behavior otherwise.

2019-06-25 22:43:38.928 [WARN ] [sitemap.internal.SitemapProviderImpl] - Filename `te923.sitemap` does not match the name `temperature` of the sitemap - please fix this as you might see unexpected behavior otherwise.

2019-06-25 22:43:41.548 [WARN ] [sitemap.internal.SitemapProviderImpl] - Filename `te923.sitemap` does not match the name `temperature` of the sitemap - please fix this as you might see unexpected behavior otherwise.

Gruß
Labm

[udo1toni]

Die Fehlermeldung ist eindeutig. Im Kopf der Datei steht bei Dir

Code: Alles auswählen

sitemap temperature 

aber Du hast die Datei te923.sitemap genannt.

Hat aber nichts mit Deinem Problem zu tun, bei welchem Du aber auch nicht weiter darauf eingegangen bist, ob Du außer sendCommand durch postUpdate zu ersetzen noch etwas anderes geändert hast.

[labm]

genau te923.sitemap

Hier der Inhalt


sitemap temperature label="Temperature"
{
Frame label="Wetterstation" {
Group item=gTe923 label="Dachboden"
}

}

[labm]

Ich verstehen den Zusammenhang nicht ganz.
1. Der Thing holt sich hier (alle 60 sekunden) von der Wetterstation den Datensatz aus dem Pfad
2. Die Rules greifen sich den Satz schneiden di Inhalte ausenander
3. Dann werden die Items mit dem Inhat befüllt
4. Anzeige in dem Sitemap

[udo1toni]

Wie erwähnt. Dein Problem mit der Sitemap ist, dass der Dateiname nicht beliebig ist, sondern dem Namen der Sitemap entsprechen muss. Entweder, Du änderst den Namen der Sitemap:

Code: Alles auswählen

sitemap te923 label="Temperature" {
    Frame label="Wetterstation" {
        Group item=gTe923 label="Dachboden"
    }
}

oder Du änderst den Dateinamen in temperature.sitemap jedenfalls muss beides zueinander passen.

Nochmal die Frage: Was genau funktioniert denn nun nicht?

1. das Script wird erfolgreich ausgeführt, wenn es mit sudo ausgeführt wird?
2. openHAB kann das script erfolgreich ausführen, das heißt, der output Channel des exec Things wird mit den Datentelegrammen gefüllt?
3. Die Daten landen im String Item te923_raw?
4. Die Rule wird getriggert und füllt die Items?
5. Die Daten werden auf der Sitemap angezeigt?

Ab welchem Punkt klappt es nicht?

Hast Du Dein Script in Betrieb (unverändert bis auf postUpdate statt sendCommand) oder hast Du

• Die Action durch die Methode ersetzt
• Die falsch gesetzten Semicola entfernt
• Die Rule dahingehend geändert, dass sie gruppenbasiert arbeitet (und damit ca. 90% Codezeilen spart)

?
Fragen über Fragen...