Und nächste Problem mpd mehrere Instancen

[lexxmm]

Leider habe ich schon das nächste Problem beim meinem Wechsel auf aktuelle sh.ng. Und bin auch leider wieder nicht im Forum fündig geworden..

mpd: class_name: MPD class_path: plugins.mpd instance: bathroom host: 192.168.177.164 port: 6601 # cycle: 2 mpd: class_name: MPD class_path: plugins.mpd instance: kitchen host: 192.168.177.164 port: 6602 # cycle: 2

sollte klappen. Tut es aber nicht. Im Log gibt es nur immer die letzte instance.

2018-10-21 20:41:36 DEBUG Main Plugins, section: mpd
2018-10-21 20:41:36 DEBUG Main Plugins __init__: pluginname = 'mpd', classpath 'plugins.mpd'
2018-10-21 20:41:36 INFO Main Loading '/usr/local/smarthome/plugins/mpd/plugin.yaml' to 'OrderedDict'
2018-10-21 20:41:36 INFO Main plugin 'mpd': Metadata paramlist = '['host', 'port', 'cycle']'
2018-10-21 20:41:36 DEBUG Main plugin 'mpd': definition = 'host' list of listparam = >['foo']<, listlen=0
2018-10-21 20:41:36 DEBUG Main plugin 'mpd': definition = 'port' list of listparam = >['foo']<, listlen=0
2018-10-21 20:41:36 DEBUG Main plugin 'mpd': definition = 'cycle' list of listparam = >['foo']<, listlen=0
2018-10-21 20:41:36 INFO Main plugin 'mpd': has no item definitions in metadata
2018-10-21 20:41:36 DEBUG Main PluginWrapper __init__: Section mpd, classname MPD, classpath plugins.mpd
2018-10-21 20:41:36 INFO Main Loading '/usr/local/smarthome/plugins/mpd/locale.yaml' to 'dict'
2018-10-21 20:41:36 DEBUG Main set plugin mpd instance to wz
2018-10-21 20:41:36 DEBUG Main plugin 'mpd': Found 'host' with value '127.0.0.1' in /etc/plugin.yaml
2018-10-21 20:41:36 DEBUG Main plugin 'mpd': Found 'port' with value '6603' in /etc/plugin.yaml
2018-10-21 20:41:36 INFO Main plugin 'mpd': value not found in plugin configuration file for parameter 'cycle' -> using default value '2' instead
2018-10-21 20:41:36 DEBUG Main Plugins: Loaded plugin 'mpd' (class 'MPD') v1.4.1: Music Player Deamon (MPD) Unterstützung
2018-10-21 20:41:36 DEBUG Main set plugin mpd instance to wz
2018-10-21 20:41:36 INFO Main Initialized plugin 'mpd' instance 'wz' from from section 'mpd'
2018-10-21 20:41:37 DEBUG Main Starting plugin 'mpd', instance 'wz'
2018-10-21 20:41:37 DEBUG mpd Scheduler: Name changed by adding plugin instance name to: MPD_wz
2018-10-21 20:41:37 DEBUG mpd MPD_wz next time: 2018-10-21 20:41:52+02:00

Jemand eine Idee? Setzen eigentlich echt so wenig Leute das mpd plugin ein oder hab nur ich Probleme? Im Forum gibt es ja kaum was zu mpd ?

Inhalt meiner plugin

mpd:
    class_name: MPD
    class_path: plugins.mpd
    instance: diele
    host: 127.0.0.1
    port: 6601

mpd:
    class_name: MPD
    class_path: plugins.mpd
    instance: wz
    host: 127.0.0.1
    port: 6603

Gruss, Micha

[bmx]

Benenne doch mal den zweiten mpd: um in mpd-wz

[lexxmm]

Das war es! Danke! Da sollte die Doku. angepasst werden

[Onkelandy]

Zur generellen mpd Frage.. ich bin ziemlich rasch auf Squeezebox umgestiegen.. http://www.hifi-forum.de/viewthread-212-1713.html
Macht aber natürlich nicht in allen Situationen Sinn.

[felix86]

Moin Zusammen,
noch kurz der Hinweis:

Problem parsing 'mpd-bad' invalid character. Valid characters are: abcdefghijklmnopqrstuvwxyzABCDEFGHIJKLMNOPQRSTUVWXYZ0123456789_abcdefghijklmnopqrstuvwxyzABCDEFGHIJKLMNOPQRSTUVWXYZ0123456789_@* 

mpd_bad geht, mpd-bad geht nicht.
Die Doku muss angepasst werden, da genau dort das falsche Beispiel mit den zwei Instanzen steht.

Gruß Felix

[Msinn]

Das war es! Danke! Da sollte die Doku. angepasst werden

Wo sollen wir denn überall erwähnen, dass keys in yaml Dateien eindeutig sein müssen?

[Msinn]

Die Doku muss angepasst werden, da genau dort das falsche Beispiel mit den zwei Instanzen steht.

Bitte die Dokumentation zum yaml Datei Format berücksichtigen!

[felix86]

Ja natürlich sollte man das.
Nur wenn man die Doku zu diesem Plugin sich anschaut, dann ist dort genau ein falsches Beispiel gebracht. Wenn man es also genau so macht, dann funktioniert es nicht. Und da es ein Beispiel auf der offiziellen Seite ist, denkt man im ersten Moment nicht daran die yaml Doku anzuschauen um zu prüfen ob dort vielleicht ein Fehler gemacht wurde.
Und da das Beispiel nunmal falsch ist, sollte man es korrigieren:
https://www.smarthomeng.de/user/plugins/mpd/README.html
Ich habe auf die schnelle nicht gesehen, dass ich es selber könnte, sonst hätte ich es getan.

[Onkelandy]

Wo er Recht hat, hat er Recht habs mal schnell als PR rein gestellt: https://github.com/smarthomeNG/plugins/pull/197