update uni handler

Author: semuadmin

INSTALLATION.md

@@ -243,11 +243,13 @@ TBC. Anyone conversant with PowerShell is welcome to contribute an equivalent in
 
 ## <a name="troubleshooting">Troubleshooting</a>
 
-1. The optional `rasterio` package is only available as an [sdist](#sdist-vs-wheel) on some Linux / ARM platforms (including Raspberry Pi OS), and the consequent [GDAL](https://gdal.org/en/stable/) build and configuration requirements may be problematic e.g. `WARNING:root:Failed to get options via gdal-config`. Refer to [rasterio installation](https://rasterio.readthedocs.io/en/stable/installation.html) and [GDAL installation](https://gdal.org/en/stable/) for assistance but - *be warned* - the process is **not** for the faint-hearted.
+1. `[Errno 13] could not open port /dev/tty**** [Errno 13] permission denied /dev/tty****` error on Linux when attempting to access serial port. Refer to [User Privileges](#userpriv).
+
+1. The optional `rasterio` package is only available as an [sdist](#sdist-vs-wheel) on some Linux / ARM platforms and the consequent [GDAL](https://gdal.org/en/stable/) build and configuration requirements may be problematic e.g. `WARNING:root:Failed to get options via gdal-config`. Refer to [rasterio installation](https://rasterio.readthedocs.io/en/stable/installation.html) and [GDAL installation](https://gdal.org/en/stable/) for assistance but - *be warned* - the process is **not** for the faint-hearted.
   
    In practice, `rasterio` is only required for automatic extents detection in PyGPSClient's Import Custom Map facility. As a workaround, extents can be entered manually, or you can try importing maps on a different platform and then copy-and-paste the relevant `usermaps_l` extents configuration to the target platform.
 
-1. The optional `cryptography` package is only available as an [sdist](#sdist-vs-wheel) on some 32-bit Linux / ARM platforms (including Raspberry Pi OS), and the consequent OpenSSL build requirements may be problematic e.g. `Building wheel for cryptography (PEP 517): finished with status 'error'`. Refer to [cryptography installation](https://github.com/semuconsulting/pyspartn/blob/main/cryptography_installation/README.md) for assistance.
+1. The optional `cryptography` package is only available as an [sdist](#sdist-vs-wheel) on some 32-bit Linux / ARM platforms and the consequent OpenSSL build requirements may be problematic e.g. `Building wheel for cryptography (PEP 517): finished with status 'error'`. Refer to [cryptography installation](https://github.com/semuconsulting/pyspartn/blob/main/cryptography_installation/README.md) for assistance.
 
 ---
 ## <a name="license">License</a>

README.md

@@ -457,7 +457,7 @@ The `pygnssutils` and `pyubxutils` libraries which underpin many of the function
 
 For further details, refer to the `pygnssutils` homepage at [https://github.com/semuconsulting/pygnssutils](https://github.com/semuconsulting/pygnssutils) or `pyubxutils` homepage at [https://github.com/semuconsulting/pyubxutils](https://github.com/semuconsulting/pyubxutils).
 
---
+---
 ## <a name="knownissues">Known Issues</a>
 
 1. Most budget USB-UART adapters (e.g. FT232, CH345, CP2102) have a bandwidth limit of around 3MB/s and may not work reliably above 115200 baud, even if the receiver supports higher baud rates. If you're using an adapter and notice significant message corruption, try reducing the baud rate to a maximum 115200.

pyproject.toml

@@ -49,7 +49,12 @@ classifiers = [
     "Topic :: Scientific/Engineering :: GIS",
 ]
 
-dependencies = ["requests>=2.28.0", "Pillow>=9.0.0", "pygnssutils>=1.1.22"]
+dependencies = [
+    "requests>=2.28.0",
+    "Pillow>=9.0.0",
+    "pygnssutils>=1.1.22",
+    "pyunigps>=0.1.4",
+]
 
 [project.scripts]
 pygpsclient = "pygpsclient.__main__:main"

src/pygpsclient/chart_frame.py

@@ -40,6 +40,7 @@
     BGCOL,
     ERRCOL,
     FGCOL,
+    INFOCOL,
     PLOTCOLS,
     READONLY,
     TRACEMODE_WRITE,
@@ -48,6 +49,7 @@
     WIDGETU6,
 )
 from pygpsclient.helpers import time2str
+from pygpsclient.strings import CONTENTCOPIED
 
 OL_WID = 1
 LBLCOL = "white"
@@ -623,6 +625,7 @@ def _on_clipboard(self, event):  # pylint: disable=unused-argument
         self.__master.clipboard_clear()
         self.__master.clipboard_append(csv)
         self.__master.update()
+        self.__app.status_label = (CONTENTCOPIED.format("chart"), INFOCOL)
 
     def _on_resize(self, event):  # pylint: disable=unused-argument
         """

src/pygpsclient/console_frame.py

@@ -43,9 +43,10 @@
     FORMAT_BOTH,
     FORMAT_HEXSTR,
     FORMAT_HEXTAB,
+    INFOCOL,
     WIDGETU3,
 )
-from pygpsclient.strings import HALTTAGWARN
+from pygpsclient.strings import CONTENTCOPIED, HALTTAGWARN
 
 HALT = "HALT"
 CONSOLELINES = 20
@@ -242,6 +243,7 @@ def _on_clipboard(self, event):  # pylint: disable=unused-argument
         self.__master.clipboard_clear()
         self.__master.clipboard_append(self.txt_console.get("1.0", END))
         self.__master.update()
+        self.__app.status_label = (CONTENTCOPIED.format("console"), INFOCOL)
 
     def _on_resize(self, event):  # pylint: disable=unused-argument
         """

src/pygpsclient/globals.py

@@ -250,8 +250,8 @@
 TOPIC_RXM = "/pp/ubx/0236/ip"
 TRACK = "track"
 TRACEMODE_WRITE = "write"
-TTYOK = ("OK", "$R:", "RESPONSE: OK")
-TTYERR = ("ERROR", "$R?", "RESPONSE: PARSING FAILD")
+TTYOK = ("OK", "$R:")
+TTYERR = ("ERROR", "$R?", "FAIL", "CAN'T FOUND DEVICE")
 TTYMARKER = "TTY<<"
 UBXPRESETS = "ubxpresets"
 UBXSIMULATOR = "ubxsimulator"

src/pygpsclient/init_presets.py

@@ -100,8 +100,10 @@
         "Unicore UM98n Restore Factory Defaults CONFIRM; freset",
         "Unicore UM98n Save Current Configuration to NVM CONFIRM; saveconfig",
         "Unicore UM98n Force COLD reset CONFIRM; reset",
-        "Unicore UM98n Enable NMEA on current port; gpgsa 1; gpgga 1; gpgll 1; gpgsv 4; gpvtg 1; gprmc 1",
-        "Unicore UM98n Enable UNI on current port; PVTSLNB 1; SATSINFOB 4; BESTNAV 1; STADOP 1",
+        "Unicore UM98n Enable basic NMEA on current port; gpgsa 1; gpgga 1; gpgll 1; gpgsv 4; gpvtg 1; gprmc 1",
+        "Unicore UM98n Enable basic UNI on current port; PVTSLNB 1; SATSINFOB 4; BESTNAVB 1; STADOPB 1",
+        "Unicore UM98n Disable basic NMEA on current port; unlog gpgsa; unlog gpgga; unlog gpgll; unlog gpgsv; unlog gpvtg; unlog gprmc",
+        "Unicore UM98n Disable basic UNI on current port; unlog PVTSLNB; unlog SATSINFOB; unlog BESTNAVB; unlog STADOPB",
         "Unicore UM98n Query Version; version",
         "Unicore UM98n Query Config; config",
         "Unicore UM98n Stop All Output on COM1; unlog COM1",

src/pygpsclient/stream_handler.py

@@ -62,7 +62,14 @@ class to read and parse incoming data from the receiver. It places
 from pyrtcm import RTCMMessageError, RTCMParseError, RTCMStreamError
 from pysbf2 import SBFMessageError, SBFParseError, SBFStreamError
 from pyubx2 import ERR_LOG, UBXMessageError, UBXParseError, UBXStreamError
-from pyubxutils import UBXSimulator
+
+try:
+    from pyubxutils import UBXSimulator
+
+    HASUBXUTILS = True
+except (ImportError, ModuleNotFoundError):
+    HASUBXUTILS = False
+
 from serial import Serial, SerialException, SerialTimeoutException
 
 from pygpsclient.globals import (
@@ -148,7 +155,7 @@ def _read_thread(
         conntype = settings["conntype"]
         inactivity_timeout = settings.get("inactivity_timeout", 0)
         if conntype == CONNECTED:
-            if settings["serial_settings"].port == UBXSIMULATOR:
+            if HASUBXUTILS and settings["serial_settings"].port == UBXSIMULATOR:
                 conntype = CONNECTED_SIMULATOR
         ttydelay = (
             self.__app.configuration.get("ttydelay_b")
@@ -238,7 +245,7 @@ def _read_thread(
                         inactivity_timeout,
                     )
 
-            elif conntype == CONNECTED_SIMULATOR:
+            elif HASUBXUTILS and conntype == CONNECTED_SIMULATOR:
                 with UBXSimulator() as stream:
                     if settings["protocol"] & TTY_PROTOCOL:
                         self._readlooptty(

src/pygpsclient/strings.py

@@ -41,6 +41,7 @@
 CONFIGRXM = "{} polled, {} key(s) loaded"
 CONFIGTITLE = "Config File"
 CONFIRM = "CONFIRM"
+CONTENTCOPIED = "Contents of {} copied to clipboard"
 DGPSYES = "\u2713"  # tick symbol
 ENDOFFILE = "End of file reached"
 FILEOPENERROR = "Error opening file {}"

src/pygpsclient/uni_handler.py

@@ -74,13 +74,11 @@ def process_data(self, raw_data: bytes, parsed_data: UNIMessage):
         if raw_data is None:
             return
         # self.logger.debug(f"data received {parsed_data.identity}")
-        self.__app.gnss_status.utc = wnotow2date(
-            parsed_data.wno, int(parsed_data.tow / 1000), parsed_data.leapsecond
-        )  # datetime.time
+        self._process_utc(parsed_data)
         if parsed_data.identity in ("BESTNAV", "BESTNAVH"):
             self._process_BESTNAV(parsed_data)
-        elif parsed_data.identity in ("PVTSLT",):
-            self._process_PVTSLT(parsed_data)
+        elif parsed_data.identity in ("PVTSLN",):
+            self._process_PVTSLN(parsed_data)
         elif parsed_data.identity in (
             "ADRNAV",
             "ADRNAVH",
@@ -96,6 +94,17 @@ def process_data(self, raw_data: bytes, parsed_data: UNIMessage):
         elif parsed_data.identity in ("STADOP", "ADRDOP", "PPPDOP"):
             self._process_STADOP(parsed_data)
 
+    def _process_utc(self, data: UNIMessage):
+        """
+        Process wno, tow and leapsecond from UNI message header.
+
+        :param UNIMessage data: parsed message
+        """
+
+        self.__app.gnss_status.utc = wnotow2date(
+            data.wno, int(data.tow / 1000), data.leapsecond
+        )
+
     def _process_pos(self, lat: float, lon: float, hmsl: float, undulation: float):
         """
         Process lat/lon/hmsl/hae.
@@ -119,8 +128,8 @@ def _process_fix(self, postype: int):
         """
 
         self.__app.gnss_status.fix = fix2desc("BESTNAV", postype)
-        self.__app.gnss_status.diff_corr = ("RTK" in self.__app.gnss_status.fix) or (
-            "PPP" in self.__app.gnss_status.fix
+        self.__app.gnss_status.diff_corr = (
+            sum(c in self.__app.gnss_status.fix for c in ("RTK", "PPP", "SBAS")) > 0
         )
 
     def _process_BESTNAV(self, data: UNIMessage):
@@ -138,11 +147,11 @@ def _process_BESTNAV(self, data: UNIMessage):
         self.__app.gnss_status.track = data.trkgnd
         self.__app.gnss_status.diff_station = data.stationid
 
-    def _process_PVTSLT(self, data: UNIMessage):
+    def _process_PVTSLN(self, data: UNIMessage):
         """
-        Process PVTSLT sentence -  Navigation position velocity time solution.
+        Process PVTSLN sentence -  Navigation position velocity time solution.
 
-        :param UNIMessage data: PVTSLT parsed message
+        :param UNIMessage data: PVTSLN parsed message
         """
 
         self._process_pos(
@@ -154,7 +163,7 @@ def _process_PVTSLT(self, data: UNIMessage):
         self.__app.gnss_status.track = data.headingdegree
         self.__app.gnss_status.pdop = data.pdop
         self.__app.gnss_status.hdop = data.hdop
-        self.__app.gnss_status.diff_age = data.bestpos_diffage
+        self.__app.gnss_status.diff_age = data.bestposdiffage
 
     def _process_ADRNAV(self, data: UNIMessage):
         """