node-poweredup 10.0.0 → 10.1.0

package/dist/utils.js

@@ -1,4 +1,3 @@
-import { EventEmitter } from "events";
 // @ts-ignore
 export const isWebBluetooth = !!(typeof navigator !== "undefined" && navigator && navigator.bluetooth);
 export const toHex = (value, length = 2) => {
@@ -51,35 +50,30 @@ export const roundAngleToNearest90 = (angle) => {
     }
     return -180;
 };
-export const calculateRamp = (device, fromPower, toPower, time) => {
-    const emitter = new EventEmitter();
+export const calculateRamp = (fromPower, toPower, time) => {
     const steps = Math.abs(toPower - fromPower);
+    if (steps === 0 || time <= 0) {
+        return [toPower];
+    }
     let delay = time / steps;
     let increment = 1;
-    if (delay < 50 && steps > 0) {
+    if (delay < 50) {
         increment = 50 / delay;
         delay = 50;
     }
     if (fromPower > toPower) {
         increment = -increment;
     }
-    let i = 0;
-    const interval = setInterval(() => {
-        let power = Math.round(fromPower + (++i * increment));
-        if (toPower > fromPower && power > toPower) {
-            power = toPower;
-        }
-        else if (fromPower > toPower && power < toPower) {
-            power = toPower;
+    return Array(Math.round(time / delay)).fill(0).map((element, index) => {
+        let value = Math.round(fromPower + (index + 1) * increment);
+        if (toPower > fromPower && value > toPower) {
+            value = toPower;
         }
-        emitter.emit("changePower", power);
-        if (power === toPower) {
-            clearInterval(interval);
-            emitter.emit("finished");
+        else if (fromPower > toPower && value < toPower) {
+            value = toPower;
         }
-    }, delay);
-    device.setEventTimer(interval);
-    return emitter;
+        return value;
+    });
 };
 export const parseColor = (color) => {
     if (color === 1 || color === 5) {

package/dist/utils.js.map

@@ -1 +1 @@
-{"version":3,"file":"utils.js","sourceRoot":"","sources":["../src/utils.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,YAAY,EAAE,MAAM,QAAQ,CAAC;AAGtC,aAAa;AACb,MAAM,CAAC,MAAM,cAAc,GAAG,CAAC,CAAC,CAAC,OAAO,SAAS,KAAK,WAAW,IAAI,SAAS,IAAI,SAAS,CAAC,SAAS,CAAC,CAAC;AAEvG,MAAM,CAAC,MAAM,KAAK,GAAG,CAAC,KAAa,EAAE,SAAiB,CAAC,EAAE,EAAE;IACvD,OAAO,KAAK,CAAC,QAAQ,CAAC,EAAE,CAAC,CAAC,QAAQ,CAAC,MAAM,EAAE,GAAG,CAAC,CAAC;AACpD,CAAC,CAAC;AAEF,MAAM,CAAC,MAAM,KAAK,GAAG,CAAC,KAAa,EAAE,SAAiB,CAAC,EAAE,EAAE;IACvD,OAAO,KAAK,CAAC,QAAQ,CAAC,CAAC,CAAC,CAAC,QAAQ,CAAC,MAAM,EAAE,GAAG,CAAC,CAAC;AACnD,CAAC,CAAC;AAEF,MAAM,CAAC,MAAM,QAAQ,GAAG,CAAC,KAAa,EAAE,EAAE;IACtC,IAAI,KAAK,KAAK,GAAG,EAAE,CAAC;QAChB,OAAO,GAAG,CAAC;IACf,CAAC;IACD,IAAI,KAAK,GAAG,GAAG,EAAE,CAAC;QACd,KAAK,GAAG,GAAG,CAAC;IAChB,CAAC;SAAM,IAAI,KAAK,GAAG,CAAC,GAAG,EAAE,CAAC;QACtB,KAAK,GAAG,CAAC,GAAG,CAAC;IACjB,CAAC;IACD,OAAO,KAAK,CAAC;AACjB,CAAC,CAAC;AAEF,MAAM,CAAC,MAAM,aAAa,GAAG,CAAC,OAAe,EAAE,EAAE;IAC7C,MAAM,KAAK,GAAG,OAAO,CAAC,QAAQ,CAAC,EAAE,CAAC,CAAC,QAAQ,CAAC,CAAC,EAAE,GAAG,CAAC,CAAC;IACpD,OAAO,CAAC,KAAK,CAAC,CAAC,CAAC,EAAE,KAAK,CAAC,CAAC,CAAC,EAAE,KAAK,CAAC,SAAS,CAAC,CAAC,EAAE,CAAC,CAAC,EAAE,KAAK,CAAC,SAAS,CAAC,CAAC,CAAC,CAAC,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC;AACrF,CAAC,CAAC;AAEF,MAAM,CAAC,MAAM,gBAAgB,GAAG,CAAC,OAAmB,EAAE,EAAE;IACpD,OAAO,KAAK,CAAC,IAAI,CAAC,OAAO,CAAC,CAAC,GAAG,CAAC,CAAC,IAAI,EAAE,EAAE,CAAC,KAAK,CAAC,IAAI,EAAE,CAAC,CAAC,CAAC,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC;AACvE,CAAC,CAAC;AAEF,MAAM,CAAC,MAAM,cAAc,GAAG,CAAC,KAAa,EAAE,EAAE;IAC5C,IAAI,KAAK,IAAI,GAAG,EAAE,CAAC;QACf,OAAO,KAAK,GAAG,CAAC,GAAG,GAAG,CAAC,CAAC,KAAK,GAAG,GAAG,CAAC,GAAG,GAAG,CAAC,CAAC,CAAC;IACjD,CAAC;SAAM,IAAI,KAAK,GAAG,CAAC,GAAG,EAAE,CAAC;QACtB,OAAO,KAAK,GAAG,CAAC,GAAG,GAAG,CAAC,CAAC,GAAG,GAAG,KAAK,CAAC,GAAG,GAAG,CAAC,CAAC,CAAC;IACjD,CAAC;IACD,OAAO,KAAK,CAAC;AACjB,CAAC,CAAC;AAEF,MAAM,CAAC,MAAM,qBAAqB,GAAG,CAAC,KAAa,EAAE,EAAE;IACnD,KAAK,GAAG,cAAc,CAAC,KAAK,CAAC,CAAC;IAC9B,IAAI,KAAK,GAAG,CAAC,GAAG,EAAE,CAAC;QACf,OAAO,CAAC,GAAG,CAAC;IAChB,CAAC;IACD,IAAI,KAAK,GAAG,CAAC,EAAE,EAAE,CAAC;QACd,OAAO,CAAC,EAAE,CAAC;IACf,CAAC;IACD,IAAI,KAAK,GAAG,EAAE,EAAE,CAAC;QACb,OAAO,CAAC,CAAC;IACb,CAAC;IACD,IAAI,KAAK,GAAG,GAAG,EAAE,CAAC;QACd,OAAO,EAAE,CAAC;IACd,CAAC;IACD,OAAO,CAAC,GAAG,CAAC;AAChB,CAAC,CAAC;AAEF,MAAM,CAAC,MAAM,aAAa,GAAG,CAAC,MAAc,EAAE,SAAiB,EAAE,OAAe,EAAE,IAAY,EAAE,EAAE;IAC9F,MAAM,OAAO,GAAG,IAAI,YAAY,EAAE,CAAC;IACnC,MAAM,KAAK,GAAG,IAAI,CAAC,GAAG,CAAC,OAAO,GAAG,SAAS,CAAC,CAAC;IAC5C,IAAI,KAAK,GAAG,IAAI,GAAG,KAAK,CAAC;IACzB,IAAI,SAAS,GAAG,CAAC,CAAC;IAClB,IAAI,KAAK,GAAG,EAAE,IAAI,KAAK,GAAG,CAAC,EAAE,CAAC;QAC1B,SAAS,GAAG,EAAE,GAAG,KAAK,CAAC;QACvB,KAAK,GAAG,EAAE,CAAC;IACf,CAAC;IACD,IAAI,SAAS,GAAG,OAAO,EAAE,CAAC;QACtB,SAAS,GAAG,CAAC,SAAS,CAAC;IAC3B,CAAC;IACD,IAAI,CAAC,GAAG,CAAC,CAAC;IACV,MAAM,QAAQ,GAAG,WAAW,CAAC,GAAG,EAAE;QAC9B,IAAI,KAAK,GAAG,IAAI,CAAC,KAAK,CAAC,SAAS,GAAG,CAAC,EAAE,CAAC,GAAG,SAAS,CAAC,CAAC,CAAC;QACtD,IAAI,OAAO,GAAG,SAAS,IAAI,KAAK,GAAG,OAAO,EAAE,CAAC;YACzC,KAAK,GAAG,OAAO,CAAC;QACpB,CAAC;aAAM,IAAI,SAAS,GAAG,OAAO,IAAI,KAAK,GAAG,OAAO,EAAE,CAAC;YAChD,KAAK,GAAG,OAAO,CAAC;QACpB,CAAC;QACD,OAAO,CAAC,IAAI,CAAC,aAAa,EAAE,KAAK,CAAC,CAAC;QACnC,IAAI,KAAK,KAAK,OAAO,EAAE,CAAC;YACpB,aAAa,CAAC,QAAQ,CAAC,CAAC;YACxB,OAAO,CAAC,IAAI,CAAC,UAAU,CAAC,CAAC;QAC7B,CAAC;IACL,CAAC,EAAE,KAAK,CAAC,CAAC;IACV,MAAM,CAAC,aAAa,CAAC,QAAQ,CAAC,CAAC;IAC/B,OAAO,OAAO,CAAC;AACnB,CAAC,CAAC;AAEF,MAAM,CAAC,MAAM,UAAU,GAAG,CAAC,KAAa,EAAE,EAAE;IACxC,IAAI,KAAK,KAAK,CAAC,IAAI,KAAK,KAAK,CAAC,EAAE,CAAC;QAC7B,KAAK,GAAG,KAAK,GAAG,CAAC,CAAC;IACtB,CAAC;IACD,OAAO,KAAK,CAAC;AACjB,CAAC,CAAA"}
+{"version":3,"file":"utils.js","sourceRoot":"","sources":["../src/utils.ts"],"names":[],"mappings":"AAGA,aAAa;AACb,MAAM,CAAC,MAAM,cAAc,GAAG,CAAC,CAAC,CAAC,OAAO,SAAS,KAAK,WAAW,IAAI,SAAS,IAAI,SAAS,CAAC,SAAS,CAAC,CAAC;AAEvG,MAAM,CAAC,MAAM,KAAK,GAAG,CAAC,KAAa,EAAE,SAAiB,CAAC,EAAE,EAAE;IACvD,OAAO,KAAK,CAAC,QAAQ,CAAC,EAAE,CAAC,CAAC,QAAQ,CAAC,MAAM,EAAE,GAAG,CAAC,CAAC;AACpD,CAAC,CAAC;AAEF,MAAM,CAAC,MAAM,KAAK,GAAG,CAAC,KAAa,EAAE,SAAiB,CAAC,EAAE,EAAE;IACvD,OAAO,KAAK,CAAC,QAAQ,CAAC,CAAC,CAAC,CAAC,QAAQ,CAAC,MAAM,EAAE,GAAG,CAAC,CAAC;AACnD,CAAC,CAAC;AAEF,MAAM,CAAC,MAAM,QAAQ,GAAG,CAAC,KAAa,EAAE,EAAE;IACtC,IAAI,KAAK,KAAK,GAAG,EAAE,CAAC;QAChB,OAAO,GAAG,CAAC;IACf,CAAC;IACD,IAAI,KAAK,GAAG,GAAG,EAAE,CAAC;QACd,KAAK,GAAG,GAAG,CAAC;IAChB,CAAC;SAAM,IAAI,KAAK,GAAG,CAAC,GAAG,EAAE,CAAC;QACtB,KAAK,GAAG,CAAC,GAAG,CAAC;IACjB,CAAC;IACD,OAAO,KAAK,CAAC;AACjB,CAAC,CAAC;AAEF,MAAM,CAAC,MAAM,aAAa,GAAG,CAAC,OAAe,EAAE,EAAE;IAC7C,MAAM,KAAK,GAAG,OAAO,CAAC,QAAQ,CAAC,EAAE,CAAC,CAAC,QAAQ,CAAC,CAAC,EAAE,GAAG,CAAC,CAAC;IACpD,OAAO,CAAC,KAAK,CAAC,CAAC,CAAC,EAAE,KAAK,CAAC,CAAC,CAAC,EAAE,KAAK,CAAC,SAAS,CAAC,CAAC,EAAE,CAAC,CAAC,EAAE,KAAK,CAAC,SAAS,CAAC,CAAC,CAAC,CAAC,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC;AACrF,CAAC,CAAC;AAEF,MAAM,CAAC,MAAM,gBAAgB,GAAG,CAAC,OAAmB,EAAE,EAAE;IACpD,OAAO,KAAK,CAAC,IAAI,CAAC,OAAO,CAAC,CAAC,GAAG,CAAC,CAAC,IAAI,EAAE,EAAE,CAAC,KAAK,CAAC,IAAI,EAAE,CAAC,CAAC,CAAC,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC;AACvE,CAAC,CAAC;AAEF,MAAM,CAAC,MAAM,cAAc,GAAG,CAAC,KAAa,EAAE,EAAE;IAC5C,IAAI,KAAK,IAAI,GAAG,EAAE,CAAC;QACf,OAAO,KAAK,GAAG,CAAC,GAAG,GAAG,CAAC,CAAC,KAAK,GAAG,GAAG,CAAC,GAAG,GAAG,CAAC,CAAC,CAAC;IACjD,CAAC;SAAM,IAAI,KAAK,GAAG,CAAC,GAAG,EAAE,CAAC;QACtB,OAAO,KAAK,GAAG,CAAC,GAAG,GAAG,CAAC,CAAC,GAAG,GAAG,KAAK,CAAC,GAAG,GAAG,CAAC,CAAC,CAAC;IACjD,CAAC;IACD,OAAO,KAAK,CAAC;AACjB,CAAC,CAAC;AAEF,MAAM,CAAC,MAAM,qBAAqB,GAAG,CAAC,KAAa,EAAE,EAAE;IACnD,KAAK,GAAG,cAAc,CAAC,KAAK,CAAC,CAAC;IAC9B,IAAI,KAAK,GAAG,CAAC,GAAG,EAAE,CAAC;QACf,OAAO,CAAC,GAAG,CAAC;IAChB,CAAC;IACD,IAAI,KAAK,GAAG,CAAC,EAAE,EAAE,CAAC;QACd,OAAO,CAAC,EAAE,CAAC;IACf,CAAC;IACD,IAAI,KAAK,GAAG,EAAE,EAAE,CAAC;QACb,OAAO,CAAC,CAAC;IACb,CAAC;IACD,IAAI,KAAK,GAAG,GAAG,EAAE,CAAC;QACd,OAAO,EAAE,CAAC;IACd,CAAC;IACD,OAAO,CAAC,GAAG,CAAC;AAChB,CAAC,CAAC;AAEF,MAAM,CAAC,MAAM,aAAa,GAAG,CAAC,SAAiB,EAAE,OAAe,EAAE,IAAY,EAAE,EAAE;IAC9E,MAAM,KAAK,GAAG,IAAI,CAAC,GAAG,CAAC,OAAO,GAAG,SAAS,CAAC,CAAC;IAC5C,IAAI,KAAK,KAAK,CAAC,IAAI,IAAI,IAAI,CAAC,EAAE,CAAC;QAC3B,OAAO,CAAC,OAAO,CAAC,CAAC;IACrB,CAAC;IACD,IAAI,KAAK,GAAG,IAAI,GAAG,KAAK,CAAC;IACzB,IAAI,SAAS,GAAG,CAAC,CAAC;IAClB,IAAI,KAAK,GAAG,EAAE,EAAE,CAAC;QACb,SAAS,GAAG,EAAE,GAAG,KAAK,CAAC;QACvB,KAAK,GAAG,EAAE,CAAC;IACf,CAAC;IACD,IAAI,SAAS,GAAG,OAAO,EAAE,CAAC;QACtB,SAAS,GAAG,CAAC,SAAS,CAAC;IAC3B,CAAC;IACD,OAAO,KAAK,CAAC,IAAI,CAAC,KAAK,CAAC,IAAI,GAAG,KAAK,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC,GAAG,CAAC,CAAC,OAAO,EAAE,KAAK,EAAE,EAAE;QAClE,IAAI,KAAK,GAAG,IAAI,CAAC,KAAK,CAAC,SAAS,GAAG,CAAC,KAAK,GAAG,CAAC,CAAC,GAAG,SAAS,CAAC,CAAC;QAC5D,IAAI,OAAO,GAAG,SAAS,IAAI,KAAK,GAAG,OAAO,EAAE,CAAC;YACzC,KAAK,GAAG,OAAO,CAAC;QACpB,CAAC;aAAM,IAAI,SAAS,GAAG,OAAO,IAAI,KAAK,GAAG,OAAO,EAAE,CAAC;YAChD,KAAK,GAAG,OAAO,CAAC;QACpB,CAAC;QACD,OAAO,KAAK,CAAC;IACjB,CAAC,CAAC,CAAC;AACP,CAAC,CAAC;AAEF,MAAM,CAAC,MAAM,UAAU,GAAG,CAAC,KAAa,EAAE,EAAE;IACxC,IAAI,KAAK,KAAK,CAAC,IAAI,KAAK,KAAK,CAAC,EAAE,CAAC;QAC7B,KAAK,GAAG,KAAK,GAAG,CAAC,CAAC;IACtB,CAAC;IACD,OAAO,KAAK,CAAC;AACjB,CAAC,CAAA"}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "node-poweredup",
-  "version": "10.0.0",
+  "version": "10.1.0",
   "description": "A Javascript module to interface with LEGO Powered Up components.",
   "homepage": "https://github.com/nathankellenicki/node-poweredup/",
   "type": "module",

package/src/consts.ts

@@ -703,3 +703,47 @@ export enum MarioColor {
     BROWN = 0x6a00,
     CYAN = 0x4201,
 }
+
+
+/**
+ * @typedef TiltDirection
+ * @param {number} NEUTRAL 0
+ * @param {number} BACKWARD 3
+ * @param {number} RIGHT 5
+ * @param {number} LEFT 7
+ * @param {number} FORWARD 9
+ * @param {number} UNKNOWN 10
+ */
+export enum TiltDirection {
+    NEUTRAL = 0,
+    BACKWARD = 3,
+    RIGHT = 5,
+    LEFT = 7,
+    FORWARD = 9,
+    UNKNOWN = 10,
+}
+
+
+/**
+ * @typedef CommandFeedback
+ * @param {number} TRANSMISSION_PENDING 0x00 waiting for previous commands to complete transmission or execution
+ * @param {number} TRANSMISSION_BUSY 0x10 waiting for device to acknowledge reception
+ * @param {number} TRANSMISSION_DISCARDED 0x44 interrupt command has been recieved or device disconnected
+ * @param {number} EXECUTION_PENDING 0x20 device is waiting for previous command to complete
+ * @param {number} EXECUTION_BUSY 0x21 device is executing the command
+ * @param {number} EXECUTION_DISCARDED 0x24 device discarded the command e.g. due to interrupt
+ * @param {number} EXECUTION_COMPLETED 0x22 device reported successful completion of command
+ * @param {number} FEEDBACK_MISSING 0x66 device disconnected or failed to report feedback
+ * @param {number} FEEDBACK_DISABLED 0x26 feedback not implemented for this command
+ */
+export enum CommandFeedback {
+    TRANSMISSION_PENDING = 0x00,
+    TRANSMISSION_BUSY = 0x10,
+    TRANSMISSION_DISCARDED = 0x44,
+    EXECUTION_PENDING = 0x20,
+    EXECUTION_BUSY = 0x21,
+    EXECUTION_DISCARDED = 0x24,
+    EXECUTION_COMPLETED = 0x22,
+    FEEDBACK_MISSING = 0x66,
+    FEEDBACK_DISABLED = 0x26,
+}

package/src/devices/absolutemotor.ts

@@ -39,34 +39,29 @@ export class AbsoluteMotor extends TachoMotor {
      * Rotate a motor by a given angle.
      * @param {number} angle Absolute position the motor should go to (degrees from 0).
      * @param {number} [speed=100] For forward, a value between 1 - 100 should be set. For reverse, a value between -1 to -100.
-     * @returns {Promise} Resolved upon successful completion of command (ie. once the motor is finished).
+     * @param {boolean} [interrupt=false] If true, previous commands are discarded.
+     * @returns {Promise<CommandFeedback>} Resolved upon completion of command (i.e. once the motor is finished).
      */
-    public gotoAngle (angle: [number, number] | number, speed: number = 100) {
+    public gotoAngle (angle: [number, number] | number, speed: number = 100, interrupt: boolean = false) {
         if (!this.isVirtualPort && angle instanceof Array) {
             throw new Error("Only virtual ports can accept multiple positions");
         }
         if (this.isWeDo2SmartHub) {
             throw new Error("Absolute positioning is not available on the WeDo 2.0 Smart Hub");
         }
-        this.cancelEventTimer();
-        return new Promise<void>((resolve) => {
-            if (speed === undefined || speed === null) {
-                speed = 100;
-            }
-            let message;
-            if (angle instanceof Array) {
-                message = Buffer.from([0x81, this.portId, 0x11, 0x0e, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, mapSpeed(speed), this._maxPower, this._brakeStyle, this.useProfile()]);
-                message.writeInt32LE(normalizeAngle(angle[0]), 4);
-                message.writeInt32LE(normalizeAngle(angle[1]), 8);
-            } else {
-                message = Buffer.from([0x81, this.portId, 0x11, 0x0d, 0x00, 0x00, 0x00, 0x00, mapSpeed(speed), this._maxPower, this._brakeStyle, this.useProfile()]);
-                message.writeInt32LE(normalizeAngle(angle), 4);
-            }
-            this.send(message);
-            this._finishedCallbacks.push(() => {
-                return resolve();
-            });
-        });
+        if (speed === undefined || speed === null) {
+            speed = 100;
+        }
+        let message;
+        if (angle instanceof Array) {
+            message = Buffer.from([0x0e, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, 0x00, mapSpeed(speed), this._maxPower, this._brakeStyle, this.useProfile()]);
+            message.writeInt32LE(normalizeAngle(angle[0]), 1);
+            message.writeInt32LE(normalizeAngle(angle[1]), 5);
+        } else {
+            message = Buffer.from([0x0d, 0x00, 0x00, 0x00, 0x00, mapSpeed(speed), this._maxPower, this._brakeStyle, this.useProfile()]);
+            message.writeInt32LE(normalizeAngle(angle), 1);
+        }
+        return this.sendPortOutputCommand(message, interrupt);
     }
 
 
@@ -75,10 +70,10 @@ export class AbsoluteMotor extends TachoMotor {
      *
      * Real zero is marked on Technic angular motors (SPIKE Prime). It is also available on Technic linear motors (Control+) but is unmarked.
      * @param {number} [speed=100] Speed between 1 - 100. Note that this will always take the shortest path to zero.
-     * @returns {Promise} Resolved upon successful completion of command (ie. once the motor is finished).
+     * @returns {Promise<CommandFeedback>} Resolved upon completion of command (i.e. once the motor is finished).
      */
     public gotoRealZero (speed: number = 100) {
-        return new Promise<void>((resolve) => {
+        return new Promise<Consts.CommandFeedback>((resolve) => {
             const oldMode = this.mode;
             let calibrated = false;
             this.on("absolute", async ({ angle }) => {
@@ -93,7 +88,7 @@ export class AbsoluteMotor extends TachoMotor {
                     if (oldMode) {
                         this.subscribe(oldMode);
                     }
-                    return resolve();
+                    return resolve(Consts.CommandFeedback.FEEDBACK_DISABLED);
                 }
             });
             this.requestUpdate();
@@ -103,14 +98,12 @@ export class AbsoluteMotor extends TachoMotor {
 
     /**
      * Reset zero to current position
-     * @returns {Promise} Resolved upon successful completion of command (ie. once the motor is finished).
+     * @param {boolean} [interrupt=false] If true, previous commands are discarded.
+     * @returns {Promise<CommandFeedback>} Resolved upon completion of command (i.e. once the motor is finished).
      */
-    public resetZero () {
-        return new Promise<void>((resolve) => {
-            const data = Buffer.from([0x81, this.portId, 0x11, 0x51, 0x02, 0x00, 0x00, 0x00, 0x00]);
-            this.send(data);
-            return resolve();
-        });
+    public resetZero (interrupt: boolean = false) {
+        const data = Buffer.from([0x51, 0x02, 0x00, 0x00, 0x00, 0x00]);
+        return this.sendPortOutputCommand(data, interrupt);
     }
 
 

package/src/devices/basicmotor.ts

@@ -21,13 +21,11 @@ export class BasicMotor extends Device {
     /**
      * Set the motor power.
      * @param {number} power For forward, a value between 1 - 100 should be set. For reverse, a value between -1 to -100. Stop is 0.
-     * @returns {Promise} Resolved upon successful issuance of the command.
+     * @param {boolean} [interrupt=false] If true, previous commands are discarded.
+     * @returns {Promise<CommandFeedback>} Resolved upon completion of command.
      */
-    public setPower (power: number, interrupt: boolean = true) {
-        if (interrupt) {
-            this.cancelEventTimer();
-        }
-        return this.writeDirect(0x00, Buffer.from([mapSpeed(power)]));
+    public setPower (power: number, interrupt: boolean = false) {
+        return this.writeDirect(0x00, Buffer.from([mapSpeed(power)]), interrupt);
     }
 
 
@@ -36,37 +34,33 @@ export class BasicMotor extends Device {
      * @param {number} fromPower For forward, a value between 1 - 100 should be set. For reverse, a value between -1 to -100. Stop is 0.
      * @param {number} toPower For forward, a value between 1 - 100 should be set. For reverse, a value between -1 to -100. Stop is 0.
      * @param {number} time How long the ramp should last (in milliseconds).
-     * @returns {Promise} Resolved upon successful completion of command.
+     * @returns {Promise<CommandFeedback>} Resolved upon completion of command.
      */
     public rampPower (fromPower: number, toPower: number, time: number) {
-        this.cancelEventTimer();
-        return new Promise((resolve) => {
-            calculateRamp(this, fromPower, toPower, time)
-            .on("changePower", (power) => {
-                this.setPower(power, false);
-            })
-            .on("finished", resolve);
+        const powerValues = calculateRamp(fromPower, toPower, time);
+        powerValues.forEach(value => {
+            this.setPower(value);
+            this.addPortOutputSleep(Math.round(time/powerValues.length));
         });
+        return this.setPower(toPower);
     }
 
 
     /**
-     * Stop the motor.
-     * @returns {Promise} Resolved upon successful issuance of the command.
+     * Stop the motor. Previous commands that have not completed are discarded.
+     * @returns {Promise<CommandFeedback>} Resolved upon completion of command.
      */
     public stop () {
-        this.cancelEventTimer();
-        return this.setPower(0);
+        return this.setPower(0, true);
     }
 
 
     /**
-     * Brake the motor.
-     * @returns {Promise} Resolved upon successful issuance of the command.
+     * Brake the motor. Previous commands that have not completed are discarded.
+     * @returns {Promise<CommandFeedback>} Resolved upon completion of command.
      */
     public brake () {
-        this.cancelEventTimer();
-        return this.setPower(Consts.BrakingStyle.BRAKE);
+        return this.setPower(Consts.BrakingStyle.BRAKE, true);
     }
 
 

package/src/devices/colordistancesensor.ts

@@ -158,7 +158,7 @@ export class ColorDistanceSensor extends Device {
      *
      * NOTE: Calling this with channel 5-8 with switch off extended channel mode for this receiver.
      * @param {number} channel Channel number, between 1-8
-     * @returns {Promise} Resolved upon successful issuance of the command.
+     * @returns {Promise<CommandFeedback>} Resolved upon completion of the command.
      */
     public setPFExtendedChannel (channel: number) {
         let address = 0;
@@ -179,7 +179,7 @@ export class ColorDistanceSensor extends Device {
      * @param {number} channel Channel number, between 1-4
      * @param {string} output Outport port, "RED" (A) or "BLUE" (B)
      * @param {number} power -7 (full reverse) to 7 (full forward). 0 is stop. 8 is brake.
-     * @returns {Promise} Resolved upon successful issuance of the command.
+     * @returns {Promise<CommandFeedback>} Resolved upon completion of the command.
      */
     public setPFPower (channel: number, output: Output, power: number) {
         let address = 0;
@@ -202,7 +202,7 @@ export class ColorDistanceSensor extends Device {
      * @param {number} channel Channel number, between 1-4
      * @param {number} powerBlue -7 (full reverse) to 7 (full forward). 0 is stop. 8 is brake.
      * @param {number} powerRed -7 (full reverse) to 7 (full forward). 0 is stop. 8 is brake.
-     * @returns {Promise} Resolved upon successful issuance of the command.
+     * @returns {Promise<CommandFeedback>} Resolved upon completion of the command.
      */
     public startPFMotors (channel: number, powerBlue: number, powerRed: number) {
         let address = 0;
@@ -221,7 +221,7 @@ export class ColorDistanceSensor extends Device {
     /**
      * Send a raw Power Functions IR command
      * @param {Buffer} message 2 byte payload making up a Power Functions protocol command. NOTE: Only specify nibbles 1-3, nibble 4 should be zeroed.
-     * @returns {Promise} Resolved upon successful issuance of the command.
+     * @returns {Promise<CommandFeedback>} Resolved upon completion of the command.
      */
     public sendPFIRMessage (message: Buffer) {
         if (this.isWeDo2SmartHub) {
@@ -239,40 +239,34 @@ export class ColorDistanceSensor extends Device {
     /**
      * Set the color of the LED on the sensor via a color value.
      * @param {Color} color
-     * @returns {Promise} Resolved upon successful issuance of the command.
+     * @returns {Promise<CommandFeedback>} Resolved upon completion of the command.
      */
     public setColor (color: number | boolean) {
-        return new Promise<void>((resolve) => {
-            if (color === false) {
-                color = 0;
-            }
-            if (this.isWeDo2SmartHub) {
-                throw new Error("Setting LED color is not available on the WeDo 2.0 Smart Hub");
-            } else {
-                this.subscribe(Mode.LED);
-                this.writeDirect(0x05, Buffer.from([color as number]));
-            }
-            return resolve();
-        });
+        if (color === false) {
+            color = 0;
+        }
+        if (this.isWeDo2SmartHub) {
+            throw new Error("Setting LED color is not available on the WeDo 2.0 Smart Hub");
+        } else {
+            this.subscribe(Mode.LED);
+            return this.writeDirect(0x05, Buffer.from([color as number]));
+        }
     }
 
     /**
      * Set the distance count value.
      * @param {number} count distance count between 0 and 2^32
-     * @returns {Promise} Resolved upon successful issuance of the command.
+     * @returns {Promise<CommandFeedback>} Resolved upon completion of the command.
      */
     public setDistanceCount (count: number) {
-        return new Promise<void>((resolve) => {
-            if (this.isWeDo2SmartHub) {
-                throw new Error("Setting distance count is not available on the WeDo 2.0 Smart Hub");
-            } else {
-		const payload = Buffer.alloc(4);
-		payload.writeUInt32LE(count % 2**32);
-                // no need to subscribe, can be set in different mode
-                this.writeDirect(0x02, payload);
-            }
-            return resolve();
-        });
+        if (this.isWeDo2SmartHub) {
+            throw new Error("Setting distance count is not available on the WeDo 2.0 Smart Hub");
+        } else {
+            const payload = Buffer.alloc(4);
+            payload.writeUInt32LE(count % 2**32);
+            // no need to subscribe, can be set in different mode
+            return this.writeDirect(0x02, payload);
+        }
     }
 
     private _pfPowerToPWM (power: number) {