update device info parsing

Author: semuadmin

README.md

@@ -108,7 +108,7 @@ For more comprehensive installation instructions, please refer to [INSTALLATION.
 3. To connect to a GNSS receiver via USB or UART port, select the device from the listbox, set the appropriate serial connection parameters and click 
 ![connect icon](https://github.com/semuconsulting/PyGPSClient/blob/master/src/pygpsclient/resources/usbport-1-24.png?raw=true). The application will endeavour to pre-select a recognised GNSS/GPS device but this is platform and device dependent. Press the ![refresh](https://github.com/semuconsulting/PyGPSClient/blob/master/src/pygpsclient/resources/iconmonstr-refresh-6-16.png?raw=true) button to refresh the list of connected devices at any point. 
     - `Rate bps` (baud rate) is typically the only setting that might need adjusting, but tweaking the `timeout` setting may improve performance on certain platforms. 
-    - When the connection is first established, PyGPSClient will send various query messages (*one for each selected protocol*) to the receiver in an attempt to establish its model and firmware version. You may see a handful of 'unknown protocol' warnings in response to some of these queries - these can be disregarded. 
+    - When the connection is first established, PyGPSClient will poll various hardware information messages (*one for each selected protocol*) to the receiver in an attempt to establish its model and firmware version. You may see a handful of 'unknown protocol' warnings in response to some of these queries - these can be disregarded. **NB:** Some receivers will not output hardware information messages at low baud rates (<38,400).
     - If you get a permissions error on attempting to connect to a serial port e.g. `[Errno 13] permission denied /dev/ttyACM0`, refer to the [Installation Guidelines - User Privileges](https://github.com/semuconsulting/PyGPSClient/blob/master/INSTALLATION.md#user-privileges).
     - The `Msg Mode` parameter defaults to `GET` i.e., periodic or poll response messages *from* a receiver. If you wish to parse streams of command or poll messages being sent *to* a receiver, set the `Msg Mode` to `SET` or `POLL`. An optional serial or socket stream inactivity timeout can also be set (in seconds; 0 = no timeout).
 4. A custom user-defined serial port can also be passed via the json configuration file setting `"userport_s":`, via environment variable `PYGPSCLIENT_USERPORT` or as a command line argument `--userport`. A special userport value of "ubxsimulator" invokes the experimental [`pyubxutils.UBXSimulator`](https://github.com/semuconsulting/pyubxutils/blob/main/src/pyubxutils/ubxsimulator.py) utility to emulate a GNSS NMEA/UBX serial stream. 

src/pygpsclient/nmea_handler.py

@@ -403,16 +403,17 @@ def _process_QTMVERNO(self, data: NMEAMessage):
         :param pynmeagps.NMEAMessage data: parsed QTMVERNO sentence
         """
 
-        verdata = {}
-        verdata["swversion"] = "N/A"
-        verdata["hwversion"] = f"Quectel {data.verstr}"
-        verdata["fwversion"] = f"{data.builddate}-{data.buildtime}"
-        verdata["romversion"] = "N/A"
-        self.__app.gnss_status.version_data = verdata
+        self.__app.gnss_status.version_data["swversion"] = "N/A"
+        self.__app.gnss_status.version_data["hwversion"] = f"Quectel {data.verstr}"
+        self.__app.gnss_status.version_data["fwversion"] = (
+            f"{data.builddate}-{data.buildtime}"
+        )
+        self.__app.gnss_status.version_data["romversion"] = NA
+        self.__app.gnss_status.version_data["gnss"] = NA
 
         if self.__app.dialog(DLGTNMEA) is not None:
             self.__app.dialog(DLGTNMEA).update_pending(data)
-        self.__app.device_label = verdata["hwversion"]
+        self.__app.device_label = self.__app.gnss_status.version_data["hwversion"]
 
     def _process_QTMVER(self, data: NMEAMessage):
         """

src/pygpsclient/sbf_handler.py

@@ -118,14 +118,18 @@ def _process_ReceiverSetup(self, data: SBFMessage):
         :param SBFMessage data: ReceiverSetup message
         """
 
-        verdata = {}
         hwver = data.ProductName.decode(ASCII, errors=BSR).replace("\x00", "")
-        verdata["hwversion"] = f"Septentrio {hwver}"
-        verdata["fwversion"] = data.RxVersion.decode(ASCII, errors=BSR).replace(
-            "\x00", ""
+        self.__app.gnss_status.version_data["hwversion"] = f"Septentrio {hwver}"
+        self.__app.gnss_status.version_data["swversion"] = (
+            data.RxVersion.decode(ASCII, errors=BSR)
+            .replace("\x00", "")
+            .replace("Unknown", "")
         )
-        verdata["romversion"] = data.GNSSFWVersion.decode(ASCII, errors=BSR).replace(
-            "\x00", ""
+        self.__app.gnss_status.version_data["fwversion"] = (
+            data.GNSSFWVersion.decode(ASCII, errors=BSR)
+            .replace("\x00", "")
+            .replace("Unknown", "")
         )
-        self.__app.gnss_status.version_data = verdata
-        self.__app.device_label = verdata["hwversion"]
+        self.__app.gnss_status.version_data["romversion"] = NA
+        self.__app.gnss_status.version_data["gnss"] = NA
+        self.__app.device_label = self.__app.gnss_status.version_data["hwversion"]

src/pygpsclient/ubx_handler.py

@@ -20,7 +20,7 @@
 
 from pyubx2 import UBXMessage, itow2utc
 
-from pygpsclient.globals import GLONASS_NMEA, UTF8
+from pygpsclient.globals import BSR, GLONASS_NMEA, UTF8
 from pygpsclient.helpers import corrage2int, fix2desc, ned2vector, svid2gnssid
 from pygpsclient.strings import DLGTSERVER, DLGTSPARTN, DLGTUBX, NA
 from pygpsclient.widget_state import VISIBLE, WDGSIGNALS, WDGSPECTRUM, WDGSYSMON
@@ -152,13 +152,15 @@ def _process_MONVER(self, msg: UBXMessage):
         gnss_supported = ""
         hw_version = ""
         sw_version = getattr(msg, "swVersion", b"N/A")
-        sw_version = sw_version.replace(b"\x00", b"").decode(UTF8)
+        sw_version = sw_version.decode(UTF8, errors=BSR).replace("\x00", "")
         sw_version = sw_version.replace("ROM CORE", "ROM")
         sw_version = sw_version.replace("EXT CORE", "Flash")
 
         for i in range(9):
-            ext = getattr(msg, f"extension_{i+1:02d}", b"").decode(
-                UTF8, errors="ignore"
+            ext = (
+                getattr(msg, f"extension_{i+1:02d}", b"")
+                .decode(UTF8, errors=BSR)
+                .replace("\x00", "")
             )
             # ext = ext.replace(b"\x00", b"")
             exts.append(ext)
@@ -185,7 +187,7 @@ def _process_MONVER(self, msg: UBXMessage):
         if hw_version == "":
             hw_version = (
                 getattr(msg, "hwVersion", b"N/A")
-                .decode(UTF8, errors="ignore")
+                .decode(UTF8, errors=BSR)
                 .replace("\x00", "")
             )
         hw_version = f"u-blox {UBXMODELS.get(hw_version, hw_version)}"

src/pygpsclient/uni_handler.py

@@ -264,5 +264,7 @@ def _process_VERSION(self, data: UNIMessage):
             f"Unicore {DEVICE.get(data.device, data.device)}"
         )
         self.__app.gnss_status.version_data["swversion"] = data.swversion
-        self.__app.gnss_status.version_data["romversion"] = data.comptime
+        self.__app.gnss_status.version_data["fwversion"] = data.comptime
+        self.__app.gnss_status.version_data["romversion"] = NA
+        self.__app.gnss_status.version_data["gnss"] = NA
         self.__app.device_label = self.__app.gnss_status.version_data["hwversion"]