update NMEA preset panel

semuadmin · semuadmin · commit 2a976358dbee · 2026-03-02T16:34:58.000Z
diff --git a/README.md b/README.md
@@ -14,7 +14,7 @@
 [Mapquest API Key](#mapquestapi) |
 [User-defined Presets](#userdefined) |
 [CLI Utilities](#cli) |
-[Known Issues](#knownissues) |
+[Troubleshooting](#troubleshoot) |
 [License](#license) |
 [Author Information](#author)
 
@@ -220,7 +220,7 @@ The UBX Configuration Dialog currently provides the following UBX configuration
 1. Message Rate panel (CFG-MSG) sets message rates per port for UBX and NMEA messages (*legacy protocols only*). Message rate is relative to navigation solution frequency e.g. a message rate of '4' means 'every 4th navigation solution' (higher = less frequent).
 1. Configuration Interface widget (CFG-VALSET, CFG-VALDEL and CFG-VALGET) queries and sets configuration for *modern protocols only*.
 1. UBX Legacy Command configuration panel providing structured updates for a range of legacy CFG-* configuration commands (*legacy protocols only*). Note: 'X' (byte) type attributes can be entered as integers or hexadecimal strings e.g. 522125312 or 0x1f1f0000. Once a command is selected, the configuration is polled and the current values displayed. The user can then amend these values as required and send the updated configuration. Some polls require input arguments (e.g. portID) - these are highlighted and will be set at default values initially (e.g. portID = 0), but can be amended by the user and re-polled using the ![refresh](https://github.com/semuconsulting/PyGPSClient/blob/master/src/pygpsclient/resources/iconmonstr-refresh-lined-24.png?raw=true) button.
-1. Preset Commands widget supports a variety of user-defined UBX commands and queries - see [user defined presets](#userdefined).
+1. Preset Commands widget supports a variety of user-defined UBX commands and queries - see [user-defined presets](#userdefined).
 
 An icon to the right of each 'SEND' 
 ![send icon](https://github.com/semuconsulting/PyGPSClient/blob/master/src/pygpsclient/resources/iconmonstr-arrow-12-24.png?raw=true) button indicates the confirmation status of the configuration command; 
@@ -238,16 +238,17 @@ warning ![warning icon](https://github.com/semuconsulting/PyGPSClient/blob/maste
 - Receiver capable of being configured via proprietary NMEA sentences, connected to the workstation via USB or UART port.
 - The facility includes support for a wide range of Quectel LG and LC series receivers via `PQTM*`, `PSTM*` and `PAIR*` sentences¹ ². Additional types may be supported in the underlying NMEA parser library [pynmeagps](https://github.com/semuconsulting/pynmeagps) in later releases (*contributions welcome*).
 
-¹ Note that Quectel receivers implement a bewildering array of different configuration protocols (based on a mixture of proprietary `PQTM*`, `PSTM*` and `PAIR*` NMEA message types). Support depends on the *specific model variant and firmware version*, and several models (e.g. LG290 and LC29) exist in a wide range of variants. Refer to the GNSS Protocol Guide for your *specific* variant.
+   ¹ Note that Quectel receivers implement a bewildering array of different configuration protocols, based on a mixture of proprietary NMEA `PQTM*`, `PSTM*` and `PAIR*` message types. Implementation depends on the *specific model variant and firmware version*, and several models (e.g. LG290 and LC29) exist in a wide range of variants. Refer to the GNSS Protocol Guide for your *specific* variant for details on the available configuration commands.
 
-² Note that several Quectel configuration commands require a Hot Restart (PQTMHOT) *or* Save (PQTMSAVEPAR or PAIR513) and Reset (PQTMSRR) before taking effect, including for example PQTMCFGCNST (Enable/Disable Constellations), PQTMCFGFIX (Configure Fix Rate), PQTMCFGSAT (Configure Satellite Masks), PQTMCFGSIGNAL (Configure Signal Masks), PAIR050 (Set Fix Rate) and PAIR864 (Set Baud Rate). Devices (e.g. some LC variants) that don't implement a software reset command may have to be physically disconnected and reconnected.
+   ² Note that several Quectel configuration commands require a Hot Restart (PQTMHOT) *or* Save (PQTMSAVEPAR or PAIR513) and Reset (PQTMSRR) before taking effect, including for example PQTMCFGCNST (Enable/Disable Constellations), PQTMCFGFIX (Configure Fix Rate), PQTMCFGSAT (Configure Satellite Masks), PQTMCFGSIGNAL (Configure Signal Masks), PAIR050 (Set Fix Rate) and PAIR864 (Set Baud Rate). Devices (e.g. some LC variants) that don't implement a software reset command may have to be physically disconnected and reconnected.
 
 **Instructions:**
 
 The NMEA Configuration Dialog currently provides the following NMEA configuration panels:
 1. Version panel shows current device hardware/firmware versions (*Double-left-click to refresh*).
 1. Dynamic configuration panel providing structured updates for supported receivers e.g. Quectel LGSERIES via PQTM* sentences, or LCSERIES via PAIR* sentences. Once a command is selected, the configuration is polled and the current values displayed. The user can then amend these values as required and send the updated configuration. Some polls require input arguments (e.g. portid or msgname) - these are highlighted and will be set at default values initially (e.g. portid = 1), but can be amended by the user and re-polled using the ![refresh](https://github.com/semuconsulting/PyGPSClient/blob/master/src/pygpsclient/resources/iconmonstr-refresh-lined-24.png?raw=true) button.
 1. Preset Commands widget supports a variety of user-defined NMEA commands and queries - see [user defined presets](#userdefined).
+1. Preset commands, once selected, can be edited or overwritten in the 'Commands' field before sending, but commands must observe the format `<talker>; <message id>; <payload as comma-separated string>; <msgmode>` (e.g. `P; QTMCFGUART; W,115200; 1` - see [user-defined presets](#userdefined)).
 
 An icon to the right of each 'SEND' 
 ![send icon](https://github.com/semuconsulting/PyGPSClient/blob/master/src/pygpsclient/resources/iconmonstr-arrow-12-24.png?raw=true) button indicates the confirmation status of the configuration command; 
@@ -442,17 +443,21 @@ that this is a User variable rather than a System/Global variable.
 ---
 ## <a name="userdefined">User Defined Presets</a>
 
-The UBX, NMEA and TTY Configuration Dialogs include the facility to send user-defined configuration messages or message sequences to a compatible receiver. These can be set up by adding appropriate comma- or semicolon-delimited message descriptions and payload definitions to the `"ubxpresets_l"`, `"nmeapresets_l"` or `"ttypresets_l"` settings in your json configuration file (see [example provided](https://github.com/semuconsulting/PyGPSClient/blob/master/pygpsclient.json#L189)). The message definition comprises a free-format text description (*avoid embedded commas or semi-colons*) followed by one or more pyubx2 (UBX), pynmeagps (NMEA) or tty (ASCII) message constructors, e,g. 
+The UBX, NMEA and TTY Configuration Dialogs include the facility to send user-defined configuration messages or message sequences to a compatible receiver. These can be set up as follows: 
 
-- UBX - `<description>, <message class>, <message id>, <payload as hexadecimal string>, <msgmode>`
-- NMEA - `<description>; <talker>; <message id>; <payload as comma-separated string>; <msgmode>`
-- TTY - `<description>; <tty command>`
+1. By manually adding appropriate comma- or semicolon-delimited message descriptions and payload definitions to the `"ubxpresets_l"`, `"nmeapresets_l"` or `"ttypresets_l"` sections of your json configuration file (see [example provided](https://github.com/semuconsulting/PyGPSClient/blob/master/pygpsclient.json#L189)). The message definition comprises a free-format text description (*avoid embedded commas or semi-colons*) followed by one or more pyubx2 (UBX), pynmeagps (NMEA) or tty (ASCII) message constructors, e.g. 
+
+   - UBX - `<description>, [<message class>, <message id>, <payload as hexadecimal string>, <msgmode>, ...]`
+   - NMEA - `<description>; [<talker>; <message id>; <payload as comma-separated string>; <msgmode>; ...]`
+   - TTY - `<description>; [<tty command>; ...]`
+
+2. By using the [Configuration Command Load/Save/Record](#recorder) facility to record commands as they are sent to the receiver, and automatically import these recorded commands into the relevant `"...presets_l"` section of the json configuration file.
 
 If the command description contains the term `CONFIRM`, a pop-up confirmation box will appear before the command is actioned.
 
-When PyGPSClient is first started, these settings are pre-populated with an initial set of preset commands, which can be saved to a \*.json configuration file and then manually removed, amended or supplemented in accordance with the user's preferences. To reinstate this initial set at a later date, insert the line `"INIT_PRESETS",` at the top of the relevant `"ubxpresets_l"`, `"nmeapresets_l"` or `"ttypresets_l"` configuration setting.
+When PyGPSClient is first started, the preset command sections are pre-populated in-memory with an initial set of preset commands, which can be saved to a json configuration file and then manually edited in accordance with the user's preferences. To reinstate this initial set at a later date, insert the line `"INIT_PRESETS"` at the top of the relevant `"ubxpresets_l"`, `"nmeapresets_l"` or `"ttypresets_l"` configuration section.
 
-The `pygpsclient.ubx2preset()`, `pygpsclient.nmea2preset()` and `pygpsclient.tty2preset()` helper functions may be used to convert a `UBXMessage`, `NMEAMessage` or ASCII text object into a preset string suitable for copying and pasting into the `"ubxpresents_l":`, `"nmeapresets_l":` or `"ttypresets_l":` JSON configuration sections:
+The `pygpsclient.ubx2preset()`, `pygpsclient.nmea2preset()` and `pygpsclient.tty2preset()` helper functions may be used to convert a `UBXMessage`, `NMEAMessage` or ASCII text object into a string suitable for copying and pasting into the `"ubxpresets_l":`, `"nmeapresets_l":` or `"ttypresets_l":` configuration file sections:
 
 ```python
 from pygpsclient import ubx2preset, nmea2preset, tty2preset
@@ -476,8 +481,6 @@ IM19 System reset CONFIRM; AT+SYSTEM_RESET
 
 Multiple commands can be concatenated on a single line. Illustrative examples are shown in the sample [pygpsclient.json](https://github.com/semuconsulting/PyGPSClient/blob/master/pygpsclient.json#L189) file.
 
-The [Configuration Command Load/Save/Record facility](#configuration-command-loadsaverecord-facility) can also be used to import recorded configuration command sequences into the presets section of the json configuration file.
-
 ---
 ## <a name="cli">Command Line Utilities</a>
 
@@ -486,11 +489,11 @@ 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>
+## <a name="troubleshoot">Troubleshooting</a>
 
 1. If you encounter persistent `WARNING>>Error parsing data stream Serial stream terminated unexpectedly` messages in the console, this may be indicative of insufficient serial port bandwidth (baudrate or timeout) for the current output message cohort (*particularly if this includes Ephemera or Observation data*). Try increasing the baudrate in the first instance.
 
-2. Most budget USB-UART adapters (e.g. FT232, CH345, CP2102, *including those embedded on development boards*) have a bandwidth limit of around 3MB/s (≈ 375000 baud) and may not work reliably above 230600 baud, even if the receiver supports higher baud rates. If you're using an adapter and notice significant message corruption (e.g. frequent `WARNING>>..invalid checksum` messages), try reducing the baud rate to a maximum 230600.
+2. Most [budget USB-UART adapters](https://www.amazon.co.uk/DSD-TECH-adapter-FT232RL-Compatible/dp/B07BBPX8B8?ref_=ast_sto_dp) (e.g. FT232, CH345, CP2102, *including those embedded on development boards*) have a bandwidth limit of around 3Mbps (≈ 375000 baud) and may not work reliably above 230600 baud, even if the receiver supports higher baud rates. If you're using an adapter and notice significant message corruption (e.g. frequent `WARNING>>..invalid checksum` messages), try reducing the baud rate to a maximum 230600.
 
 3. Some Linux Wayland platforms appear to require Toplevel dialog windows to be non-transient (`transient_dialog_b: 0`) for the window 'maximise' icon to work properly.
 
diff --git a/RELEASE_NOTES.md b/RELEASE_NOTES.md
@@ -4,6 +4,7 @@
 
 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
diff --git a/images/nmeaconfig_widget.png b/images/nmeaconfig_widget.png
diff --git a/src/pygpsclient/nmea_preset_frame.py b/src/pygpsclient/nmea_preset_frame.py
@@ -18,12 +18,14 @@
     VERTICAL,
     Button,
     E,
+    Entry,
     Frame,
     Label,
     Listbox,
     N,
     S,
     Scrollbar,
+    StringVar,
     W,
 )
 
@@ -39,7 +41,9 @@
     ICON_WARNING,
     NMEA_PRESET,
     OKCOL,
+    VALNONBLANK,
 )
+from pygpsclient.helpers import validate
 from pygpsclient.strings import (
     CONFIRM,
     DLGACTION,
@@ -80,6 +84,8 @@ def __init__(self, app: Frame, parent: Frame, *args, **kwargs):
         self._img_warn = ImageTk.PhotoImage(Image.open(ICON_WARNING))
         self._preset_command = None
         self._configfile = None
+        self._command = StringVar()
+        self._confirm = False
         self._body()
         self._do_layout()
         self._attach_events()
@@ -90,6 +96,16 @@ def _body(self):
         Set up frame and widgets.
         """
 
+        self._lbl_command = Label(
+            self,
+            text="Command",
+        )
+        self._ent_command = Entry(
+            self,
+            textvariable=self._command,
+            relief="sunken",
+            width=55,
+        )
         self._lbl_presets = Label(self, text=LBLNMEAPRESET, anchor=W)
         self._lbx_preset = Listbox(
             self,
@@ -119,15 +135,17 @@ def _do_layout(self):
         Layout widgets.
         """
 
-        self._lbl_presets.grid(column=0, row=0, columnspan=6, padx=3, sticky=EW)
+        self._lbl_command.grid(column=0, row=0, padx=3, sticky=W)
+        self._ent_command.grid(column=1, row=0, columnspan=5, padx=3, sticky=EW)
+        self._lbl_presets.grid(column=0, row=1, columnspan=6, padx=3, sticky=EW)
         self._lbx_preset.grid(
-            column=0, row=1, columnspan=3, rowspan=20, padx=3, pady=3, sticky=EW
+            column=0, row=2, columnspan=3, rowspan=20, padx=3, pady=3, sticky=EW
         )
-        self._scr_presetv.grid(column=2, row=1, rowspan=20, sticky=(N, S, E))
-        self._scr_preseth.grid(column=0, row=21, columnspan=3, sticky=EW)
-        self._btn_send_command.grid(column=3, row=1, padx=3, ipadx=3, ipady=3, sticky=E)
+        self._scr_presetv.grid(column=2, row=2, rowspan=20, sticky=(N, S, E))
+        self._scr_preseth.grid(column=0, row=22, columnspan=3, sticky=EW)
+        self._btn_send_command.grid(column=3, row=2, padx=3, ipadx=3, ipady=3, sticky=E)
         self._lbl_send_command.grid(
-            column=3, row=2, padx=3, ipadx=3, ipady=3, sticky=EW
+            column=3, row=3, padx=3, ipadx=3, ipady=3, sticky=EW
         )
 
         cols, rows = self.grid_size()
@@ -152,27 +170,31 @@ def reset(self):
         self.__app.configuration.init_presets("nmea")
         for i, preset in enumerate(self.__app.configuration.get("nmeapresets_l")):
             self._lbx_preset.insert(i, preset)
+        self._command.set("")
+        self._confirm = False
 
     def _on_select_preset(self, *args, **kwargs):  # pylint: disable=unused-argument
         """
         Preset command has been selected.
         """
 
-        idx = self._lbx_preset.curselection()
-        self._preset_command = self._lbx_preset.get(idx)
+        cmd = self._lbx_preset.get(self._lbx_preset.curselection())
+        self._confirm = CONFIRM in cmd
+        self._command.set(cmd[cmd.find(";", 1) + 1 :].strip())
 
     def _on_send_preset(self, *args, **kwargs):  # pylint: disable=unused-argument
         """
         Preset command send button has been clicked.
         """
 
-        if self._preset_command in ("", None):
-            self.__container.status_label = ("Select preset", ERRCOL)
+        if not self._ent_command.validate(VALNONBLANK):
+            self.__container.status_label = ("Select preset or enter command", ERRCOL)
             return
+        self._preset_command = self._command.get()
 
         confids = []
         try:
-            if CONFIRM in self._preset_command:
+            if self._confirm:
                 if ConfirmBox(self, DLGACTION, DLGACTIONCONFIRM).show():
                     confids = self._do_user_defined(self._preset_command)
                     status = CONFIRMED
@@ -212,7 +234,7 @@ def _do_user_defined(self, command: str) -> list:
         confids = []
         try:
             seg = command.split(";")
-            for i in range(1, len(seg), 4):
+            for i in range(0, len(seg), 4):
                 talker = seg[i].strip()
                 msg_id = seg[i + 1].strip()
                 payload = seg[i + 2].strip()
