[neuer Adapter] smartgarden adapter for Gardena smart system
-
@jpgorganizer Ich glaube hier:
https://github.com/jpgorganizer/ioBroker.smartgarden/blob/master/lib/api.js#L1066
fehlt die Prüfung:
if(!err && state) ... -
@jpgorganizer Die Prüfung auf Error und obj/state fehlt auch an vielen anderen Stellen, an denen getState verwendet wird. Dadurch crasht der Adapter, sobald einer der States nicht existiert/gelesen werden kann, beispielweise hier: https://github.com/jpgorganizer/ioBroker.smartgarden/blob/master/lib/api.js#L188
-
du hast zwar recht, dass die Prüfung fehlt, aber die Frage ist eine ganz andere, nämlich warum sollte der State nicht da sein.
Grüße
jpgorganizer -
@jpgorganizer
Sorry, wenn ich dich gerade zuspamme hier. Ich hätte für zukünftige Versionen noch zwei Wünsche für die MOWER devices:-
Voraussichtlicher nächster Start des Mähers
Das müsste ja prognostizierbar sein, indem man misst, wie schnell die Batterieladung voranschreitet, solange er auf OK_CHARGING steht. Dann könnte man die Restladedauer bis 100% schätzen. Natürlich funktioniert das nur, solange das Mähen laut Timer gerade aktiv ist und nicht in den Ruhephasen. Da liefert die API leider keinen Wert der Schedules. -
Voraussichtliche Rest-Mähdauer
Das müsste auch wieder anhand der Restbatterieladung ermittelbar sein, indem man historisch misst und speichert, bei welcher Restladung der Mäher in die Station fährt und wie lange es bis zu dieser Restladung dauert.
Ist natürlich für den Betrieb nicht essentiell, aber wäre schon cool
Danke auf jeden Fall schon mal für die Arbeit bis hierher. Wenn du möchtest, dass ich gewisse Dinge selbst einbaue und als Pull Request im Git mache, gib mir einfach Bescheid.
-
-
@jpgorganizer sagte in [neuer Adapter] smartgarden adapter for Gardena smart system:
du hast zwar recht, dass die Prüfung fehlt, aber die Frage ist eine ganz andere, nämlich warum sollte der State nicht da sein.
Das Object
smartgarden.0.testVariableexistiert bei mir. Es ist allerdings "leer", hat also keinen Wert. Vermutlich ist daher "state" als Variablenull.Du legst das Objekt ja mit
adapter.setObjectAsync(an. Ich weiß nicht, ob das vielleicht nur das Objekt anlegt, aber keinen State wie escreateStateodersetStatemachen würde. -
@jpgorganizer Habe es gerade getestet, indem ich dem State via Admin einfach einen Wert zugewiesen habe. Schon ist der Fehler weg. Problem scheint wirklich zu sein, dass du nur ein Objekt anlegst, aber keinen State.
-
@jpgorganizer
Noch eine Sache (leider): Der StatelastErrorCode_valueexistiert scheinbar nur, wenn er das erste Mal von der API gesendet wurde. Vielleicht kannst du den vorab schon anlegen, sonst kann ich ihn nicht korrekt in meiner Vis einbinden. -
@StrathCole sagte in [neuer Adapter] smartgarden adapter for Gardena smart system:
Das Object smartgarden.0.testVariable existiert bei mir. Es ist allerdings "leer", hat also keinen Wert. Vermutlich ist daher "state" als Variable null.
ja, genau das wird es sein, und deshalb wollte ich ja auch wissen auf welcher Wert in der Config steht ....
Die anderen Vorschläge muss ich mir mal ansehen. Grundsätzlich finde ich es sehr gut, wenn Ideen benannt werden. Damit kann der Adapter nur "besser" werden.
Grüße
jpgorganizer -
@jpgorganizer sagte in [neuer Adapter] smartgarden adapter for Gardena smart system:
Die anderen Vorschläge muss ich mir mal ansehen. Grundsätzlich finde ich es sehr gut, wenn Ideen benannt werden. Damit kann der Adapter nur "besser" werden.
Wie gesagt, wenn ich etwas als Pull Request selbst umsetzen soll, gib Bescheid. Habe inzwischen selbst ein paar Adapter gebaut.
-
Version 0.3.0 ist auf github
@dslraser
das Problem mit readonly States sollte nun gefixed sein. Ich kann es bei mir nicht testen, da das Problem bei mir nicht auftritt. Bitte testen@StrathCole
Problem mit testVariable sollte gefixed sein. Bitte testen@StrathCole
deine Ideen mit den Vorhersagen zur Laufzeit etc. des Mähers und auch für die Pre-Definition von States bitte je einen Request in github aufmachen. Dann geht es uns nicht verloren. Insbesondere die Vorhersage gefällt mir. Ich denke am WoEnde kann ich da was machen. Morgen/Übermorgen muss ich erst meine Bewässerungsanlage fertigstellen und kann erst am Abend wieder was machen.Grüße
jpgorganizer -
@jpgorganizer sagte in [neuer Adapter] smartgarden adapter for Gardena smart system:
das Problem mit readonly States sollte nun gefixed sein. Ich kann es bei mir nicht testen, da das Problem bei mir nicht auftritt. Bitte testen
Version 0.3.0
funktioniert jetzt bei mir auch ohne Expertenmodus.einzelne Ventile stoppen funktioniert, aber der DP
.stop_all_valves_imitSTOP_UNTIL_NEXT_TASKfunktioniert bei mir nicht. -
@jpgorganizer sagte in [neuer Adapter] smartgarden adapter for Gardena smart system:
bitte je einen Request in github aufmachen
Danke, hab ich gemacht. Den Test mit der neuen Version werde ich bei Gelegenheit machen, vielleicht warte ich aber noch bis nach dem Wochenende
Falls es bis dahin schon wieder ne neue Version gibt. -
Habe noch ein Issue angelegt für die Objekttypen etc.
-
@StrathCole sagte in [neuer Adapter] smartgarden adapter for Gardena smart system:
Habe noch ein Issue angelegt für die Objekttypen etc.
Das wäre wirklich Sinnvoll, da ich im Moment in meinem Blockly für die Bewässerung immer erst umformatieren muß in Zahlen. Ich wollte das gestern schon schreiben, aber das hast Du ja nun schon getan.
-
siehe hier
https://forum.iobroker.net/post/408094 -
ich habe eben einen Sensor in die Erde gesteckt. Eigentlich kann der diese Daten senden.
Im Adapter kommt aber nix von diesen Werten an. Vielleicht kannst Du Dir das auch mal ansehen.
-
@dslraser sagte in [neuer Adapter] smartgarden adapter for Gardena smart system:
Im Adapter kommt aber nix von diesen Werten an.
entweder hat der Gardena Service nichts gesendet oder es kam zu einem Fehler im Adapter.
steht im Log was interessantes, was evtl einen Hinweis geben könnte?
Gruss
jpgorganizer -
@jpgorganizer
welchen log level soll ich dazu einstellen ?
-
-
@jpgorganizer
ich habe den Adapter mal angehalten und den Sensor aus den Objekten gelöscht, danach den Adapter gestartet. Zuerst waren keine Datenpunkte da, dann habe ich in der App auf die einzelnen Werte geklickt, dann werden die aktualisiert. Dann kamen weitere Datenpunkte, aber keiner dieser Sensorwerte. Also keine Temperatur/Lichtintensität/Bodenfeuchte.
Die Objekte sehen genau so aus wie oben gezeigt.
Im Log sind auch keine dieser Werte zu finden.
