iobroker.e3oncan 0.6.15 → 0.6.17

package/README.md

@@ -21,7 +21,7 @@ In parallel **reading of datapoints** (ReadByDid) is supported. Informations not
 
 **Writing of datapoints** via UDSonCAN (WriteByDid) is supported as well. By writing to datapoints it's possible to change setpoints, schedules and so on. It's even possible to add new schedules e.g. for domestic hot water circulation pump.
 
-Writing of data is triggered by storing the corresponding state with `Acknowledged` not checked (ack=false) - yes, it's that simple! The datapoint will be read again from device and stored in the state two seconds after writing. If state not get's acknowledged, please take a look to the logs.
+Writing of data is triggered by storing the corresponding state with `Acknowledged` not checked (ack=false) - yes, it's that simple! The datapoint will be read again from device and stored in the state 2.5 seconds after writing. If state not get's acknowledged, please take a look to the logs.
 
 Writing is restricted to a set of datapoints using a **white list**. The list is stored in the info section of each device, e.g. at `e3oncan.0.vitocal.info.udsDidsWritable`. You can add more datapoints by editing this state. Make sure, **not** to check `Acknowledged` when saving the state.
 
@@ -39,13 +39,13 @@ A python based implementation of a pure listening approach using MQTT messaging
 * Currently only Linux based systems are supported.
 * CAN adapter is up and visible in system, e.g. as "can0" (use ifconfig to check).
 * Refer to open3e project for further details
-* **Make sure, open3e is not running during initial setup!** This could cause communication errors in both applications.
+* **Make sure, no other UDSonCAN client (e.g. Open3Eclient.py) is running during initial setup!** This could cause communication errors in both applications.
 
 All services provided by this adapter are based on device list of your specific Viessmann E3 setup. Therefore you have to follow following steps for first setup:
 
 **Configuration**
 * When installation of adapter has finished a confuration dialog will show up to configure up to two CAN bus adapters (tab "CAN ADAPTER")
-* Edit name of adapter and check the "Connect to adapter" checkbox at least the external adapter
+* Edit name of adapter and check the "Connect to adapter" checkbox at least for the external adapter
 * When you're done, press "SAVE" button to apply the changes. This step is **mandatory**. The instance will restart and connect to the CAN adapter.
 * Go to tab "LIST OF UDS DEVICES" and do a scan for devices available on bus by pressing the scan button. **This will take a few seconds.** You may watch the activities in a 2nd browser tab by looking on the logging info of the adapter.
 * You may change the naming of the devices at 2nd column. Those names will be used to store all collected data in ioBoker's object tree. Again press "SAVE" button when you did your changes.
@@ -65,11 +65,11 @@ You may use datapoints informations on tab "LIST OF DATAPOINTS" for reference (o
 | ID | Data| Unit |
 | ------|:--- |------|
 | 0x250 | Active Power L1, L2, L3, Total |  W |
-| 0x252 | Reactive Power L1, L2, L3, Total | W |
+| 0x252 | Reactive Power L1, L2, L3, Total | VA |
 | 0x254 | Current, L1, L2, L3, cosPhi | A, - |
 | 0x256 | Voltage, L1, L2, L3, Frequency | V, Hz |
 | 0x258 | Cumulated Import, Export | kWh |
-| 0x25A | Total Active Power, Total Reactive Power | W |
+| 0x25A | Total Active Power, Total Reactive Power | W, VA |
 | 0x25C | Cumulated Import | kWh |
 
 # Hints and limitations
@@ -102,6 +102,14 @@ Yes, that is possible under certain conditions:
     Placeholder for the next version (at the beginning of the line):
     ### **WORK IN PROGRESS**
 -->
+### 0.6.17 (2024-01-29)
+* (MyHomeMyData) Added/removed datapoints to/from list of writable dids
+* (MyHomeMyData) Preparations for device specific list of writable dids
+
+### 0.6.16 (2024-01-27)
+* (MyHomeMyData) Improvements based on findings in review as of 2024-01-25
+* (MyHomeMyData) Checkbox for data collectiton on internal bus is now checked per default
+
 ### 0.6.15 (2024-01-23)
 * (MyHomeMyData) Fix for Utf8 codec for handling of special characters, e.g. umlauts
 

package/admin/jsonConfig.json

@@ -560,7 +560,7 @@
                             "tooltip": "Activate communication",
                             "filter": false,
                             "sort": false,
-                            "default": false
+                            "default": true
                         },
                         {   "type": "selectSendTo",
                             "title": "Select device",

package/io-package.json

@@ -1,8 +1,34 @@
 {
   "common": {
     "name": "e3oncan",
-    "version": "0.6.15",
+    "version": "0.6.17",
     "news": {
+      "0.6.17": {
+        "en": "Added/removed datapoints to/from list of writable dids\nPreparations for device specific list of writable dids",
+        "de": "Hinzugefügt/entfernte Datenpunkte zu/aus der Liste der schreibbaren Taten\nVorbereitungen für die gerätespezifische Liste der schreibbaren Taten",
+        "ru": "Добавлены/удалены точки данных в/из списка заслуживающих доверия дел\nПодготовка к конкретному списку реальных дел",
+        "pt": "Datapoints adicionados/removed para / da lista de dids writable\nPreparações para a lista específica do dispositivo de dids",
+        "nl": "Toegevoegd/verwijderde datapunten aan/van lijst van schrijfbare dids\nVoorbereidingen voor apparaatspecifieke lijst van schrijfbare dids",
+        "fr": "Ajout/retrait de points de données à la liste des dids en écriture\nPréparations pour la liste spécifique des dids à écrire",
+        "it": "Datapoint aggiunti/rimovibili da/per l'elenco di fazioni scrivibili\nPreparazioni per l'elenco specifico del dispositivo di fazioni scrivibili",
+        "es": "Puntos de datos añadidos/removidos a/desde la lista de didas\nPreparaciones para la lista específica del dispositivo de didas",
+        "pl": "Dodano / usunięto punkty danych do / z listy zapisanych did\nPreparaty do specyficznego dla urządzenia wykazu dysków pisanych",
+        "uk": "Додано/видалення точок даних до/з списку збитих\nПрепарати для пристрою специфічного переліку життєздатних",
+        "zh-cn": "已添加/删除数据点到/从可写作的列表中\n编写设备特定可写行为清单"
+      },
+      "0.6.16": {
+        "en": "Improvements based on findings in review as of 2024-01-25\nCheckbox for data collectiton on internal bus is now checked per default",
+        "de": "Verbesserungen auf der Grundlage der Ergebnisse in der Überprüfung ab 2024-01-25\nCheckbox für Datensammler im internen Bus wird nun standardmäßig überprüft",
+        "ru": "Совершенствование на основе результатов, рассмотренных на 2024-01-25 годы\nCheckbox для сбора данных на внутреннем автобусе теперь проверяется по умолчанию",
+        "pt": "Aperfeiçoamentos baseados em achados em revisão a partir de 2024-01-25\nCaixa de seleção para coleta de dados no ônibus interno é agora verificada por padrão",
+        "nl": "Verbeteringen op basis van de resultaten van het onderzoek per 2024-01-25\nCheckbox voor gegevensverzameling op interne bus is nu standaard aangevinkt",
+        "fr": "Améliorations fondées sur les conclusions de l'examen en 2024-01-25\nLa case à cocher pour les données collectiton sur le bus interne est maintenant cochée par défaut",
+        "it": "Miglioramenti basati sui risultati in esame a partire dal 2024-01-25\nLa casella di controllo per i dati collettivi sul bus interno è ora controllata per impostazione predefinita",
+        "es": "Mejoras basadas en los resultados examinados al 2024-01-25\nCheckbox for data collectiton on internal bus is now check per default",
+        "pl": "Poprawki oparte na ustaleniach w przeglądzie z 2024- 01- 25\nSkrzynka kontrolna dla kolekcji danych w autobusie wewnętrznym jest teraz sprawdzana na domyślnym",
+        "uk": "Удосконалення на основі пошуку в рецензії на 2024-01-25\nCheckbox для збору даних у внутрішньому автобусі тепер перевіряється за замовчуванням",
+        "zh-cn": "根据截至2024-01-25年审查结果作出的改进\n内部总线上数据集合的复选框现在按默认检查"
+      },
       "0.6.15": {
         "en": "Fix for Utf8 codec for handling of special characters, e.g. umlauts",
         "de": "Fix für Utf8-Codec zum Umgang mit Sonderzeichen, z.B. umlauts",
@@ -67,32 +93,6 @@
         "pl": "Ulepszony układ okna konfiguracji skanowania urządzenia",
         "uk": "Покращений макет діалогу конфігурації для сканування пристроїв",
         "zh-cn": "设备扫描配置对话框的改进版式"
-      },
-      "0.6.10": {
-        "en": "Removed code for Rawmode because it's never activated",
-        "de": "Entfernter Code für Rawmode, weil es nie aktiviert ist",
-        "ru": "Удаленный код для Rawmode, потому что он никогда не активирован",
-        "pt": "Código removido para Rawmode porque nunca foi ativado",
-        "nl": "Verwijderde code voor Rawmode omdat het nooit is geactiveerd",
-        "fr": "Code supprimé pour Rawmode car il n'est jamais activé",
-        "it": "Codice rimosso per Rawmode perché non è mai attivato",
-        "es": "Código eliminado para Rawmode porque nunca se activa",
-        "pl": "Usunięto kod dla Rawmode ponieważ nigdy nie jest aktywowany",
-        "uk": "Видалити код для Rawmode, тому що він ніколи не активований",
-        "zh-cn": "删除了Rawmode的代码,因为它从未被激活"
-      },
-      "0.6.9": {
-        "en": "Bugfix: Only Linux is supported",
-        "de": "Bugfix: Nur Linux wird unterstützt",
-        "ru": "Исправление: Только Linux поддерживается",
-        "pt": "Bugfix: Apenas o Linux é suportado",
-        "nl": "Bugfix: Alleen Linux wordt ondersteund",
-        "fr": "Correction : Seul Linux est pris en charge",
-        "it": "Bugfix: Solo Linux è supportato",
-        "es": "Bugfix: Sólo Linux es compatible",
-        "pl": "Bugfix: Tylko Linux jest obsługiwany",
-        "uk": "Помилки: Тільки Linux підтримується",
-        "zh-cn": "错误修正 : 仅支持 Linux"
       }
     },
     "title": "Viessmann E3 CAN do",

package/lib/canCollect.js

@@ -57,7 +57,7 @@ class collect {
             await this.storage.setOpMode('standby');
 
             // Stop Timeout:
-            if (this.timeoutHandle) await clearTimeout(this.timeoutHandle);
+            if (this.timeoutHandle) await ctx.clearTimeout(this.timeoutHandle);
             this.timeoutHandle = null;
 
             this.stat.state = 'stopped';
@@ -71,10 +71,6 @@ class collect {
         }
     }
 
-    sleep(milliseconds) {
-        return new Promise(resolve => setTimeout(resolve, milliseconds));
-    }
-
     async msgCollect(ctx, msg) {
 
         if (await this.storage.getOpMode() == 'standby') return;    // No communication in mode 'standby'
@@ -109,7 +105,7 @@ class collect {
                     this.data.databytes = this.data.databytes.concat(candata.slice(1));
                     if (this.data.databytes.length >= this.data.len) {
                         // All data received
-                        if (this.timeoutHandle) await clearTimeout(this.timeoutHandle);
+                        if (this.timeoutHandle) await ctx.clearTimeout(this.timeoutHandle);
                         this.stat.cntCommOk += 1;
                         if (!(this.data.did in this.ts)) { this.ts[this.data.did] = tsNow; }
                         if ( (this.config.delay == 0) || ((tsNow >= this.ts[this.data.did])) ) {
@@ -162,7 +158,7 @@ class collect {
                                 this.data.databytes = candata.slice(5);
                                 this.data.len = candata[4];
                             }
-                            this.timeoutHandle = setTimeout(this.onTimeout, this.config.timeout, ctx, this);
+                            this.timeoutHandle = ctx.setTimeout(this.onTimeout, this.config.timeout, ctx, this);
                             this.data.collecting = true;
                             break;
                         case 0xb5:
@@ -181,7 +177,7 @@ class collect {
                             this.data.D0expected = candata[0]+1;
                             this.data.databytes = candata.slice(4);
                             this.data.len = D3-0xb0;
-                            this.timeoutHandle = setTimeout(this.onTimeout, this.config.timeout, ctx, this);
+                            this.timeoutHandle = ctx.setTimeout(this.onTimeout, this.config.timeout, ctx, this);
                             this.data.collecting = true;
                             break;
                         default:

package/lib/canUds.js

@@ -9,21 +9,21 @@ class scheduleLoop {
         this.schedHandle = null;
     }
 
-    async startSchedule() {
+    async startSchedule(ctx) {
         if (this.schedule == 0) {
             await this.ctxGlobal.log.silly('UDS schedule one time: '+this.ctxLocal.canIDhex+'.'+JSON.stringify(this.dids));
             await this.ctxLocal.pushCmnd(this.ctxGlobal, 'read', this.dids);
             this.schedHandle = null;
         } else {
-            this.schedHandle = setInterval(async () => {
+            this.schedHandle = ctx.setInterval(async () => {
                 await this.loop();
             }, this.schedule*1000);
         }
     }
 
-    async stopSchedule() {
+    async stopSchedule(ctx) {
         try {
-            if (this.schedHandle) await clearInterval(this.schedHandle);
+            if (this.schedHandle) await ctx.clearInterval(this.schedHandle);
             await this.ctxGlobal.log.silly('UDS schedule stopped: '+String(this.schedule)+' '+this.ctxLocal.canIDhex+'.'+JSON.stringify(this.dids));
         } catch (e) {
             // Dod nothing
@@ -110,7 +110,6 @@ class uds {
                 type: 'channel',
                 common: {
                     name: this.config.stateBase+' commands',
-                    role: 'device'
                 },
                 native: {},
             });
@@ -139,7 +138,7 @@ class uds {
         if (opMode == 'normal') {
             for (const sched of Object.values(this.schedules)) {
                 // Start schedules on startup and do one-time schedules
-                await sched.startSchedule();
+                await sched.startSchedule(ctx);
             }
         }
         if (opMode == 'normal') {
@@ -147,7 +146,7 @@ class uds {
         } else {
             await ctx.log.silly('UDS worker started in mode '+opMode+' on '+this.config.stateBase);
         }
-        this.cmndsHandle = setInterval(async () => {
+        this.cmndsHandle = ctx.setInterval(async () => {
             await this.cmndsLoop(ctx);
         }, this.cmndsUpdateTime);
     }
@@ -164,18 +163,18 @@ class uds {
 
             // Stop loops:
             for (const sched of Object.values(this.schedules)) {
-                await sched.stopSchedule();
+                await sched.stopSchedule(ctx);
             }
-            if (this.cmndsHandle) await clearInterval(this.cmndsHandle);
+            if (this.cmndsHandle) await ctx.clearInterval(this.cmndsHandle);
 
             // Stop Timeout:
-            if (this.timeoutHandle) await clearTimeout(this.timeoutHandle);
+            if (this.timeoutHandle) await ctx.clearTimeout(this.timeoutHandle);
             this.timeoutHandle = null;
 
             // Stop worker:
             this.callback = null;
             if (opMode == 'normal') {
-                await ctx.unsubscribeStates(ctx.namespace+'.'+this.config.stateBase+'.*');
+                ctx.unsubscribeStates(ctx.namespace+'.'+this.config.stateBase+'.*');
                 await ctx.log.info('UDS worker stopped on '+this.config.stateBase);
             } else {
                 await ctx.log.silly('UDS worker stopped in mode '+opMode+' on '+this.config.stateBase);
@@ -201,12 +200,12 @@ class uds {
         this.data.state = comState;
     }
 
-    async setDidDone(coolDownTime) {
+    async setDidDone(ctx, coolDownTime) {
         // Finalize communication for recent did
         this.coolDownTs = new Date().getTime()+coolDownTime;
         if (this.cmndsQueue.length == 0) this.busy = false;
         await this.setComState(0);
-        if (this.timeoutHandle) await clearTimeout(this.timeoutHandle);
+        if (this.timeoutHandle) await ctx.clearTimeout(this.timeoutHandle);
     }
 
     async setDidStart(ctx, did, mode, len) {
@@ -228,9 +227,9 @@ class uds {
         }
         const tsNow = new Date().getTime();
         const minWaiting = this.coolDownTs - tsNow;
-        if (minWaiting > 0) await this.sleep(minWaiting);
+        if (minWaiting > 0) await this.sleep(ctx, minWaiting);
         this.busy = true;
-        this.timeoutHandle = await setTimeout(this.onTimeout, this.config.timeout, ctx, this);
+        this.timeoutHandle = await ctx.setTimeout(this.onTimeout, this.config.timeout, ctx, this);
         this.data.did = did;
         this.data.tsRequest = tsNow;
     }
@@ -269,8 +268,8 @@ class uds {
         }
     }
 
-    sleep(milliseconds) {
-        return new Promise(resolve => setTimeout(resolve, milliseconds));
+    sleep(ctx, milliseconds) {
+        return new Promise(resolve => ctx.setTimeout(resolve, milliseconds));
     }
 
     async cmndsLoop(ctx) {
@@ -309,7 +308,7 @@ class uds {
             ctxLocal.stat.cntCommTimeoutPerDid[did]  = 1;
         }
         if (ctxLocal.callback) await ctxLocal.callback(ctxGlobal, ctxLocal, ['timeout', {'did':ctxLocal.data.did,'didInfo':{'id':'','len':0},'val':''}]);
-        await ctxLocal.setDidDone(0);
+        await ctxLocal.setDidDone(ctxGlobal, 0);
     }
 
     async onUdsStateChange(ctx, ctxWorker, id, state) {
@@ -369,7 +368,7 @@ class uds {
                     byteArr = await this.storage.encodeDataCAN(ctx, this, did, await JSON.parse(state.val));
                     if (byteArr) {
                         await this.pushCmnd(ctx, 'write', [[did,byteArr]]);
-                        setTimeout(function(ctxWorker,did){ctxWorker.cmndsQueue.push({'mode':'read', 'did': did});},2500,this,did);    // Read value after 2500 ms
+                        ctx.setTimeout(function(ctxWorker,did){ctxWorker.cmndsQueue.push({'mode':'read', 'did': did});},2500,this,did);    // Read value after 2500 ms
                     } else {
                         ctx.log.error('User command UDS WriteByDid on '+this.config.stateBase+': Encoding of data failed.');
                     }
@@ -382,7 +381,7 @@ class uds {
                 byteArr = this.storage.toByteArray(await JSON.parse(state.val));
                 if (byteArr) {
                     await this.pushCmnd(ctx, 'write', [[did,byteArr]]);
-                    setTimeout(function(ctxWorker,did){ctxWorker.cmndsQueue.push({'mode':'read', 'did': did});},2500,this,did);    // Read value after 2500 ms
+                    ctx.setTimeout(function(ctxWorker,did){ctxWorker.cmndsQueue.push({'mode':'read', 'did': did});},2500,this,did);    // Read value after 2500 ms
                 } else {
                     ctx.log.error('User command UDS WriteByDid on '+this.config.stateBase+': Encoding of data failed.');
                 }
@@ -397,7 +396,7 @@ class uds {
                         byteArr = await this.storage.encodeDataCAN(ctx, this, did, await JSON.parse(state.val));
                         if (byteArr) {
                             await this.pushCmnd(ctx, 'write', [[did,byteArr]]);
-                            setTimeout(function(ctxWorker,did){ctxWorker.cmndsQueue.push({'mode':'read', 'did': did});},2500,this,did);    // Read value after 2500 ms
+                            ctx.setTimeout(function(ctxWorker,did){ctxWorker.cmndsQueue.push({'mode':'read', 'did': did});},2500,this,did);    // Read value after 2500 ms
                         } else {
                             ctx.log.error('User command UDS WriteByDid on '+this.config.stateBase+': Encoding of data failed.');
                         }
@@ -434,7 +433,7 @@ class uds {
                         byteArr = await ctxWorker.storage.encodeDataCAN(ctx, ctxWorker, did, treeDict);
                         if (byteArr) {
                             await ctxWorker.pushCmnd(ctx, 'write', [[did,byteArr]]);
-                            setTimeout(function(ctxWorker,did){ctxWorker.cmndsQueue.push({'mode':'read', 'did': did});},2500,ctxWorker,did);    // Read value after 2500 ms
+                            ctx.setTimeout(function(ctxWorker,did){ctxWorker.cmndsQueue.push({'mode':'read', 'did': did});},2500,ctxWorker,did);    // Read value after 2500 ms
                         } else {
                             ctx.log.error('User command UDS WriteByDid on '+ctxWorker.config.stateBase+': Encoding of data failed.');
                         }
@@ -540,7 +539,7 @@ class uds {
                     } else {
                         ctx.log.error('UDS worker error on '+this.config.stateBase+': Negative response reading did '+String(this.data.did)+'. Code=0x'+Number(candata[3]).toString(16));
                     }
-                    await this.setDidDone(0);
+                    await this.setDidDone(ctx, 0);
                     break;
                 }
                 if ( (candata.length == 8) && ((candata[0] >> 4) == 0) && (candata[1] == this.readByDidProt.SIDrx) ) {
@@ -554,7 +553,7 @@ class uds {
                         this.data.len = candata[0]-3;
                         this.data.databytes = candata.slice(4,4+this.data.len);
                         this.storage.decodeDataCAN(ctx, this, this.data.did, this.data.databytes.slice(0,this.data.len));
-                        await this.setDidDone(0);
+                        await this.setDidDone(ctx, 0);
                         break;
                     } else {
                         // Did does not match
@@ -564,7 +563,7 @@ class uds {
                         } else {
                             ctx.log.error('UDS worker on '+this.config.stateBase+': Did mismatch MF. Expected='+String(this.data.did)+'; Received='+String(didRx));
                         }
-                        await this.setDidDone(1000);
+                        await this.setDidDone(ctx, 1000);
                         break;
                     }
                 }
@@ -589,7 +588,7 @@ class uds {
                         } else {
                             ctx.log.error('UDS worker on '+this.config.stateBase+': Did mismatch MF. Expected='+String(this.data.did)+'; Received='+String(didRx));
                         }
-                        await this.setDidDone(1000);
+                        await this.setDidDone(ctx, 1000);
                         break;
                     }
                 }
@@ -600,7 +599,7 @@ class uds {
                 }
                 await this.calcStat();
                 this.stat.cntCommBadProtocol += 1;
-                await this.setDidDone(2500);
+                await this.setDidDone(ctx, 2500);
                 break;
 
             case 2: // waitForCFrbd
@@ -614,7 +613,7 @@ class uds {
                         await this.calcStat();
                         ctx.log.silly('UDS worker on '+this.config.stateBase+': MF completed. candata: '+this.storage.arr2Hex(candata));
                         this.storage.decodeDataCAN(ctx, this, this.data.did, this.data.databytes.slice(0,this.data.len));
-                        await this.setDidDone(0);
+                        await this.setDidDone(ctx, 0);
                     } else {
                         // More data to come
                         this.data.D0 += 1;
@@ -628,7 +627,7 @@ class uds {
                         ctx.log.error('UDS worker on '+this.config.stateBase+': Bad frame. candata: '+this.storage.arr2Hex(candata));
                     }
                     this.stat.cntCommBadProtocol += 1;
-                    await this.setDidDone(2500);
+                    await this.setDidDone(ctx, 2500);
                 }
                 break;
 
@@ -637,7 +636,7 @@ class uds {
                     // Negative response
                     this.stat.cntCommNR += 1;
                     ctx.log.error('UDS worker error on '+this.config.stateBase+': Negative response writing did '+String(this.data.did)+'. Code=0x'+Number(candata[3]).toString(16));
-                    await this.setDidDone(0);
+                    await this.setDidDone(ctx, 0);
                     break;
                 }
                 if ( (candata.length == 8) && (candata[0] == 0x03) && (candata[1] == this.writeByDidProt.SIDrx) ) {
@@ -649,7 +648,7 @@ class uds {
                         ctx.log.silly('UDS worker on '+this.config.stateBase+': writeByDid SF confirmation received.');
                         await this.calcStat();
                         this.storage.storeStatistics(ctx, this, false);
-                        await this.setDidDone(0);
+                        await this.setDidDone(ctx, 0);
                         break;
                     } else {
                         // Did does not match
@@ -657,14 +656,14 @@ class uds {
                         ctx.log.error('UDS worker on '+this.config.stateBase+': Did mismatch writeByDid SF. Expected='+String(this.data.did)+'; Received='+String(didRx));
                         await this.calcStat();
                         this.storage.storeStatistics(ctx, this, true);
-                        await this.setDidDone(1000);
+                        await this.setDidDone(ctx, 1000);
                         break;
                     }
                 }
                 ctx.log.error('UDS worker on '+this.config.stateBase+': Bad frame for writeByDid SF. candata: '+this.storage.arr2Hex(candata));
                 this.stat.cntCommBadProtocol += 1;
                 await this.calcStat();
-                await this.setDidDone(2500);
+                await this.setDidDone(ctx, 2500);
                 break;
 
             case 4: // waitForFFFwbd
@@ -672,7 +671,7 @@ class uds {
                     // Negative response
                     this.stat.cntCommNR += 1;
                     ctx.log.error('UDS worker error on '+this.config.stateBase+': Negative response writing did '+String(this.data.did)+'. Code=0x'+Number(candata[3]).toString(16));
-                    await this.setDidDone(0);
+                    await this.setDidDone(ctx, 0);
                     break;
                 }
                 if ( (candata.length == 8) && (candata[0] == 0x30) && (candata[1] == 0x00) ) {
@@ -682,7 +681,7 @@ class uds {
                     if ((ST<20) || (ST>127)) ST=50; // Accept ST 20 .. 127 ms. Default to 50 ms.
                     while (this.data.txPos < this.data.len) {
                         // More data to send
-                        await this.sleep(ST);
+                        await this.sleep(ctx, ST);
                         const frame = [this.data.D0].concat(this.data.databytes.slice(this.data.txPos,this.data.txPos+7));
                         await this.sendFrame(ctx, frame);
                         this.data.txPos += 7;
@@ -695,7 +694,7 @@ class uds {
                 ctx.log.error('UDS worker on '+this.config.stateBase+': Bad frame for writeByDid MF. candata: '+this.storage.arr2Hex(candata));
                 this.stat.cntCommBadProtocol += 1;
                 await this.calcStat();
-                await this.setDidDone(2500);
+                await this.setDidDone(ctx, 2500);
                 break;
 
             default:
@@ -705,7 +704,7 @@ class uds {
                 } else {
                     ctx.log.error('UDS worker on '+this.config.stateBase+': Bad state value: '+String(await this.getComState()));
                 }
-                await this.setDidDone(2500);
+                await this.setDidDone(ctx, 2500);
         }
         this.commBusy = false;
     }

package/lib/codecs.test.js

@@ -208,7 +208,7 @@ describe('codecs.js => O3EInt8().encode()', () => {
     codecEncode('Positive Number, scale=10, signed=false', [0xfb], E3.O3Ecodecs.O3EInt8, 1, 'test5', {scale:10.0,signed:false,offset: 0}, 25.1);
 });
 
-describe('codecs.js => O3EInt16().decode()', () => {
+describe('codecs.js => O3EInt16().encode()', () => {
     codecEncode('Positive Number, scale=10, signed=true ', [0x32,0x00], E3.O3Ecodecs.O3EInt16, 2, 'test1', {scale:10.0,signed:true, offset: 0}, 5);
     codecEncode('Positive Number, scale=10, signed=true ', [0xff,0x7f], E3.O3Ecodecs.O3EInt16, 2, 'test2', {scale:10.0,signed:true, offset: 0}, (2**15-1)/10);
     codecEncode('Negative Number, scale=10, signed=true ', [0x00,0x80], E3.O3Ecodecs.O3EInt16, 2, 'test3', {scale:10.0,signed:true, offset: 0}, -(2**15)/10);