Comments (4)
Das hatte ich doch gerade auf Anfrage umgestellt. Ecovacs erwartet z.b. eine 1000 für den silent Mode.
Und der Adapter sollte eine 1 für silent liefern.
Wird für das waterlevel im Adapter nochmal gemappt von den Strings auf die numerischen Werte?
Irgendwie stecke ich da wohl noch nicht so drin... Ich implementiere jetzt nur noch nach Abnahme der Spezifikation von dir ;-)
Erst Dokumentation, dann Implementierung. Sollte ich eigentlich von der Arbeit kennen, da Belange ich das auch immer von den Entwicklern :-D
from ecovacs-deebot.js.
Das hatte ich doch gerade auf Anfrage umgestellt. Ecovacs erwartet z.b. eine 1000 für den silent Mode.
Und der Adapter sollte eine 1 für silent liefern.
Wird für das waterlevel im Adapter nochmal gemappt von den Strings auf die numerischen Werte?
Nicht explizit. Im Object des Adapters ist das sogar als numerischer Wert definiert.
Das wird wohl irgendwie indirekt passieren.
Irgendwie stecke ich da wohl noch nicht so drin... Ich implementiere jetzt nur noch nach Abnahme der Spezifikation von dir ;-)
Deine Implementierung für die Saugkraft und die Anpassung war doch soweit ok 👍
Da gab es bisher auch noch gar keine fertige Implemetierung für die nicht-950 Geräte.
Mir war nur aufgefallen, dass es bei den Konstanten für
WATER_LEVEL_TO_ECOVACS
und
WATER_LEVEL_FROM_ECOVACS
einmal als String und einmal als numerischer Wert abgebildet ist.
Und dass wir schauen sollten, dass wir es soweit wie eben möglich einheitlich gestalten sollten.
Ich hoffe, dass Du das nicht als Kritik verstanden hast :)
Ich habe die Issues mal angelegt, damit wir das alles besser im Blick behalten können und natürlich auch Bugs geordneter angehen und nachvollziehen können.
Erst Dokumentation, dann Implementierung. Sollte ich eigentlich von der Arbeit kennen, da Belange ich das auch immer von den Entwicklern :-D
Ich implementiere jetzt nur noch nach Abnahme der Spezifikation von dir ;-)
Das können wir gerne so machen. Ich bin mir aber nicht sicher, ob wir das strikt einhalten können oder müssen :)
from ecovacs-deebot.js.
Hm, ich verstehe das immer noch nicht so richtig. Die Ecovacs API erwartet doch numerische Werte von 1-4 beim waterLevel, oder?
Der iobroker Adapter schickt bei mir folgenden Befehl an die API lt Log
set water level: 4
Was machen dann die Strings? Der vacBotCommand ersetzt ja nur, wenn der Lookup auch erfolgreich ist. Ich würde also denken, dass der einfach 1zu1 den Wert durchreicht und die Übersetzung eigentlich raus kann (außer man nutzt direkt die Library und möchte lieber Strings übergeben)?
class SetWaterLevel extends VacBotCommand_950type {
constructor(level) {
if (constants_type.WATER_LEVEL_TO_ECOVACS.hasOwnProperty(level)) {
level = constants_type.WATER_LEVEL_TO_ECOVACS[level];
}
super('setWaterInfo', {
'amount': level
});
}
}
from ecovacs-deebot.js.
Was machen dann die Strings? Der vacBotCommand ersetzt ja nur, wenn der Lookup auch erfolgreich ist. Ich würde also denken, dass der einfach 1zu1 den Wert durchreicht und die Übersetzung eigentlich raus kann (außer man nutzt direkt die Library und möchte lieber Strings übergeben)?
Ja, Du hast recht! 👍
Ich schmeiße das gerade mal komplett raus :)
Done: 40f3cb1
from ecovacs-deebot.js.
Related Issues (20)
- Implementation of the ability to also use the yeedi API HOT 16
- Crash while trying to read undefined data HOT 9
- Are there anyway to connect to the Ecovacs Robot without login? HOT 1
- No "ready" event received HOT 2
- Waterbox info HOT 4
- CleanReport and CleanSum HOT 12
- Move Left Right MoveTurnAround ? HOT 16
- ioBroker Warn Message HOT 1
- GetCleanLog serving old logs HOT 10
- Custom logging function HOT 1
- vacbot.disconnect(); HOT 27
- AirFreshening / aromamode not working HOT 20
- Request failed with status code 502/504 HOT 3
- No error event fired for "Charging dock not found" HOT 7
- Shortcut Command to remove current Map or Restore Map Backup ??? HOT 1
- Map get lost working on backup image HOT 1
- GB/WW api endpoint not found HOT 1
- Retrieve values for air quality returns code 500 HOT 10
- Some feedback on testing with the X2 Omni. HOT 4
- is it support R96? HOT 3
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from ecovacs-deebot.js.