Resolve a few bugs (#22)

* Change step to 500 as with 50 it is very difficult to find 0

* Update release version

* Protect against serial number being not set

* Prevent values lower than min or higher than max

Author: rjpearce

.github/workflows/docker-image.yml

@@ -13,7 +13,7 @@ on:
       - "main"
 
 env: 
-  RELEASE: 3.0.2
+  RELEASE: 3.0.3
 
 jobs:
   docker:

sofar-hyd-ep.json

@@ -276,7 +276,7 @@
 				"name": "Desired Power",
 				"min": -5000,
 				"max": 5000,
-				"step": 50,
+				"step": 500,
 				"initial": 0,
 				"mode": "slider",
 				"value_template": "{{ value_json.desired_power }}",
@@ -762,7 +762,9 @@
 				"state_class": "measurement",
 				"entity_category": "diagnostic",
 				"value_template": "{{ value_json.battery_soc }}"
-			}
+			},
+			"min": 0,
+			"max": 100
 		},
 		{
 			"name": "battery_soh",

sofar2mqtt-v2.py

@@ -225,6 +225,14 @@ def read_and_publish(self):
                 # Inverter will return maximum 16-bit integer value when data not available (eg. grid usage when grid down)
                 if value == 65535:
                     value = 0
+                if 'min' in register:
+                    if value < register['min']:
+                        logging.error(f"Value for {register['name']}: {str(value)} is lower than min allowed value: {register['min']}. Ignoring value")
+                        continue
+                if 'max' in register:
+                    if value > register['max']:
+                        logging.error(f"Value for {register['name']}: {str(value)} is greater than max allowed value: {register['max']}. Ignoring value")
+                        continue
                 if 'function' in register:
                     if register['function'] == 'multiply':
                         value = value * register['factor']
@@ -283,14 +291,18 @@ def publish_state(self):
         time.sleep(self.refresh_interval)
 
     def publish_mqtt_discovery(self):
+        if 'serial_number' not in self.data:
+            logging.error("Serial number could not be determined, skipping publish")
+            return False
+
         sn = self.data['serial_number']
         payload = {
             "device": {
                "identifiers": [f"sofar2mqtt_python_bridge_{sn}"],
                "manufacturer": "Sofar2Mqtt-Python",
                "model": "Bridge",
                "name": "Sofar2Mqtt Python Bridge",
-               "sw_version": "1.35.3"
+               "sw_version": "3.0.3"
             },
             "device_class": "connectivity",
             "entity_category": "diagnostic",