[neuer Adapter] smartgarden adapter for Gardena smart system

jpgorganizer

@StrathCole und steht deshalb auf ????

StrathCole

@jpgorganizer sagte in [neuer Adapter] smartgarden adapter for Gardena smart system:

@StrathCole und steht deshalb auf ????

true

StrathCole

@jpgorganizer Ich glaube hier:
https://github.com/jpgorganizer/ioBroker.smartgarden/blob/master/lib/api.js#L1066
fehlt die Prüfung:
if(!err && state) ...

StrathCole

@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

jpgorganizer

@StrathCole

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

StrathCole

@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.

StrathCole

@jpgorganizer sagte in [neuer Adapter] smartgarden adapter for Gardena smart system:

@StrathCole

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.testVariable existiert bei mir. Es ist allerdings "leer", hat also keinen Wert. Vermutlich ist daher "state" als Variable null.

Du legst das Objekt ja mit adapter.setObjectAsync( an. Ich weiß nicht, ob das vielleicht nur das Objekt anlegt, aber keinen State wie es createState oder setState machen würde.

StrathCole

@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.

StrathCole

@jpgorganizer
Noch eine Sache (leider): Der State lastErrorCode_value existiert 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.

jpgorganizer

@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

StrathCole

@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.

jpgorganizer

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

dslraser

@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_i mit STOP_UNTIL_NEXT_TASK funktioniert bei mir nicht.

StrathCole

@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.

StrathCole

Habe noch ein Issue angelegt für die Objekttypen etc.

dslraser

@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.👍