update config panels

Author: semuadmin

.vscode/settings.json

@@ -21,7 +21,6 @@
     "telemetry.feedback.enabled": false,
     "python.analysis.addHoverSummaries": false,
     "python-envs.defaultEnvManager": "ms-python.python:venv",
-    "python-envs.pythonProjects": [],
     "python.venvPath": "${userHome}/pygpsclient/bin/python3",
     "python.systemPath": "/usr/local/bin/python3.14",
     "python.defaultInterpreterPath": "${userHome}/pygpsclient/bin/python3",

README.md

@@ -1,5 +1,16 @@
 # PyGPSClient Release Notes
 
+### RELEASE 1.6.5
+
+FIXES:
+1. Fix custom map import spurious validation error.
+
+ENHANCEMENTS:
+1. Add preset description entry field to Configuration Command Recorder import facility.
+1. Make TTY Presets dialog fully resizeable.
+1. Update NMEA config panel to allow preset commands to be edited or entered manually before sending.
+1. Update NMEA config panel to recognise correct PAIR responses for some Quectel set and poll commands (e.g. a PAIR864 Set baud rate command corresponds to a PAIR865 poll response).
+
 ### RELEASE 1.6.4
 
 FIXES:
@@ -8,7 +19,7 @@ FIXES:
 
 ENHANCEMENTS:
 1. Improve Base Station receiver configuration handling in the NTRIP Caster/Socket Server dialog.
-1. Add connected device descriptor to status bar (e.g. "u-blox ZED-F9P", "Unicore UM981S", "Septentrio mosaic-X5" or "Quectel LG290AG03"). **FYI** This is based on a series of query messages (*one for each enabled protocol*) sent approximately 3 seconds after the connection is started (*you may see 'unknown protocol' warnings in response to some of these messages; these can be ignored)*. Failing this, a generic descriptor is displayed on receipt of a message protocol unique to a particular manufacturer (*e.g. "u-blox" on receipt of a UBX message, "Unicore" on receipt of a UNI message, etc.*). Note that some (mainly older) devices may not return a meaningful descriptor, in which case "N/A" will be displayed.
+1. Add connected device descriptor to status bar (e.g. "u-blox ZED-F9P", "Unicore UM981S", "Septentrio mosaic-X5" or "Quectel LG290AG03"). **FYI** This is based on a poll of hardware information messages (*one for each enabled protocol*) sent approximately 3 seconds after the connection is started (*you may see 'unknown protocol' warnings in response to some of these messages; these can be ignored)*. Failing this, a generic descriptor is displayed on receipt of a message protocol unique to a particular manufacturer (*e.g. "u-blox" on receipt of a UBX message, "Unicore" on receipt of a UNI message, etc.*). Note that some (mainly older) devices may not return a meaningful descriptor, in which case "N/A" will be displayed. Note also that some receivers will not output hardware information or other status messages at low baud rates (< 38,400)
 1. Minor updates to serial port configuration panel (additional timeout values).
 1. Mininum pynmeagps version updated to v1.1.1.
 

RELEASE_NOTES.md

@@ -54,7 +54,7 @@ dependencies = [
     "Pillow>=9.0.0",
     "pygnssutils>=1.1.22",
     "pyunigps>=0.2.0",
-    "pynmeagps>=1.1.1",
+    "pynmeagps>=1.1.2",
 ]
 
 [project.scripts]

images/app.png

@@ -8,4 +8,4 @@
 :license: BSD 3-Clause
 """
 
-__version__ = "1.6.4"
+__version__ = "1.6.5"

images/nmeaconfig_widget.png

@@ -97,7 +97,7 @@ def __init__(self):
             DLGTTTY: {
                 CLASS: TTYPresetDialog,
                 DLG: None,
-                RESIZE: False,
+                RESIZE: True,
             },
             DLGTRECORD: {
                 CLASS: RecorderDialog,

pyproject.toml

@@ -98,6 +98,8 @@
 )
 # alternative POLL dictionary names for where POLL command
 # doesn't correspond to SET (fudge for Quectel)
+# e.g. a PAIR864 (set baud rate) command corresponds to PAIR865 (poll baud rate)
+# (for PAIR commands, the poll is typically one digit higher than the set)
 ALT_POLL_NAMES = {
     "AIR050": "AIR051",
     "AIR058": "AIR059",
@@ -111,8 +113,8 @@
     "AIR100": "AIR101",
     "AIR104": "AIR105",
     "AIR400": "AIR401",
-    "AIR410": "AIR421",
-    "AIR420": "AIR411",
+    "AIR410": "AIR411",
+    "AIR420": "AIR421",
     "AIR432": "AIR433",
     "AIR434": "AIR435",
     "AIR436": "AIR437",
@@ -225,7 +227,7 @@ def _body(self):
             command=self._on_refresh,
             font=self.__app.font_md,
         )
-        self._lbl_command = Label(self, text="", width=30, anchor=W)
+        self._lbl_command = Label(self, text="", anchor=W)
         self._frm_container = Frame(self)
         self._can_container = Canvas(self._frm_container)
         self._frm_attrs = Frame(self._can_container)
@@ -246,31 +248,45 @@ def _do_layout(self):
         Layout widgets.
         """
 
-        self._lbl_cfg_dyn.grid(column=0, row=0, columnspan=4, padx=3, sticky=EW)
+        self._lbl_cfg_dyn.grid(column=0, row=0, columnspan=3, padx=3, pady=3, sticky=EW)
         self._lbx_cfg_cmd.grid(
-            column=0, row=1, columnspan=2, rowspan=6, padx=3, pady=3, sticky=EW
+            column=0, row=1, columnspan=1, rowspan=10, padx=3, pady=3, sticky=EW
+        )
+        self._scr_cfg_cmd.grid(column=1, row=1, rowspan=10, sticky=(N, S, W))
+        self._btn_send_command.grid(
+            column=2, row=1, ipadx=3, ipady=3, padx=3, pady=3, sticky=W
+        )
+        self._lbl_send_command.grid(
+            column=2, row=2, ipadx=3, ipady=3, padx=3, pady=3, sticky=W
+        )
+        self._btn_refresh.grid(
+            column=2, row=3, ipadx=3, ipady=3, padx=3, pady=3, sticky=W
+        )
+        self._lbl_command.grid(
+            column=0, row=11, columnspan=3, padx=3, pady=3, sticky=EW
         )
-        self._scr_cfg_cmd.grid(column=1, row=1, rowspan=6, sticky=(N, S, E))
-        self._btn_send_command.grid(column=3, row=1, ipadx=3, ipady=3, sticky=W)
-        self._lbl_send_command.grid(column=3, row=2, ipadx=3, ipady=3, sticky=W)
-        self._btn_refresh.grid(column=3, row=3, ipadx=3, ipady=3, sticky=W)
-        self._lbl_command.grid(column=0, row=7, columnspan=4, padx=3, sticky=EW)
         self._frm_container.grid(
-            column=0, row=8, columnspan=4, rowspan=15, padx=3, sticky=NSEW
+            column=0,
+            row=12,
+            columnspan=3,
+            rowspan=15,
+            padx=3,
+            pady=3,
+            sticky=NSEW,
         )
         self._can_container.grid(
-            column=0, row=0, columnspan=3, rowspan=15, padx=3, sticky=NSEW
+            column=0,
+            row=0,
+            columnspan=3,
+            rowspan=15,
+            padx=3,
+            pady=3,
+            sticky=NSEW,
         )
-        self._scr_container_ver.grid(column=3, row=0, rowspan=15, sticky=(N, S, E))
+        self._scr_container_ver.grid(column=3, row=0, rowspan=15, sticky=(N, S, W))
         self._scr_container_hor.grid(
-            column=0, row=15, columnspan=4, rowspan=15, sticky=EW
+            column=0, row=15, columnspan=3, rowspan=15, sticky=EW
         )
-
-        cols, rows = self.grid_size()
-        for i in range(cols):
-            self.grid_columnconfigure(i, weight=1)
-        for i in range(rows):
-            self.grid_rowconfigure(i, weight=1)
         self.option_add("*Font", self.__app.font_sm)
 
     def _attach_events(self):
@@ -406,7 +422,7 @@ def _do_poll_cfg(self, *args, **kwargs):  # pylint: disable=unused-argument
             return
 
         msg = penddlg = pendcfg = None
-        # use alternate names for some NMEA PAIR/PQTM POLL commands
+        # use alternate name for some NMEA PAIR/PQTM POLL commands
         cfg_id = ALT_POLL_NAMES.get(self._cfg_id, self._cfg_id)
         # set any POLL arguments to specified or default values
         # e.g. portid = "1", msgname="RMC"
@@ -477,23 +493,26 @@ def _do_poll_args(self, cfg_id: str) -> dict:
             pass
         return args
 
-    def update_status(self, msg: object):
+    def update_status(self, msg: NMEAMessage | UBXMessage):
         """
         UBXHandler or NMEAHandler module has received expected command response
         and forwarded it to this module, entry widgets are pre-populated with
         current configuration values and confirmation status is updated.
 
-        :param object msg: UBXMessage or NMEAMessage response
+        :param NMEAMessage | UBXMessage msg: UBXMessage or NMEAMessage response
         """
 
+        # self.logger.debug(f"{msg.identity=}")
         ok = False
-        # strip off any variant suffix from cfg_id
-        # e.g. "QTMCFGUART_CURR" -> "PQTMCFGUART"
-        cfg_id = (
-            "P" + self._cfg_id.rsplit("_", 1)[0]
-            if self._protocol == NMEA
-            else self._cfg_id
-        )
+        if self._protocol == NMEA:
+            # use alternate name for some NMEA PAIR/PQTM POLL commands
+            # e.g. a PAIR864 (set baud rate) command corresponds to PAIR865 (poll baud rate)
+            cfg_id = ALT_POLL_NAMES.get(self._cfg_id, self._cfg_id)
+            # strip off any variant suffix from cfg_id
+            # e.g. "QTMCFGUART_CURR" -> "PQTMCFGUART"
+            cfg_id = "P" + cfg_id.rsplit("_", 1)[0]
+        else:
+            cfg_id = self._cfg_id
 
         # if this message identity matches an expected response
         if msg.identity in (cfg_id, ACK, NAK):
@@ -530,13 +549,13 @@ def _clear_widgets(self):
             wdg.destroy()
             wdg = None
         Label(self._frm_attrs, text="Attribute", width=12, anchor=W).grid(
-            column=0, row=0, padx=3, sticky=(W)
+            column=0, row=0, padx=3, sticky=W
         )
         Label(self._frm_attrs, text="Value", width=20, anchor=W).grid(
-            column=1, row=0, padx=3, sticky=(W)
+            column=1, row=0, padx=3, sticky=W
         )
         Label(self._frm_attrs, text="Type", width=5, anchor=W).grid(
-            column=2, row=0, padx=3, sticky=(W)
+            column=2, row=0, padx=3, sticky=W
         )
 
     def _add_widgets(self, pdict: dict, row: int, index: int) -> int:

src/pygpsclient/_version.py

@@ -271,6 +271,8 @@
 VALDMY = 8
 VALLEN = 9
 VALBOOL = 10
+VALREGEX = 11
+VALCUSTOM = 12
 WAYPOINT = "waypoint"
 WIDGETU1 = (200, 200)  # small widget size
 WIDGETU2 = (300, 200)  # medium widget size