Test Adapter SmartControl 0.3.x-0.6.x Latest
-
@crunchip sagte in Test Adapter SmartControl 0.3.x GitHub (ab 12.09.20):
@Mic was mir noch aufgefallen ist bzw fragen wollte
da ich in der Vergangenheit, mir schon bestimmte Sachen anlegen wollte ( was technisch aber noch nicht vorhanden ist)
schmeisst mir der Adapter aber dann error wegen falsch konfiguriert, obwohl der Haken deaktiviert ist,
also greift scheinbar dieses nicht, bzw dann wohl nur, wenn alle Pflichtfelder ausgefüllt sind
Derzeit wird tatsächlich auch geprüft, wenn Zeile nicht aktiviert wird, Grund war die Datenpunkte unter
smartcontrol.0.options- denn hier können einzelne Zeilen z.B. über VIS dann aktiviert werden.
Ich muss da die Beschreibung anpassen.
Alternativ könnte ich auch eine Option einbauen, dass grundsätzlich deaktivierte Zeilen nicht geprüft werden? -
@dslraser sagte in Test Adapter SmartControl 0.3.x GitHub (ab 12.09.20):
Das hier machen alle (ADAPTER_REQUESTED_TERMINATION) (ausser smartcontrol)
Echt seltsam, so sieht es bei mir aus:
smartcontrol.1 2020-09-13 18:20:46.479 info (23536) Subscribing to all target devices and trigger states. 5 trigger schedules activated... smartcontrol.1 2020-09-13 18:20:45.502 info (23536) Adapter admin configuration successfully validated... smartcontrol.1 2020-09-13 18:20:45.043 info (23536) starting. Version 0.3.1 in /opt/iobroker/node_modules/iobroker.smartcontrol, node: v12.16.1, js-controller: 3.1.6 host.ctioBroker 2020-09-13 18:20:43.976 info instance system.adapter.smartcontrol.1 started with pid 23536 host.ctioBroker 2020-09-13 18:20:41.484 info instance system.adapter.smartcontrol.1 terminated with code 11 (ADAPTER_REQUESTED_TERMINATION) smartcontrol.1 2020-09-13 18:20:40.680 info (23521) Terminated (ADAPTER_REQUESTED_TERMINATION): Without reason smartcontrol.1 2020-09-13 18:20:40.679 info (23521) terminating smartcontrol.1 2020-09-13 18:20:40.678 info (23521) Stopping adapter instance successfully proceeded... smartcontrol.1 2020-09-13 18:20:40.677 info (23521) (6) schedules cleared... smartcontrol.1 2020-09-13 18:20:40.675 info (23521) (0) timers were active and have been cleared... smartcontrol.1 2020-09-13 18:20:40.670 info (23521) Got terminate signal TERMINATE_YOURSELF host.ctioBroker 2020-09-13 18:20:40.666 info stopInstance system.adapter.smartcontrol.1 send kill signal host.ctioBroker 2020-09-13 18:20:40.659 info stopInstance system.adapter.smartcontrol.1 (force=false, process=true)Plattform: linux (Proxmox Debian CT)
Node.js: v12.16.1
js-controller 3.1.6 -
@Mic sagte in Test Adapter SmartControl 0.3.x GitHub (ab 12.09.20):
dann der Datenpunkt gelöscht wird, und was danach das Zielgerät auslöst.
ok, ich hatte ja keinen DP gelöscht, sondern nur Umbenannt
-
ausgelöst wurde in meinem Fall nicht, daher, Fallbeispiel (ich bastel vormittags an irgendwelchen DP's, welche aber erst zur abendlichen Stunde oder gar erst bei erreichen eines bestimmten Wertes(Temperatur), ausgelöst werden)
den "Fehler" bekomme ich dann aber erst mit, wenn -
A) ich in die Instanz gehe/neu starte( habe ich in den Fall aber nicht, weil ich gar nicht auf die Idee kam) da ich den DP nicht gelöscht, sondern nur umbenannt habe
-
B) das Auslösen eintritt
-
-
Update auf 0.3.2
- (Mic-M) New feature: In the adapter configuration, tab 'Further Options' > 'Input Validation', you can now select if deactivated configuration table rows should be validated as well.
- (Mic-M) Fix adapter-check.iobroker.in error [E144] "common.installedFrom field found in io-package.json. Must be removed."
- (Mic-M) Fix for adapter unload: check schedule variable for undefined.
- (Mic-M) Fixed debug log line
-
@Mic sagte in Test Adapter SmartControl 0.3.x GitHub (ab 12.09.20):
@crunchip sagte in Test Adapter SmartControl 0.3.x GitHub (ab 12.09.20):
@Mic was mir noch aufgefallen ist bzw fragen wollte
da ich in der Vergangenheit, mir schon bestimmte Sachen anlegen wollte ( was technisch aber noch nicht vorhanden ist)
schmeisst mir der Adapter aber dann error wegen falsch konfiguriert, obwohl der Haken deaktiviert ist,
also greift scheinbar dieses nicht, bzw dann wohl nur, wenn alle Pflichtfelder ausgefüllt sind
Derzeit wird tatsächlich auch geprüft, wenn Zeile nicht aktiviert wird, Grund war die Datenpunkte unter
smartcontrol.0.options- denn hier können einzelne Zeilen z.B. über VIS dann aktiviert werden.
Ich muss da die Beschreibung anpassen.
Alternativ könnte ich auch eine Option einbauen, dass grundsätzlich deaktivierte Zeilen nicht geprüft werden?Mit 0.3.2 nun neue Option (per Default deaktiviert):
-
@crunchip sagte in Test Adapter SmartControl 0.3.x GitHub (ab 12.09.20):
@Mic sagte in Test Adapter SmartControl 0.3.x GitHub (ab 12.09.20):
dann der Datenpunkt gelöscht wird, und was danach das Zielgerät auslöst.
ok, ich hatte ja keinen DP gelöscht, sondern nur Umbenannt
-
ausgelöst wurde in meinem Fall nicht, daher, Fallbeispiel (ich bastel vormittags an irgendwelchen DP's, welche aber erst zur abendlichen Stunde oder gar erst bei erreichen eines bestimmten Wertes(Temperatur), ausgelöst werden)
den "Fehler" bekomme ich dann aber erst mit, wenn -
A) ich in die Instanz gehe/neu starte( habe ich in den Fall aber nicht, weil ich gar nicht auf die Idee kam) da ich den DP nicht gelöscht, sondern nur umbenannt habe
-
B) das Auslösen eintritt
Ob umbenannt oder gelöscht ist hier egal, denn der Quellcode findet so einfach den Datenpunkt nicht mehr. Das wird dir auch im JS-Adapter so passieren usw.
Ich müsste ja ansonsten eine z.B. minütliche Prüfung einbauen, ob alle Datenpunkte noch existieren, aber das ist aus Performance-Sicht nicht zielführend.
Oder was hättest du für einen Vorschlag zur Umsetzung? -
-
@Mic ist mir gleich, wie es der Masse besser passt, hätte das anfangs nur ganz praktisch gefunden, wenn man der Reihe nach seine ganzen DP's schon mal anlegen kann und dann nach und nach aktivieren. Gerade weil es auch mit dem Benennen/Einteilen der Datenpunkte (zur besseren Übersicht) anfangs blöd war
-
@crunchip sagte in Test Adapter SmartControl 0.3.x GitHub (ab 12.09.20):
@Mic ist mir gleich, wie es der Masse besser passt, hätte das anfangs nur ganz praktisch gefunden, wenn man der Reihe nach seine ganzen DP's schon mal anlegen kann und dann nach und nach aktivieren. Gerade weil es auch mit dem Benennen/Einteilen der Datenpunkte (zur besseren Übersicht) anfangs blöd war
Hast Recht, aber ist doch schon umgesetzt
Siehe Version 0.3.2 -
@Mic, ich finde es nahezu unfassbar mit welcher Hingabe und Geschwindigkeit Du hier den Adapter nach vorne bringst. RESPEKT
Bist jetzt konnte ich tatsächlich alles was ich so an Blockly´s hatte in den Adapter migrieren, echt toll.Ich würde mir wünschen, wenn der Adapter so weit fertig ist, das Du Dir dein ein oder anderen Adapter vornimmst, "forkst" und diesen dann ebenfalls so usernah weiterentwickelst
Danke
-
@Mic sagte in Test Adapter SmartControl 0.3.x GitHub (ab 12.09.20):
Das wird dir auch im JS-Adapter so passieren
eben, da wird es sofort angemeckert
@Mic sagte in Test Adapter SmartControl 0.3.x GitHub (ab 12.09.20):
aber das ist aus Performance-Sicht nicht zielführend.
das war mir schon bewusst, daher
hab ich da auch keine Idee/Vorschlag, bleibt einfach nur ....dran denken, wo der entsprechende DP überall eingebaut ist@Mic sagte in Test Adapter SmartControl 0.3.x GitHub (ab 12.09.20):
Hast Recht, aber ist doch schon umgesetzt
bist a Fuchs
-
@Mic
ich habe meine Instanz nochmal gelöscht und dann von GitHub die 0.3.2 installiert.Jetzt ist das Log okay
smartcontrol.0 2020-09-13 19:53:48.896 info (7452) Subscribing to all target devices and trigger states. 0 trigger schedules activated... smartcontrol.0 2020-09-13 19:53:48.498 info (7452) Adapter admin configuration successfully validated... smartcontrol.0 2020-09-13 19:53:46.466 info (7452) starting. Version 0.3.2 in /opt/iobroker/node_modules/iobroker.smartcontrol, node: v12.18.3, js-controller: 3.1.6 host.iobroker 2020-09-13 19:53:44.821 info instance system.adapter.smartcontrol.0 started with pid 7452 host.iobroker 2020-09-13 19:53:42.282 info instance system.adapter.smartcontrol.0 terminated with code 11 (ADAPTER_REQUESTED_TERMINATION) smartcontrol.0 2020-09-13 19:53:41.759 info (7437) Terminated (ADAPTER_REQUESTED_TERMINATION): Without reason smartcontrol.0 2020-09-13 19:53:41.758 info (7437) terminating smartcontrol.0 2020-09-13 19:53:41.757 info (7437) Stopping adapter instance successfully proceeded... smartcontrol.0 2020-09-13 19:53:41.756 info (7437) (1) schedules cleared... smartcontrol.0 2020-09-13 19:53:41.755 info (7437) (0) timers were active and have been cleared... smartcontrol.0 2020-09-13 19:53:41.753 info (7437) Got terminate signal TERMINATE_YOURSELF host.iobroker 2020-09-13 19:53:41.749 info stopInstance system.adapter.smartcontrol.0 send kill signal -
@Mic sagte in Test Adapter SmartControl 0.3.x GitHub (ab 12.09.20):
Mit 0.3.2 nun neue Option (per Default deaktiviert):
jetzt bin ich aber grad verwirrt, installiert, anschliessend im log
smartcontrol.0 2020-09-13 19:56:36.124 error (22323) [_asyncOnReady()] – 1 error(s) occurred while processing state generation of options. smartcontrol.0 2020-09-13 19:56:36.122 error (22323) [tableConditions] We were not able to generate a valid state path. This is what was determined to be not valid: [smartcontrol.0.options.Conditions.Is Front Door Locked?.active].smartcontrol.0.options.Conditions.Is Front Door Lockedgehört ja zu deinen voreingestellten Beispielen, hab die Beispiele noch nicht gelöscht und sind seit beginn ja deaktiviert -
@crunchip Kam bei mir auch mit der neuen Version.
Ich habe den gelöscht. Aber war doch etwas verwundert....
-
@MichMein sagte in Test Adapter SmartControl 0.3.x GitHub (ab 12.09.20):
@Mic, ich finde es nahezu unfassbar mit welcher Hingabe und Geschwindigkeit Du hier den Adapter nach vorne bringst. RESPEKT
Bist jetzt konnte ich tatsächlich alles was ich so an Blockly´s hatte in den Adapter migrieren, echt toll.Ich würde mir wünschen, wenn der Adapter so weit fertig ist, das Du Dir dein ein oder anderen Adapter vornimmst, "forkst" und diesen dann ebenfalls so usernah weiterentwickelst
Danke
Vielen dank für dein tolles Feedback
-
@Mic said in Test Adapter SmartControl 0.3.x GitHub (ab 12.09.20):
Vielen dank für dein tolles Feedback
nu, wo @MichMein recht hat, hat er recht!
-
@crunchip sagte in Test Adapter SmartControl 0.3.x GitHub (ab 12.09.20):
Das wird dir auch im JS-Adapter so passieren
eben, da wird es sofort angemeckert
Nur als Nachtrag.
Auch der JS-Adapter wird hier nicht sofort erkennen, wenn du einen Datenpunkt umbenennst. Beispielconst datenpunkt = '0_userdata.0.example_state'; setTimeout(() => { log(`State-Wert: ${getState(datenpunkt).val}`) }, 60*1000);Der wird das erst anmeckern, sobald das setTimeout() ausgeführt wird.
-
@crunchip sagte in Test Adapter SmartControl 0.3.x GitHub (ab 12.09.20):
@Mic sagte in Test Adapter SmartControl 0.3.x GitHub (ab 12.09.20):
Mit 0.3.2 nun neue Option (per Default deaktiviert):
jetzt bin ich aber grad verwirrt, installiert, anschliessend im log
smartcontrol.0 2020-09-13 19:56:36.124 error (22323) [_asyncOnReady()] – 1 error(s) occurred while processing state generation of options. smartcontrol.0 2020-09-13 19:56:36.122 error (22323) [tableConditions] We were not able to generate a valid state path. This is what was determined to be not valid: [smartcontrol.0.options.Conditions.Is Front Door Locked?.active].smartcontrol.0.options.Conditions.Is Front Door Lockedgehört ja zu deinen voreingestellten Beispielen, hab die Beispiele noch nicht gelöscht und sind seit beginn ja deaktiviert@Chaot sagte in Test Adapter SmartControl 0.3.x GitHub (ab 12.09.20):
@crunchip Kam bei mir auch mit der neuen Version.
Ich habe den gelöscht. Aber war doch etwas verwundert....Sind noch "alte / legacy" Beispiele. Hab ich mittlerweile angepasst, aber das sieht man wohl nur, wenn man den Adapter löscht und neu installiert
.
Siehe hier aus der io-package.json:
Wahrscheinlich liegt es an dem Sonderzeichen "?" (Fragezeichen), ich hab beim Adapter Refactoring so einiges umgebaut und verbessert, und auch paar strengere Regeln eingebaut
Cool wäre evtl. noch eine "Auf Werkseinstellungen zurücksetzen" Funktion, falls das einfach zu implementieren ist
-
Auf die schnelle noch ein Issue (Feature Request) eingebaut:
Anfrage war von @hetti720.3.3
- (Mic-M) New feature for Motion sensors: the timer duration and brightness threshold can now be changed via states. These states are available under
smartcontrol.0.options.TriggerMotion.xxx.<duration|briThreshold>for each of your motion sensors. Please note that any change will cause an adapter restart (for initializing and clearing all existing timers to apply the new values). Issue #18
Jetzt kann man für die Bewegungsmelder die Werte für den Timer (Einschaltdauer) und Helligkeits-Grenze in Datenpunkten einstellen. Für jeden Bewegungsmelder werden hierfür die entsprechenden Datenpunkte erstellt:
Bitte testen
- (Mic-M) New feature for Motion sensors: the timer duration and brightness threshold can now be changed via states. These states are available under
-
@Mic ok, aber zumindest kamen bis jetzt die Error's in JS immer unmittelbar zeitnah, so das es gleich aufgefallen ist
und ich dann fragend da saß...
"mist, welches script läuft denn nun schon wieder nicht" -
@crunchip sagte in Test Adapter SmartControl 0.3.x GitHub (ab 12.09.20):
@Mic ok, aber zumindest kamen bis jetzt die Error's in JS immer unmittelbar zeitnah, so das es gleich aufgefallen ist
und ich dann fragend da saß... "mist, welches script läuft denn nun schon wieder nicht"Jo, und im JS.Adapter kannst halt auch einfach globales Suchen machen für bei Datenpunkt-Änderungen. Das geht über sämtliche Adapter und deren individueller Einstellungen natürlich nicht.
