[Gelöst] MQTT für WLED, rgb reading mit # klappt nicht

[DeeSPe]

Sorry, ich hatte nicht gesehen dass das Internal mit der IP den Namen basierend auf dem IODev hat.
Habe die Funktion "WLED_getNames" nun oben nochmal entsprechend angepasst.
Nun sollte es klappen.

Gruß
Dan

EDIT: Musste noch eine kleine Änderung in WLED_getNames machen.
EDIT2: Musste die Ermittlung der IP Adresse auch noch einmal anpassen.

[Beta-User]

Ähm, kurz der Hinweis: Ich lese mit Interesse mit, habe aber im Moment noch keine Idee, wie man daraus eine sinnvolle attrTemplate-Erweiterung basteln könnte...

Hinfreich wäre es aber in jedem Fall, wenn man die IP-Adresse auch in einem Reading "verewigen" könnte, dann können es auch Leute nutzen, die MQTT2_CLIENT als IO verwenden, und den Zugriff auf den IODev-Namen könnte man über über eine etwas andere Auswertung dynamisieren. Steinbruch-Schnipsel:

Code Auswählen Erweitern

InternalVal('DEVICE','LASTInputDev',AttrVal('DEVICE','IODev',InternalVal('DEVICE','IODev',undef)->{NAME})) }
müßte angepaßt also sowas ergeben:

Code Auswählen Erweitern

InternalVal($dev,'LASTInputDev',AttrVal($dev,'IODev',InternalVal($dev,'IODev',undef)->{NAME})) }

[DeeSPe]

Ähm, kurz der Hinweis: Ich lese mit Interesse mit, habe aber im Moment noch keine Idee, wie man daraus eine sinnvolle attrTemplate-Erweiterung basteln könnte...

Hinfreich wäre es aber in jedem Fall, wenn man die IP-Adresse auch in einem Reading "verewigen" könnte, dann können es auch Leute nutzen, die MQTT2_CLIENT als IO verwenden, und den Zugriff auf den IODev-Namen könnte man über über eine etwas andere Auswertung dynamisieren. Steinbruch-Schnipsel:

Code Auswählen Erweitern

InternalVal('DEVICE','LASTInputDev',AttrVal('DEVICE','IODev',InternalVal('DEVICE','IODev',undef)->{NAME})) }
müßte angepaßt also sowas ergeben:

Code Auswählen Erweitern

InternalVal($dev,'LASTInputDev',AttrVal($dev,'IODev',InternalVal($dev,'IODev',undef)->{NAME})) }

Ich hatte auch schon überlegt wie das in ein AttrTemplate zu gießen wäre. Bisher bin ich aber nicht auf eine Lösung gekommen wie man durch AttrTemplate die benötigten Funktionen in die "99_myUtils.pm" bekommt. Oder hast Du da eine Idee?
Bei MQTT2 steht das IODev doch im Reading IODev. So habe ich es jetzt zumindest im Code umgesetzt.

Gruß
Dan

[Beta-User]

Readings sind uU. trügerisch, insbes. was IODev angeht (siehe andere Diskussion dazu).

myUtils-Code dazu zur Verfügung zu stellen ist kein Problem, das passiert an anderer Stelle auch "ständig"... (ebus, Sauger, z2m-Thermostate, sonos2mqtt...). Allerdings gleich die Warnung: Jedenfalls ich werde den nicht so "dreckig" einchecken , wenn, dann muss der auf ein "akzeptables Qualitäts-Level" wie in etwa die anderen Beispiele auch, die in https://svn.fhem.de/trac/browser/trunk/fhem/contrib/AttrTemplate/ zu finden sind .

[TomLee]

Weniger sauber oder nicht solange das nicht vertemplated  worden ist, gehört die Variablendeklaration von $chash wieder ergänzt die bei der letzten Änderung untergegangen ist.

Code Auswählen Erweitern

my $chash = $defs{$dev};

[DeeSPe]

Oh, das wusste ich bisher nicht dass man Code so für AttrTemplate zur Verfügung stellen kann.
Was meinst Du mit "dreckig" und "akzeptables Qualitäts-Level"? Dass ich den Code in so eine Datei auslagern soll?
Denn ich habe mir beim Code schreiben Mühe gegeben und finde ihn alles andere als "dreckig".

Gruß
Dan

[DeeSPe]

Weniger sauber oder nicht solange das nicht vertemplated  worden ist, gehört die Variablendeklaration von $chash wieder ergänzt die bei der letzten Änderung untergegangen ist.

Code Auswählen Erweitern

my $chash = $defs{$dev};

Danke für den Hinweis.
Habe ich wohl versehentlich vorhin beim ändern der $ip Variablen mit rausgelöscht.
Ist wieder eingefügt.

Gruß
Dan

P.S. Danke für's Testen und Probleme finden.

[Beta-User]

Oh, das wusste ich bisher nicht dass man Code so für AttrTemplate zur Verfügung stellen kann.

Dann mal Willkommen an Bord .

Was meinst Du mit "dreckig" und "akzeptables Qualitäts-Level"? Dass ich den Code in so eine Datei auslagern soll?
Denn ich habe mir beim Code schreiben Mühe gegeben und finde ihn alles andere als "dreckig".

War nicht persönlich gemeint, und das sieht auch prinzipiell "ganz ok" aus, erfüllt aber nicht diese beiden "essentials" (aus meiner Warte):
- gepackaged
- perlcritic meckert höchstens bei Kleinigkeiten auf Level 3 rum (Pflaster sind nur erlaubt, was den package-Namen angeht; hier sollte das ohne weitere Meckereien gehen, "verhauen lassen" geht kostenlos online unter perlcritic.com)

[australien]

P.S. Danke für's Testen und Probleme finden.

Wieso steht eigentlich bei setList

Code Auswählen Erweitern

effectname:{"select,".join(",",sort(split(",",ReadingsVal($name,".effects","Solid,Police"))))} {WLED_setName($NAME,"effect",$EVTPART1)}
"Solid,Police", dass sind die einzigen Effecte die ich sehe!?
Also warum die im ReadinsVal sind, ist mir schon klar, aber warum wird sonst nichts gefunden, bei den Paletten funktioniert es.

[DeeSPe]

Wieso steht eigentlich bei setList

Code Auswählen Erweitern

effectname:{"select,".join(",",sort(split(",",ReadingsVal($name,".effects","Solid,Police"))))} {WLED_setName($NAME,"effect",$EVTPART1)}
"Solid,Police", dass sind die einzigen Effecte die ich sehe!?
Also warum die im ReadinsVal sind, ist mir schon klar, aber warum wird sonst nichts gefunden, bei den Paletten funktioniert es.

Das ist nur als Fallback angegeben damit das neu entstandene Dropdown schon einen Inhalt hat bevor man 1x einen Effekt per Nummer ausgewählt hat.
Hast Du denn schon 1x einen Effekt per Nummer aufgerufen (z.B. "set <WLED> effect 10")? Denn nur so entsteht der richtige Inhalt der Dropdowns.

Gruß
Dan

[DeeSPe]

Dann mal Willkommen an Bord .
War nicht persönlich gemeint, und das sieht auch prinzipiell "ganz ok" aus, erfüllt aber nicht diese beiden "essentials" (aus meiner Warte):
- gepackaged
- perlcritic meckert höchstens bei Kleinigkeiten auf Level 3 rum (Pflaster sind nur erlaubt, was den package-Namen angeht; hier sollte das ohne weitere Meckereien gehen, "verhauen lassen" geht kostenlos online unter perlcritic.com)

Ich versuche mal Deine Punkte umzusetzen und gebe Dir dann wieder Bescheid.

Gruß
Dan

[australien]

Das ist nur als Fallback angegeben damit das neu entstandene Dropdown schon einen Inhalt hat bevor man 1x einen Effekt per Nummer ausgewählt hat.
Hast Du denn schon 1x einen Effekt per Nummer aufgerufen (z.B. "set <WLED> effect 10")? Denn nur so entsteht der richtige Inhalt der Dropdowns.

Gruß
Dan

Ich hatte es mehrmals, wie beschrieben, mit

Code Auswählen Erweitern

set  MQTT2_WLED_garage effect 1; set  MQTT2_WLED_garage palette 1
gemacht, kein Erfolg.
Nun hab ich es mit

Code Auswählen Erweitern

set  MQTT2_WLED_garage effect 11
alleine probiert, siehe da, alles da!

Warum, keine Ahnung, aber ist so.

Sorry für meine Nachfragen und danke für die Hilfe!
Robert

[TomLee]

Kommt bei irgendwem der hier mitliest und WLED auch nutzt in dem Info-Objekt des JSON die IP mit ?
Ich nutze noch 0.12.0, da kommt zwar lip mit ist aber leer

Code Auswählen Erweitern

"lip":"" und ip gibts gar nicht.

Bei irgendeiner Version muss sie ja mal dabei gewesen sein:
Info-Objekt

No value may be changed by means of this API.

[DeeSPe]

Ich nutze noch 0.12.0, da kommt zwar lip mit ist aber leer

Code Auswählen Erweitern

"lip":"" und ip gibts gar nicht.

Ist bei mir auch so.

Gruß
Dan

[TomLee]

Was für eine Version nutzt du denn ?

Es steht zwar nix dazu in der Doku, aber es wär auch toll (ich brauchs nicht) die Presets mit dem vergebenen Namen setzen zu können, in dem state-Objekt kommt zwar die ID mit, aber nicht der Name, - Wunschdenken on - in der neuesten Version vlt. doch aber noch nicht dokumentiert ?