Discovery Adapter neugestaltung

[thewhobox]

Hallo liebe Community,

ich habe mich in den letzen Tagen mit dem Discovery Adapter beschäftigt um mal auszuprobieren was ich machen muss damit auch mein Adapter damit funktioniert und den Server erkennt.

Dabei sind mir aber viele Sachen aufgefallen:

Spoiler

• per Upnp erkannte Geräte werden pro IP nur einmal gelistet (Beispiel LG-TV schickt mehrere Antworten, da zweiter Bildschirm, webos, rootdevice, etc. Aber nur das erste wird gespeichert).
• Das hat dann zur Folge, dass viele Adapter für jede IP einen eigenen UPNP scan durchführen. (Was den Adapter nur unnötig langsam macht.
• Manche Adapter haben als Typ ['ip', 'upnp'] wobei hier upnp irrelevant, da es vorher keinen Typ upnp gab.
• Ebenso erkannte Geräte über mDNS werden als Typ IP gelistet und müssen somit ebenfalls oft bei jeder IP ausgeführt werden.
• Es gibt Adapter die nur einmal ausgeführt werden müssen, da sie einen anderen Scan durchführen. Da sie als Typ aber IP haben machen Sie den Scan für jede IP, was ebenfalls unnötig ist.

Ich habe mal ein Fork des Discovery Adapters gemacht und damit ein bisschen rumgespielt.
Folgende änderungen habe ich gemacht:

Spoiler

• Es werden alle Upnp Geräte aufgelistet (mehrere pro IP möglich)
• Adapter die eigene Ssdp suchen machen wurden angepasst mit dem Typ upnp (werden also nur einmal pro upnp ausgeführt und machen keine eigenen für alle IPs).
• Der Code von vielen Adapter konnte damit ebenfalls erheblich reduziert werden.
• Dummy Gerät (0.0.0.0) mit Typ "once" hinzugefügt für Adapter die nur einmal ausgeführt werden müssen.

So weit ich es erkennen kann läuft mitlerweile alles ohne Probleme, aber ich kann einige Adapter nicht testen.

Was suche ich also?
Deswegen suche ich hier gerne User die das testen können oder auch gerne nur eine Rückmeldung ob so ein System überhaupt Sinn macht, bzw Verbesserungsvorschläge.
Wenn ihr Lust habt mitzuhelfen schreibt mir einfach ne Nachricht wo drin steht welchen Adapter Ihr testen könnt.

Folgende Adapter müssen noch getestet werden:

• Bosesoundtuch
• Broadlink
• chromecast
• Daikin
• Deconz
• Denon
• eBus
• eKey
• emby
• eNet
• G-Homa
• Harmony
• hp-ilo
• HUE
• KNX
• Loxone
• MaxCube
• miHome
• Samsung
• Upnp
• Wifilight
• Yeelight-2
• hf-lpb100
• wifi-mi-light

[crunchip]

Discover.txt

[crunchip]

@crunchip discover1.txt

[crunchip]

@crunchip discover2.txt

[crunchip]

@crunchip discover 1.3.1.txt

[crunchip]

@crunchip version1.3.2.txt

[crunchip]

@crunchip V1.3.3.txt

[crunchip]

@crunchip 1.3.4.txt

[sigi234]

Hallo, kann diesen testen......
UPNP

[thewhobox]

@sigi234 Gerne.
Mich würde es auch interessieren wie viel Zeit die Suche benötigt bevor und nachdem du meine Version installiert hast

[sigi234]

@thewhobox sagte in Discovery Adapter neugestaltung:

@sigi234 Gerne.
Mich würde es auch interessieren wie viel Zeit die Suche benötigt bevor und nachdem du meine Version installiert hast

Eigentlich ein bisschen länger

[thewhobox]

@sigi234 Echt?
Ich habe gerade mal einen Vergleich gemacht.
Den originalen Adapter installiert und Geräte gesucht (Upnp, Ping) -> 42 Sekunden
Adapter aus meinen Repo installiert (Upnp, Ping, UPD) -> 32 Sekunden

Und mein Netzwerk ist mit 9 IPs momentan noch recht klein.