fix: minimum lux value and duplicate serial numbers

Author: jasper-seinhorst

package-lock.json

@@ -1,7 +1,7 @@
 {
   "name": "homebridge-omnik",
   "displayName": "Omnik Inverter",
-  "version": "1.2.0",
+  "version": "1.2.1",
   "description": "Add your Omnik-Inverter to Homekit",
   "license": "Apache-2.0",
   "author": "Jasper Seinhorst",

package.json

@@ -10,14 +10,14 @@ export default class CurrentPowerProduction implements OmnikAccessory {
     this.accessory.getService(this.Service.AccessoryInformation)
       .setCharacteristic(this.Characteristic.Manufacturer, 'Omnik')
       .setCharacteristic(this.Characteristic.Model, device.name)
-      .setCharacteristic(this.Characteristic.SerialNumber, device.serial);
-
-
+      .setCharacteristic(this.Characteristic.SerialNumber, `${device.serial}-power-production`);
     this.powerService = this.accessory.getService(this.Service.LightSensor) || this.accessory.addService(this.Service.LightSensor);
   }
 
   public beat(powerProduction: PowerProduction) {
+    const minimumValue = 0.0001;
     const consumption = powerProduction.now;
-    this.powerService.setCharacteristic(this.Characteristic.CurrentAmbientLightLevel, consumption);
+    const newValue = consumption > minimumValue ? consumption : minimumValue;
+    this.powerService.setCharacteristic(this.Characteristic.CurrentAmbientLightLevel, newValue);
   }
 }

src/Accessories/CurrentPowerProduction.ts

@@ -10,14 +10,14 @@ export default class TodayYield implements OmnikAccessory {
     this.accessory.getService(this.Service.AccessoryInformation)
       .setCharacteristic(this.Characteristic.Manufacturer, 'Omnik')
       .setCharacteristic(this.Characteristic.Model, device.name)
-      .setCharacteristic(this.Characteristic.SerialNumber, device.serial);
-
-
+      .setCharacteristic(this.Characteristic.SerialNumber, `${device.serial}-today-yield`);
     this.powerService = this.accessory.getService(this.Service.LightSensor) || this.accessory.addService(this.Service.LightSensor);
   }
 
   public beat(powerProduction: PowerProduction) {
+    const minimumValue = 0.0001;
     const consumption = powerProduction.today / 100;
-    this.powerService.setCharacteristic(this.Characteristic.CurrentAmbientLightLevel, consumption);
+    const newValue = consumption > minimumValue ? consumption : minimumValue;
+    this.powerService.setCharacteristic(this.Characteristic.CurrentAmbientLightLevel, newValue);
   }
 }

src/Accessories/TodayYield.ts

@@ -10,14 +10,14 @@ export default class TotalYield implements OmnikAccessory {
     this.accessory.getService(this.Service.AccessoryInformation)
       .setCharacteristic(this.Characteristic.Manufacturer, 'Omnik')
       .setCharacteristic(this.Characteristic.Model, device.name)
-      .setCharacteristic(this.Characteristic.SerialNumber, device.serial);
-
-
+      .setCharacteristic(this.Characteristic.SerialNumber, `${device.serial}-total-yield`);
     this.powerService = this.accessory.getService(this.Service.LightSensor) || this.accessory.addService(this.Service.LightSensor);
   }
 
   public beat(powerProduction: PowerProduction) {
+    const minimumValue = 0.0001;
     const consumption = powerProduction.total / 10;
-    this.powerService.setCharacteristic(this.Characteristic.CurrentAmbientLightLevel, consumption);
+    const newValue = consumption > minimumValue ? consumption : minimumValue;
+    this.powerService.setCharacteristic(this.Characteristic.CurrentAmbientLightLevel, newValue);
   }
 }

src/Accessories/TotalYield.ts

@@ -9,12 +9,11 @@ export class OmnikPlugin implements DynamicPlatformPlugin {
   public readonly Service: typeof Service = this.api.hap.Service;
   public readonly Characteristic: typeof Characteristic = this.api.hap.Characteristic;
   public readonly accessories: PlatformAccessory[] = [];
-  private heartBeatInterval;
+  private heartBeatInterval: number;
   private devices: OmnikAccessory[] = [];
   private omnikApi: OmnikApi;
   private device: OmnikDevice;
 
-
   constructor(public readonly log: Logger, public readonly config: PlatformConfig, public readonly api: API) {
     this.heartBeatInterval = (config.pollInterval || 5) * 60 * 1000; // minutes to miliseconds
     this.api.on('didFinishLaunching', () => {