ardupilot-methodic-configurator 2.7.0__py3-none-any.whl → 2.7.1__py3-none-any.whl

ardupilot_methodic_configurator/__init__.py

@@ -12,4 +12,4 @@ from ardupilot_methodic_configurator.internationalization import load_translatio
 
 _ = load_translation()
 
-__version__ = "2.7.0"
+__version__ = "2.7.1"

ardupilot_methodic_configurator/__main__.py

@@ -33,6 +33,7 @@ import argcomplete
 
 from ardupilot_methodic_configurator import _, __version__
 from ardupilot_methodic_configurator.backend_filesystem import LocalFilesystem
+from ardupilot_methodic_configurator.backend_filesystem_freedesktop import FreeDesktop
 from ardupilot_methodic_configurator.backend_filesystem_program_settings import ProgramSettings
 from ardupilot_methodic_configurator.backend_flightcontroller import FlightController
 from ardupilot_methodic_configurator.backend_internet import verify_and_open_url
@@ -136,10 +137,13 @@ def connect_to_fc_and_set_vehicle_type(args: argparse.Namespace) -> tuple[Flight
     flight_controller = FlightController(reboot_time=args.reboot_time, baudrate=args.baudrate)
 
     error_str = flight_controller.connect(args.device, log_errors=False)
+
     if error_str:
         if args.device and _("No serial ports found") not in error_str:
             logging_error(error_str)
         conn_sel_window = ConnectionSelectionWindow(flight_controller, error_str, default_baudrate=args.baudrate)
+        # Set up startup notification for the connection selection window
+        FreeDesktop.setup_startup_notification(conn_sel_window.root)  # type: ignore[arg-type]
         conn_sel_window.root.mainloop()
 
     vehicle_type = args.vehicle_type
@@ -210,6 +214,8 @@ def vehicle_directory_selection(state: ApplicationState) -> Union[VehicleProject
             )
         )
     vehicle_dir_window = VehicleProjectOpenerWindow(state.vehicle_project_manager)
+    # Set up startup notification for the vehicle directory selection window
+    FreeDesktop.setup_startup_notification(vehicle_dir_window.root)  # type: ignore[arg-type]
     vehicle_dir_window.root.mainloop()
 
     if state.vehicle_project_manager.reset_fc_parameters_to_their_defaults:
@@ -343,6 +349,9 @@ def component_editor(state: ApplicationState) -> None:
     elif should_open_firmware_documentation(state.flight_controller):
         open_firmware_documentation(state.flight_controller.info.firmware_type)
 
+        # Set up startup notification for the component editor window
+        FreeDesktop.setup_startup_notification(component_editor_window.root)  # type: ignore[arg-type]
+
     # Run the GUI
     component_editor_window.root.mainloop()
 
@@ -499,7 +508,7 @@ def main() -> None:
     args = create_argument_parser().parse_args()
 
     # Create desktop icon if needed (only on first run in venv)
-    ProgramSettings.create_desktop_icon_if_needed()
+    FreeDesktop.create_desktop_icon_if_needed()
 
     state = ApplicationState(args)
 

ardupilot_methodic_configurator/backend_filesystem_freedesktop.py

@@ -0,0 +1,275 @@
+"""
+Handles FreeDesktop.org compliance and desktop integration features.
+
+This includes creating desktop entries for application launchers, managing startup
+notifications according to the FreeDesktop Startup Notification specification,
+and ensuring proper integration with Linux desktop environments.
+
+This file is part of ArduPilot Methodic Configurator. https://github.com/ArduPilot/MethodicConfigurator
+
+SPDX-FileCopyrightText: 2024-2025 Amilcar do Carmo Lucas <amilcar.lucas@iav.de>
+
+SPDX-License-Identifier: GPL-3.0-or-later
+"""
+
+import re
+import subprocess
+import tkinter as tk
+from logging import debug as logging_debug
+from logging import error as logging_error
+from os import chmod as os_chmod
+from os import environ as os_environ
+from os import makedirs as os_makedirs
+from os import name as os_name
+from os import path as os_path
+from shutil import which as shutil_which
+from sys import platform as sys_platform
+from typing import Optional, Union
+
+from ardupilot_methodic_configurator.backend_filesystem_program_settings import ProgramSettings
+
+
+class FreeDesktop:
+    """
+    A class responsible for FreeDesktop.org compliance and desktop integration.
+
+    This includes creating desktop entries for application launchers, managing startup
+    notifications according to the FreeDesktop Startup Notification specification,
+    and ensuring proper integration with Linux desktop environments.
+    """
+
+    def __init__(self) -> None:
+        pass
+
+    @staticmethod
+    def _is_linux_system() -> bool:
+        """Check if running on a Linux system."""
+        return os_name == "posix" and sys_platform.startswith("linux")
+
+    @staticmethod
+    def _get_desktop_file_path() -> str:
+        """Get the path where the desktop file should be created."""
+        return os_path.expanduser("~/.local/share/applications/ardupilot_methodic_configurator.desktop")
+
+    @staticmethod
+    def _desktop_icon_exists(desktop_file_path: str) -> bool:
+        """Check if the desktop icon already exists."""
+        return os_path.exists(desktop_file_path)
+
+    @staticmethod
+    def _get_virtual_env_path() -> Optional[str]:
+        """Get the virtual environment path from environment variables."""
+        return os_environ.get("VIRTUAL_ENV")
+
+    @staticmethod
+    def _create_desktop_entry_content(venv_path: str, icon_path: str) -> str:
+        """Create the desktop entry file content."""
+        # Try to use python executable directly for better compatibility
+        python_exe = os_path.join(venv_path, "bin", "python")
+        if os_path.exists(python_exe):
+            # Use python executable directly
+            exec_cmd = f"{python_exe} -m ardupilot_methodic_configurator"
+        else:
+            # Fallback to bash -c method
+            bash_path = shutil_which("bash") or "/bin/bash"
+            activate_cmd = f"source {venv_path}/bin/activate && ardupilot_methodic_configurator"
+            exec_cmd = f'{bash_path} -c "{activate_cmd}"'
+
+        return f"""[Desktop Entry]
+Version=1.0
+Name=ArduPilot Methodic Configurator
+Comment=A clear ArduPilot configuration sequence
+Exec={exec_cmd}
+Icon={icon_path}
+Terminal=true
+Type=Application
+Categories=Development;
+Keywords=ardupilot;arducopter;drone;parameters;configuration;scm
+StartupWMClass=ArduPilotMethodicConfigurator
+StartupNotify=true
+"""
+
+    @staticmethod
+    def _ensure_applications_dir_exists(desktop_file_path: str) -> str:
+        """Ensure the applications directory exists and return it."""
+        apps_dir = os_path.dirname(desktop_file_path)
+        os_makedirs(apps_dir, exist_ok=True)
+        return apps_dir
+
+    @staticmethod
+    def _write_desktop_file(desktop_file_path: str, content: str) -> None:
+        """Write the desktop file content to disk."""
+        with open(desktop_file_path, "w", encoding="utf-8") as f:
+            f.write(content)
+
+    @staticmethod
+    def _set_desktop_file_permissions(desktop_file_path: str) -> None:
+        """Set appropriate permissions on the desktop file."""
+        os_chmod(desktop_file_path, 0o644)
+
+    @staticmethod
+    def _update_desktop_database(apps_dir: str) -> None:
+        """Update the desktop database if the command is available."""
+        update_desktop_db_cmd = shutil_which("update-desktop-database")
+        if update_desktop_db_cmd:
+            subprocess.run([update_desktop_db_cmd, apps_dir], check=False, capture_output=True)  # noqa: S603
+
+    @staticmethod
+    def create_desktop_icon_if_needed() -> None:
+        """
+        Create a desktop icon for the application if running in a virtual environment and icon doesn't exist.
+
+        This function detects if we're running in a virtual environment and creates a desktop
+        entry that activates the venv and runs the application with the correct icon.
+        """
+        # Only create desktop icon on Linux systems
+        if not FreeDesktop._is_linux_system():
+            return
+
+        # Check if desktop icon already exists
+        desktop_file_path = FreeDesktop._get_desktop_file_path()
+        if FreeDesktop._desktop_icon_exists(desktop_file_path):
+            return
+
+        # Check if we're in a virtual environment
+        venv_path = FreeDesktop._get_virtual_env_path()
+        if not venv_path:
+            return
+
+        # Find the icon path
+        icon_path = ProgramSettings.application_icon_filepath()
+        if not icon_path:
+            return
+
+        # Create the desktop entry content
+        desktop_entry = FreeDesktop._create_desktop_entry_content(venv_path, icon_path)
+
+        # Ensure the applications directory exists
+        apps_dir = FreeDesktop._ensure_applications_dir_exists(desktop_file_path)
+
+        # Write the desktop file
+        try:
+            FreeDesktop._write_desktop_file(desktop_file_path, desktop_entry)
+            FreeDesktop._set_desktop_file_permissions(desktop_file_path)
+            FreeDesktop._update_desktop_database(apps_dir)
+
+        except (OSError, subprocess.SubprocessError):
+            logging_error("Failed to create application launch desktop icon")
+
+    @staticmethod
+    def _get_desktop_startup_id() -> Union[str, None]:
+        """
+        Get the DESKTOP_STARTUP_ID environment variable.
+
+        Returns:
+            The startup ID string if set, None otherwise.
+
+        """
+        return os_environ.get("DESKTOP_STARTUP_ID")
+
+    @staticmethod
+    def _send_startup_notification_complete(startup_id: str) -> None:
+        """
+        Send the startup notification "remove" message to indicate the application has started.
+
+        This implements the freedesktop.org startup notification protocol.
+
+        Args:
+            startup_id: The DESKTOP_STARTUP_ID that was passed to the application
+
+        """
+        if not startup_id:
+            return
+
+        # Validate startup_id to prevent shell injection (should only contain alphanumeric chars, hyphens, underscores)
+        if not re.match(r"^[a-zA-Z0-9_-]+$", startup_id):
+            logging_debug("Invalid startup_id format: %s", startup_id)
+            return
+
+        try:
+            # Find the full path to xdg-startup-notify for security
+            xdg_notify_path = shutil_which("xdg-startup-notify")
+            if xdg_notify_path:
+                # Try to use xdg-startup-notify if available (part of xdg-utils)
+                result = subprocess.run(  # noqa: S603
+                    [xdg_notify_path, "remove", startup_id], capture_output=True, timeout=1.0, check=False
+                )
+                if result.returncode == 0:
+                    logging_debug("Sent startup notification completion for ID: %s", startup_id)
+                else:
+                    logging_debug("xdg-startup-notify failed: %s", result.stderr.decode().strip())
+            else:
+                logging_debug("xdg-startup-notify not found in PATH")
+        except (subprocess.TimeoutExpired, subprocess.SubprocessError, FileNotFoundError):
+            # If xdg-startup-notify is not available or fails, try manual X11 approach
+            FreeDesktop._send_startup_notification_x11(startup_id)
+
+    @staticmethod
+    def _send_startup_notification_x11(startup_id: str) -> None:
+        """
+        Send startup notification completion using direct X11 ClientMessage.
+
+        Args:
+            startup_id: The DESKTOP_STARTUP_ID that was passed to the application
+
+        """
+        if not tk:
+            return
+
+        try:
+            # Create a temporary Tk instance to access X11 if we don't have one yet
+            temp_root = tk.Tk()
+            temp_root.withdraw()  # Hide the window
+
+            # Try to send the message using Tk's send command
+            # Format: "remove: ID=<startup_id>"
+            message = f"remove: ID={startup_id}"
+
+            # Use Tk's send command to broadcast to the root window
+            # This is a bit of a hack, but Tk doesn't expose X11 messaging directly
+            try:
+                temp_root.eval(f"send -async . {{event generate . <<StartupComplete>> -data {{{message}}}}}")
+
+                # Also try to use the X11 atoms if available
+                # _NET_STARTUP_INFO is the atom we need to send
+                temp_root.eval(f"send -async . {{wm command . _NET_STARTUP_INFO {{{message}}}}}")
+
+            except Exception:  # pylint: disable=broad-exception-caught
+                # If all else fails, just log that we tried
+                logging_debug("Could not send X11 startup notification message")
+
+            temp_root.destroy()
+
+        except Exception as e:  # pylint: disable=broad-exception-caught
+            logging_debug("Failed to send X11 startup notification: %s", e)
+
+    @staticmethod
+    def setup_startup_notification(main_window: tk.Tk) -> None:
+        """
+        Set up startup notification for the application.
+
+        Checks for DESKTOP_STARTUP_ID and sends the completion message when the window is ready.
+
+        Args:
+            main_window: The main Tkinter window
+
+        """
+        if not FreeDesktop._is_linux_system():
+            return
+        startup_id = FreeDesktop._get_desktop_startup_id() or ""
+        if startup_id:
+            logging_debug("Startup notification ID: %s", startup_id)
+
+            # Send the completion message after the window is mapped
+            def on_map(event: tk.Event) -> None:
+                if event and event.widget == main_window:
+                    FreeDesktop._send_startup_notification_complete(startup_id)
+                    # Remove the binding after first map
+                    main_window.unbind("<Map>", on_map_handler)
+
+            # Bind to the Map event to know when the window is first shown
+            on_map_handler = main_window.bind("<Map>", on_map)
+
+            # Also try to send immediately in case the window is already mapped
+            if main_window.winfo_viewable():
+                FreeDesktop._send_startup_notification_complete(startup_id)

ardupilot_methodic_configurator/backend_filesystem_program_settings.py

@@ -9,7 +9,6 @@ SPDX-License-Identifier: GPL-3.0-or-later
 """
 
 # from sys import exit as sys_exit
-import subprocess
 from contextlib import suppress as contextlib_suppress
 from glob import glob as glob_glob
 from importlib.resources import files as importlib_files
@@ -17,10 +16,7 @@ from json import dump as json_dump
 from json import load as json_load
 from logging import debug as logging_debug
 from logging import error as logging_error
-from os import chmod as os_chmod
-from os import environ as os_environ
 from os import makedirs as os_makedirs
-from os import name as os_name
 from os import path as os_path
 from os import sep as os_sep
 from pathlib import Path
@@ -28,8 +24,6 @@ from platform import system as platform_system
 from re import escape as re_escape
 from re import match as re_match
 from re import sub as re_sub
-from shutil import which as shutil_which
-from sys import platform as sys_platform
 from typing import Any, Optional, Union
 
 from platformdirs import site_config_dir, user_config_dir
@@ -422,116 +416,3 @@ class ProgramSettings:
         """
         filepath, _error_msg = ProgramSettings.motor_diagram_filepath(frame_class, frame_type)
         return filepath != "" and os_path.exists(filepath)
-
-    @staticmethod
-    def _is_linux_system() -> bool:
-        """Check if running on a Linux system."""
-        return os_name == "posix" and sys_platform.startswith("linux")
-
-    @staticmethod
-    def _get_desktop_file_path() -> str:
-        """Get the path where the desktop file should be created."""
-        return os_path.expanduser("~/.local/share/applications/ardupilot_methodic_configurator.desktop")
-
-    @staticmethod
-    def _desktop_icon_exists(desktop_file_path: str) -> bool:
-        """Check if the desktop icon already exists."""
-        return os_path.exists(desktop_file_path)
-
-    @staticmethod
-    def _get_virtual_env_path() -> Optional[str]:
-        """Get the virtual environment path from environment variables."""
-        return os_environ.get("VIRTUAL_ENV")
-
-    @staticmethod
-    def _create_desktop_entry_content(venv_path: str, icon_path: str) -> str:
-        """Create the desktop entry file content."""
-        # Try to use python executable directly for better compatibility
-        python_exe = os_path.join(venv_path, "bin", "python")
-        if os_path.exists(python_exe):
-            # Use python executable directly
-            exec_cmd = f"{python_exe} -m ardupilot_methodic_configurator"
-        else:
-            # Fallback to bash -c method
-            bash_path = shutil_which("bash") or "/bin/bash"
-            activate_cmd = f"source {venv_path}/bin/activate && ardupilot_methodic_configurator"
-            exec_cmd = f'{bash_path} -c "{activate_cmd}"'
-
-        return f"""[Desktop Entry]
-Version=1.0
-Name=ArduPilot Methodic Configurator
-Comment=A clear ArduPilot configuration sequence
-Exec={exec_cmd}
-Icon={icon_path}
-Terminal=true
-Type=Application
-Categories=Development;
-Keywords=ardupilot;arducopter;drone;parameters;configuration;scm
-"""
-
-    @staticmethod
-    def _ensure_applications_dir_exists(desktop_file_path: str) -> str:
-        """Ensure the applications directory exists and return it."""
-        apps_dir = os_path.dirname(desktop_file_path)
-        os_makedirs(apps_dir, exist_ok=True)
-        return apps_dir
-
-    @staticmethod
-    def _write_desktop_file(desktop_file_path: str, content: str) -> None:
-        """Write the desktop file content to disk."""
-        with open(desktop_file_path, "w", encoding="utf-8") as f:
-            f.write(content)
-
-    @staticmethod
-    def _set_desktop_file_permissions(desktop_file_path: str) -> None:
-        """Set appropriate permissions on the desktop file."""
-        os_chmod(desktop_file_path, 0o644)
-
-    @staticmethod
-    def _update_desktop_database(apps_dir: str) -> None:
-        """Update the desktop database if the command is available."""
-        update_desktop_db_cmd = shutil_which("update-desktop-database")
-        if update_desktop_db_cmd:
-            subprocess.run([update_desktop_db_cmd, apps_dir], check=False, capture_output=True)  # noqa: S603
-
-    @staticmethod
-    def create_desktop_icon_if_needed() -> None:
-        """
-        Create a desktop icon for the application if running in a virtual environment and icon doesn't exist.
-
-        This function detects if we're running in a virtual environment and creates a desktop
-        entry that activates the venv and runs the application with the correct icon.
-        """
-        # Only create desktop icon on Linux systems
-        if not ProgramSettings._is_linux_system():
-            return
-
-        # Check if desktop icon already exists
-        desktop_file_path = ProgramSettings._get_desktop_file_path()
-        if ProgramSettings._desktop_icon_exists(desktop_file_path):
-            return
-
-        # Check if we're in a virtual environment
-        venv_path = ProgramSettings._get_virtual_env_path()
-        if not venv_path:
-            return
-
-        # Find the icon path
-        icon_path = ProgramSettings.application_icon_filepath()
-        if not icon_path:
-            return
-
-        # Create the desktop entry content
-        desktop_entry = ProgramSettings._create_desktop_entry_content(venv_path, icon_path)
-
-        # Ensure the applications directory exists
-        apps_dir = ProgramSettings._ensure_applications_dir_exists(desktop_file_path)
-
-        # Write the desktop file
-        try:
-            ProgramSettings._write_desktop_file(desktop_file_path, desktop_entry)
-            ProgramSettings._set_desktop_file_permissions(desktop_file_path)
-            ProgramSettings._update_desktop_database(apps_dir)
-
-        except (OSError, subprocess.SubprocessError):
-            logging_error("Failed to create application launch desktop icon")

ardupilot_methodic_configurator/configuration_manager.py

@@ -16,7 +16,8 @@ from csv import writer as csv_writer
 from logging import error as logging_error
 from logging import info as logging_info
 from pathlib import Path
-from typing import Callable, Optional
+from time import time
+from typing import Callable, Literal, Optional
 from webbrowser import open as webbrowser_open  # to open the web documentation
 
 from ardupilot_methodic_configurator import _
@@ -36,6 +37,8 @@ ShowWarningCallback = Callable[[str, str], None]  # (title, message) -> None
 ShowErrorCallback = Callable[[str, str], None]  # (title, message) -> None
 ShowInfoCallback = Callable[[str, str], None]  # (title, message) -> None
 AskRetryCancelCallback = Callable[[str, str], bool]  # (title, message) -> bool
+ExperimentChoice = Literal["close", True, False]
+ExperimentChoiceCallback = Callable[[str, str, list[str]], ExperimentChoice]
 
 
 class OperationNotPossibleError(Exception):
@@ -76,6 +79,8 @@ class ConfigurationManager:  # pylint: disable=too-many-public-methods, too-many
 
         self._at_least_one_changed = False
 
+        self._last_time_asked_to_save: float = 0
+
     # frontend_tkinter_parameter_editor_table.py API start
     @property
     def connected_vehicle_type(self) -> str:
@@ -183,7 +188,7 @@ class ConfigurationManager:  # pylint: disable=too-many-public-methods, too-many
             show_error(_("Fatal error reading parameter files"), f"{exp}")
             raise
 
-    def should_copy_fc_values_to_file(self, selected_file: str) -> tuple[bool, Optional[dict], Optional[str]]:
+    def _should_copy_fc_values_to_file(self, selected_file: str) -> tuple[bool, Optional[dict], Optional[str]]:
         """
         Check if flight controller values should be copied to the specified file.
 
@@ -207,7 +212,7 @@ class ConfigurationManager:  # pylint: disable=too-many-public-methods, too-many
             return True, relevant_fc_params, auto_changed_by
         return False, None, auto_changed_by
 
-    def copy_fc_values_to_file(self, selected_file: str, relevant_fc_params: dict) -> bool:
+    def _copy_fc_values_to_file(self, selected_file: str, relevant_fc_params: dict) -> bool:
         """
         Copy FC values to the specified file.
 
@@ -222,7 +227,75 @@ class ConfigurationManager:  # pylint: disable=too-many-public-methods, too-many
         params_copied = self._local_filesystem.copy_fc_values_to_file(selected_file, relevant_fc_params)
         return bool(params_copied)
 
-    def get_file_jump_options(self, selected_file: str) -> dict[str, str]:
+    def handle_copy_fc_values_workflow(
+        self,
+        selected_file: str,
+        ask_user_choice: ExperimentChoiceCallback,
+        show_info: ShowInfoCallback,
+    ) -> ExperimentChoice:
+        """
+        Handle the complete workflow for copying FC values to file with user interaction.
+
+        Args:
+            selected_file: The configuration file to potentially update.
+            ask_user_choice: Callback to ask user for choice (Yes/No/Close).
+            show_info: Callback to show information messages.
+
+        Returns:
+            ExperimentChoice: "close" if user chose to close, True if copied, False if no copy.
+
+        """
+        should_copy, relevant_fc_params, auto_changed_by = self._should_copy_fc_values_to_file(selected_file)
+        if should_copy and relevant_fc_params and auto_changed_by:
+            msg = _(
+                "This configuration step requires external changes by: {auto_changed_by}\n\n"
+                "The external tool experiment procedure is described in the tuning guide.\n\n"
+                "Choose an option:\n"
+                "* CLOSE - Close the application and go perform the experiment\n"
+                "* YES - Copy current FC values to {selected_file} (if you've already completed the experiment)\n"
+                "* NO - Continue without copying values (if you haven't performed the experiment yet,"
+                " but know what you are doing)"
+            ).format(auto_changed_by=auto_changed_by, selected_file=selected_file)
+
+            user_choice = ask_user_choice(_("Update file with values from FC?"), msg, [_("Close"), _("Yes"), _("No")])
+
+            if user_choice is True:  # Yes option
+                params_copied = self._copy_fc_values_to_file(selected_file, relevant_fc_params)
+                if params_copied:
+                    show_info(
+                        _("Parameters copied"),
+                        _("FC values have been copied to {selected_file}").format(selected_file=selected_file),
+                    )
+            return user_choice
+        return False
+
+    def handle_file_jump_workflow(
+        self,
+        selected_file: str,
+        gui_complexity: str,
+        ask_user_confirmation: AskConfirmationCallback,
+    ) -> str:
+        """
+        Handle the complete workflow for file jumping with user interaction.
+
+        Args:
+            selected_file: The current configuration file.
+            gui_complexity: The GUI complexity setting ("simple" or other).
+            ask_user_confirmation: Callback to ask user for confirmation.
+
+        Returns:
+            str: The destination file to jump to, or the original file if no jump.
+
+        """
+        jump_options = self._get_file_jump_options(selected_file)
+        for dest_file, msg in jump_options.items():
+            if gui_complexity == "simple" or ask_user_confirmation(
+                _("Skip some steps?"), _(msg) if msg else _("Skip to {dest_file}?").format(dest_file=dest_file)
+            ):
+                return dest_file
+        return selected_file
+
+    def _get_file_jump_options(self, selected_file: str) -> dict[str, str]:
         """
         Get available file jump options for the selected file.
 
@@ -235,11 +308,43 @@ class ConfigurationManager:  # pylint: disable=too-many-public-methods, too-many
         """
         return self._local_filesystem.jump_possible(selected_file)
 
+    def handle_write_changes_workflow(
+        self,
+        annotate_params_into_files: bool,
+        ask_user_confirmation: AskConfirmationCallback,
+    ) -> bool:
+        """
+        Handle the workflow for writing changes to intermediate parameter file.
+
+        Args:
+            at_least_one_param_edited: Whether any parameters have been edited.
+            annotate_params_into_files: Whether to annotate documentation into files.
+            ask_user_confirmation: Callback to ask user for confirmation.
+
+        Returns:
+            bool: True if changes were written, False otherwise.
+
+        """
+        elapsed_since_last_ask = time() - self._last_time_asked_to_save
+        # if annotate parameters into files is true, we always need to write to file, because
+        # the parameter metadata might have changed, or not be present in the file.
+        # In that situation, avoid asking multiple times to write the file, by checking the time last asked
+        # But only if annotate_params_into_files is True
+        if self._has_unsaved_changes() or (annotate_params_into_files and elapsed_since_last_ask > 1.0):
+            msg = _("Do you want to write the changes to the {current_filename} file?").format(
+                current_filename=self.current_file
+            )
+            if ask_user_confirmation(_("One or more parameters have been edited"), msg):
+                self._export_current_file(annotate_doc=annotate_params_into_files)
+                self._last_time_asked_to_save = time()
+                return True
+        return False
+
     def should_download_file_from_url_workflow(
         self,
         selected_file: str,
-        ask_confirmation: Callable[[str, str], bool],
-        show_error: Callable[[str, str], None],
+        ask_confirmation: AskConfirmationCallback,
+        show_error: ShowErrorCallback,
     ) -> bool:
         """
         Handle file download workflow with injected GUI callbacks.
@@ -1360,7 +1465,7 @@ class ConfigurationManager:  # pylint: disable=too-many-public-methods, too-many
             }
         )
 
-    def has_unsaved_changes(self) -> bool:
+    def _has_unsaved_changes(self) -> bool:
         """
         Check if any changes have been made that need to be saved.
 
@@ -1408,7 +1513,7 @@ class ConfigurationManager:  # pylint: disable=too-many-public-methods, too-many
     def _write_current_file(self) -> None:
         self._local_filesystem.write_last_uploaded_filename(self.current_file)
 
-    def export_current_file(self, annotate_doc: bool) -> None:
+    def _export_current_file(self, annotate_doc: bool) -> None:
         # Convert domain model parameters to Par objects for export
         export_params = self.get_parameters_as_par_dict()
 

ardupilot_methodic_configurator/configuration_steps_ArduCopter.json

@@ -214,7 +214,7 @@
             "wiki_text": "Follow the blog instructions and use Mission Planner instead of this tool to configure the mandatory hardware parameters.",
             "wiki_url": "",
             "external_tool_text": "Mission Planner",
-            "external_tool_url": "https://github.com/ArduPilot/MethodicConfigurator/blob/latest/TUNING_GUIDE_ArduCopter.md#212-configure-mandatory-hardware-parameters-17",
+            "external_tool_url": "https://ardupilot.org/copter/docs/configuring-hardware.html",
             "mandatory_text": "100% mandatory (0% optional)",
             "auto_changed_by": "Mission Planner. If you have not done this step in Mission Planner yet, close this application and use Mission Planner",
             "old_filenames": ["11_mp_setup_mandatory_hardware.param"]

ardupilot_methodic_configurator/frontend_tkinter_base_window.py

@@ -101,7 +101,7 @@ class BaseWindow:
         if root_tk:
             self.root = tk.Toplevel(root_tk)
         else:
-            self.root = tk.Tk()
+            self.root = tk.Tk(className="ArduPilotMethodicConfigurator")
             # Only set icon for main windows, and only outside test environments
             self._setup_application_icon()
 

ardupilot_methodic_configurator/frontend_tkinter_component_editor.py

@@ -114,21 +114,22 @@ class ComponentEditorWindow(ComponentEditorWindowBase):
             if isinstance(protocol_combobox, PairTupleCombobox):
                 # Rebuild the (key, display) pairs for PairTupleCombobox
                 protocol_tuples = [(p, p) for p in protocols]
-                # Update the combobox entries using set_entries_tuple
+                # Get current selection and validate it against new protocols
                 current_selection = protocol_combobox.get_selected_key()
-                protocol_combobox.set_entries_tuple(protocol_tuples, current_selection)
-
-                # Validate the current selection
-                selected_protocol = protocol_combobox.get_selected_key() or ""
-                if selected_protocol and selected_protocol not in protocol_combobox.list_keys:
-                    # Clear the selection when current key is no longer allowed
-                    protocol_combobox.set_entries_tuple(protocol_tuples, None)
-                    _component: str = " > ".join(protocol_path)
+                if current_selection and current_selection not in protocols:
+                    # Current selection is not valid for new protocols, clear it
+                    invalid_selection = current_selection
+                    current_selection = None
+                    component: str = " > ".join(protocol_path)
                     err_msg = _(
-                        "On {_component} the selected\nprotocol '{selected_protocol}' "
+                        "On {component} the selected\nprotocol '{invalid_selection}' "
                         "is not available for the selected connection Type."
                     )
-                    err_msg = err_msg.format(**locals())
+                    err_msg = err_msg.format(component=component, invalid_selection=invalid_selection)
+
+                # Update the combobox entries using set_entries_tuple with validated selection
+                protocol_combobox.set_entries_tuple(protocol_tuples, current_selection)
+
                 if err_msg:
                     show_error_message(_("Error"), err_msg)
                     protocol_combobox.configure(style="comb_input_invalid.TCombobox")

ardupilot_methodic_configurator/frontend_tkinter_parameter_editor.py

@@ -11,7 +11,6 @@ SPDX-License-Identifier: GPL-3.0-or-later
 """
 
 import sys
-import time
 import tkinter as tk
 from argparse import ArgumentParser, Namespace
 
@@ -21,17 +20,18 @@ from logging import error as logging_error
 from logging import getLevelName as logging_getLevelName
 from logging import warning as logging_warning
 from tkinter import filedialog, messagebox, ttk
-from typing import Literal, Optional, Union
+from typing import Optional, Union
 
 # from logging import critical as logging_critical
 from webbrowser import open as webbrowser_open  # to open the blog post documentation
 
 from ardupilot_methodic_configurator import _, __version__
 from ardupilot_methodic_configurator.backend_filesystem import LocalFilesystem
+from ardupilot_methodic_configurator.backend_filesystem_freedesktop import FreeDesktop
 from ardupilot_methodic_configurator.backend_filesystem_program_settings import ProgramSettings
 from ardupilot_methodic_configurator.backend_flightcontroller import FlightController
 from ardupilot_methodic_configurator.common_arguments import add_common_arguments
-from ardupilot_methodic_configurator.configuration_manager import ConfigurationManager
+from ardupilot_methodic_configurator.configuration_manager import ConfigurationManager, ExperimentChoice
 from ardupilot_methodic_configurator.frontend_tkinter_autoresize_combobox import AutoResizeCombobox
 from ardupilot_methodic_configurator.frontend_tkinter_base_window import (
     BaseWindow,
@@ -166,7 +166,6 @@ class ParameterEditorWindow(BaseWindow):  # pylint: disable=too-many-instance-at
         self.tempcal_imu_progress_window: ProgressWindow
         self.file_upload_progress_window: ProgressWindow
         self.skip_button: ttk.Button
-        self.last_time_asked_to_save: float = 0
         self.gui_complexity = str(ProgramSettings.get_setting("gui_complexity"))
 
         self.root.title(
@@ -209,6 +208,10 @@ class ParameterEditorWindow(BaseWindow):  # pylint: disable=too-many-instance-at
         # this one should be on top of the previous one hence the longer time
         if UsagePopupWindow.should_display("parameter_editor"):
             self.root.after(100, self.__display_usage_popup_window(self.root))  # type: ignore[arg-type]
+
+        # Set up startup notification for the main application window
+        FreeDesktop.setup_startup_notification(self.root)  # type: ignore[arg-type]
+
         self.root.mainloop()
 
     def __create_conf_widgets(self, version: str) -> None:
@@ -509,40 +512,40 @@ class ParameterEditorWindow(BaseWindow):  # pylint: disable=too-many-instance-at
         finally:
             self.tempcal_imu_progress_window.destroy()
 
-    def __handle_dialog_choice(self, result: list, dialog: tk.Toplevel, choice: Optional[bool]) -> None:
+    def __handle_dialog_choice(self, result: list, dialog: tk.Toplevel, choice: ExperimentChoice) -> None:
         result.append(choice)
         dialog.destroy()
 
-    def __should_copy_fc_values_to_file(self, selected_file: str) -> None:  # pylint: disable=too-many-locals
-        should_copy, relevant_fc_params, auto_changed_by = self.configuration_manager.should_copy_fc_values_to_file(
-            selected_file
-        )
-        if should_copy and relevant_fc_params and auto_changed_by:
-            msg = _(
-                "This configuration step requires external changes by: {auto_changed_by}\n\n"
-                "The external tool experiment procedure is described in the tuning guide.\n\n"
-                "Choose an option:\n"
-                "* CLOSE - Close the application and go perform the experiment\n"
-                "* YES - Copy current FC values to {selected_file} (if you've already completed the experiment)\n"
-                "* NO - Continue without copying values (if you haven't performed the experiment yet,"
-                " but know what you are doing)"
-            ).format(auto_changed_by=auto_changed_by, selected_file=selected_file)
-
+    def __should_copy_fc_values_to_file(self, selected_file: str) -> None:
+        def ask_user_choice(title: str, message: str, options: list[str]) -> ExperimentChoice:  # pylint: disable=too-many-locals
+            """GUI callback for asking user choice with custom dialog."""
             # Create custom dialog with Close, Yes, No buttons
             dialog = tk.Toplevel(self.root)
             # Hide dialog initially to prevent flickering
             dialog.withdraw()
             dialog.transient(self.root)
-            dialog.title(_("Update file with values from FC?"))
+            dialog.title(title)
             dialog.resizable(width=False, height=False)
             dialog.protocol("WM_DELETE_WINDOW", dialog.destroy)
 
             # Message text
-            message_label = tk.Label(dialog, text=msg, justify=tk.LEFT, padx=20, pady=10)
+            message_label = tk.Label(dialog, text=message, justify=tk.LEFT, padx=20, pady=10)
             message_label.pack(padx=10, pady=10)
 
+            # Clickable link to tuning guide
+            safe_font_config = get_safe_font_config()
+            link_label = tk.Label(
+                dialog,
+                text=_("Click here to open the Tuning Guide relevant Section"),
+                fg="blue",
+                cursor="hand2",
+                font=(str(safe_font_config["family"]), int(safe_font_config["size"]), "underline"),
+            )
+            link_label.pack(pady=(0, 10))
+            link_label.bind("<Button-1>", lambda _e: self.configuration_manager.open_documentation_in_browser(selected_file))
+
             # Result variable
-            result: list[Optional[Literal[True, False]]] = [None]
+            result: list[ExperimentChoice] = []
 
             # Button frame
             button_frame = tk.Frame(dialog)
@@ -551,25 +554,31 @@ class ParameterEditorWindow(BaseWindow):  # pylint: disable=too-many-instance-at
             # Close button (default)
             close_button = tk.Button(
                 button_frame,
-                text=_("Close"),
+                text=options[0],  # "Close"
                 width=10,
-                command=lambda: self.__handle_dialog_choice(result, dialog, choice=None),
+                command=lambda: self.__handle_dialog_choice(result, dialog, choice="close"),
             )
             close_button.pack(side=tk.LEFT, padx=5)
 
             # Yes button
             yes_button = tk.Button(
-                button_frame, text=_("Yes"), width=10, command=lambda: self.__handle_dialog_choice(result, dialog, choice=True)
+                button_frame,
+                text=options[1],
+                width=10,  # "Yes"
+                command=lambda: self.__handle_dialog_choice(result, dialog, choice=True),
             )
             yes_button.pack(side=tk.LEFT, padx=5)
 
             # No button
             no_button = tk.Button(
-                button_frame, text=_("No"), width=10, command=lambda: self.__handle_dialog_choice(result, dialog, choice=False)
+                button_frame,
+                text=options[2],
+                width=10,  # "No"
+                command=lambda: self.__handle_dialog_choice(result, dialog, choice=False),
             )
             no_button.pack(side=tk.LEFT, padx=5)
 
-            dialog.bind("<Return>", lambda _event: self.__handle_dialog_choice(result, dialog, None))
+            dialog.bind("<Return>", lambda _event: self.__handle_dialog_choice(result, dialog, choice="close"))
 
             # Center the dialog on the parent window
             dialog.deiconify()
@@ -592,23 +601,27 @@ class ParameterEditorWindow(BaseWindow):  # pylint: disable=too-many-instance-at
 
             # Wait until dialog is closed
             self.root.wait_window(dialog)
-            response = result[-1] if len(result) > 1 else None
+            return result[-1] if result else "close"
 
-            if response is True:  # Yes option
-                _params_copied = self.configuration_manager.copy_fc_values_to_file(selected_file, relevant_fc_params)
-            elif response is None:  # Close option
-                sys.exit(0)
-            # If response is False (No option), do nothing and continue
+        result = self.configuration_manager.handle_copy_fc_values_workflow(
+            selected_file,
+            ask_user_choice,
+            show_info_popup,
+        )
+
+        if result == "close":
+            # User chose to close the application
+            sys.exit(0)
 
     def __should_jump_to_file(self, selected_file: str) -> str:
-        jump_options = self.configuration_manager.get_file_jump_options(selected_file)
-        for dest_file, msg in jump_options.items():
-            if self.gui_complexity == "simple" or messagebox.askyesno(
-                _("Skip some steps?"), _(msg) if msg else _("Skip to {dest_file}?").format(**locals())
-            ):
-                self.file_selection_combobox.set(dest_file)
-                return dest_file
-        return selected_file
+        dest_file = self.configuration_manager.handle_file_jump_workflow(
+            selected_file,
+            self.gui_complexity,
+            ask_yesno_popup,
+        )
+        if dest_file != selected_file:
+            self.file_selection_combobox.set(dest_file)
+        return dest_file
 
     def __should_download_file_from_url(self, selected_file: str) -> None:
         self.configuration_manager.should_download_file_from_url_workflow(
@@ -687,7 +700,7 @@ class ParameterEditorWindow(BaseWindow):  # pylint: disable=too-many-instance-at
 
     def on_upload_selected_click(self) -> None:
         self.write_changes_to_intermediate_parameter_file()
-        selected_params = self.parameter_editor_table.get_upload_selected_params(str(self.gui_complexity))
+        selected_params = self.parameter_editor_table.get_upload_selected_params(self.gui_complexity)
         if selected_params:
             if self.configuration_manager.fc_parameters:
                 self.upload_selected_params(selected_params)
@@ -792,20 +805,10 @@ class ParameterEditorWindow(BaseWindow):  # pylint: disable=too-many-instance-at
         self.on_param_file_combobox_change(None)
 
     def write_changes_to_intermediate_parameter_file(self) -> None:
-        elapsed_since_last_ask = time.time() - self.last_time_asked_to_save
-        # if annotate parameters into files is true, we always need to write to file, because
-        # the parameter metadata might have changed, or not be present in the file.
-        # In that situation, avoid asking multiple times to write the file, by checking the time last asked
-        # But only if self.annotate_params_into_files.get()
-        if self.configuration_manager.has_unsaved_changes() or (
-            self.annotate_params_into_files.get() and elapsed_since_last_ask > 1.0
-        ):
-            msg = _("Do you want to write the changes to the {current_filename} file?").format(
-                current_filename=self.configuration_manager.current_file
-            )
-            if messagebox.askyesno(_("One or more parameters have been edited"), msg.format(**locals())):
-                self.configuration_manager.export_current_file(annotate_doc=self.annotate_params_into_files.get())
-        self.last_time_asked_to_save = time.time()
+        self.configuration_manager.handle_write_changes_workflow(
+            self.annotate_params_into_files.get(),
+            ask_yesno_popup,
+        )
 
     def close_connection_and_quit(self) -> None:
         focused_widget = self.parameter_editor_table.view_port.focus_get()

ardupilot_methodic_configurator/frontend_tkinter_show.py

@@ -19,7 +19,7 @@ from ardupilot_methodic_configurator import _
 
 
 def show_error_message(title: str, message: str) -> None:
-    root = tk.Tk()
+    root = tk.Tk(className="ArduPilotMethodicConfigurator")
     # Set the theme to 'alt'
     style = ttk.Style()
     style.theme_use("alt")
@@ -29,7 +29,7 @@ def show_error_message(title: str, message: str) -> None:
 
 
 def show_warning_message(title: str, message: str) -> None:
-    root = tk.Tk()
+    root = tk.Tk(className="ArduPilotMethodicConfigurator")
     # Set the theme to 'alt'
     style = ttk.Style()
     style.theme_use("alt")

{ardupilot_methodic_configurator-2.7.0.dist-info → ardupilot_methodic_configurator-2.7.1.dist-info}/METADATA

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: ardupilot_methodic_configurator
-Version: 2.7.0
+Version: 2.7.1
 Summary: A clear configuration sequence for ArduPilot vehicles
 Author-email: Amilcar do Carmo Lucas <amilcar.lucas@iav.de>
 Maintainer-email: Amilcar do Carmo Lucas <amilcar.lucas@iav.de>
@@ -56,7 +56,7 @@ Requires-Dist: matplotlib==3.9.4; python_version < "3.10"
 Requires-Dist: matplotlib==3.10.3; python_version >= "3.10" and python_version < "3.14"
 Requires-Dist: matplotlib==3.10.7; python_version >= "3.14"
 Requires-Dist: numpy==2.0.2; python_version < "3.13"
-Requires-Dist: numpy==2.2.2; python_version >= "3.13"
+Requires-Dist: numpy==2.3.4; python_version >= "3.13"
 Requires-Dist: pillow==11.3.0
 Requires-Dist: pip_system_certs==4.0
 Requires-Dist: platformdirs==4.4.0

{ardupilot_methodic_configurator-2.7.0.dist-info → ardupilot_methodic_configurator-2.7.1.dist-info}/RECORD

@@ -1,13 +1,14 @@
 ardupilot_methodic_configurator/AP_Motors_test.json,sha256=t9IN8K8titisdD4k24bFWXnml_SlYPWOY52CT_6jC4g,105884
 ardupilot_methodic_configurator/AP_Motors_test_schema.json,sha256=Z2ajo4IkDx80ZvTIY-lxkD_2X6nO9q-iVP-LHMBaW5w,6224
-ardupilot_methodic_configurator/__init__.py,sha256=hmPJJ6EPNLj0YkqCkchRKnVVmHx6IXPOVpTjvHjtUJw,456
-ardupilot_methodic_configurator/__main__.py,sha256=kQ7CFBtAuX9ahAYWGYOXGrtnE_6sYZiBHvYUMSVIPnc,22519
+ardupilot_methodic_configurator/__init__.py,sha256=T3lj3jDVku_7dCvZPsUfvl-skibVx6ta22GuW0SJm-w,456
+ardupilot_methodic_configurator/__main__.py,sha256=snUgab35hT7AlzlsETu5sLeIlksZMYXpgkxz3r957Es,23117
 ardupilot_methodic_configurator/annotate_params.py,sha256=0fymVFfP4y3bcNi3qJU51R7DwIMmy9XqFTRiLWk-CMU,27381
 ardupilot_methodic_configurator/argparse_check_range.py,sha256=ZxQQb7gXypG-fcrn_5rjOzq6MjVtFHT6nz0e8kJFF_Y,2474
 ardupilot_methodic_configurator/backend_filesystem.py,sha256=xVHUVkpRyKqAY2M3SoD3bVwyTgdGdh9mWS5_nF1Ditc,43152
 ardupilot_methodic_configurator/backend_filesystem_configuration_steps.py,sha256=fcItXCpuHDJsTzG-yD8Zf0eLD4dWbxU3AcjcFuEYhr0,14572
+ardupilot_methodic_configurator/backend_filesystem_freedesktop.py,sha256=d0M_jHSMrEx0RfJMQCRW0HpArDH79N-_2i5PPo-a0VM,10890
 ardupilot_methodic_configurator/backend_filesystem_json_with_schema.py,sha256=GuAhGf2liZenohgnEv9vSxQYwtuNPeZ88l1WfW1NUPs,6525
-ardupilot_methodic_configurator/backend_filesystem_program_settings.py,sha256=LDijaVMKLqc-b2ndk9iKgdKHNrFStr-1U0mZm1zt1Ds,21630
+ardupilot_methodic_configurator/backend_filesystem_program_settings.py,sha256=-CxHgX6uoTYumwEreYX6yLXx8JxC1Je2LAxcPelSrhc,16874
 ardupilot_methodic_configurator/backend_filesystem_vehicle_components.py,sha256=aDJoHk9Ovo8-AC1VkrhEypwuQD6GuByobe6r0UuTQoY,20005
 ardupilot_methodic_configurator/backend_flightcontroller.py,sha256=CDx8VjR0LO9j9uKGca99z1Ip0Lul4WhH3YpozZzgA7Q,63124
 ardupilot_methodic_configurator/backend_flightcontroller_info.py,sha256=nd4XxRLsBhAIzeMWmjUsHbOpkp4AI5nAohojyyjM0-A,12260
@@ -15,8 +16,8 @@ ardupilot_methodic_configurator/backend_internet.py,sha256=GMVZOGnyraaSsvSjbv0H2
 ardupilot_methodic_configurator/backend_mavftp.py,sha256=N4_IGDQ8dq8YfjOg6NhLkf5ZnhMMDhENkmPSzk_56aw,70861
 ardupilot_methodic_configurator/battery_cell_voltages.py,sha256=br5g5SBDss6au7HV1khpQemzo0CO9CxxHLG9roz82Yc,3143
 ardupilot_methodic_configurator/common_arguments.py,sha256=T1ioTOZn1VASBWrx968kgD8WIzE25E1_dExobkzuGoE,1140
-ardupilot_methodic_configurator/configuration_manager.py,sha256=P6qanuWepYrJedpF3vzjaea-P84vYx9BWLyeX44-ANc,65311
-ardupilot_methodic_configurator/configuration_steps_ArduCopter.json,sha256=es7arbnZMvreuDjQCUlTj5IPD2iJ7aw_icGp5oAKfkY,71012
+ardupilot_methodic_configurator/configuration_manager.py,sha256=vy83pv1NczXuinLU5YPlwLO4Xq6u_3VZbrtrz7k9Kus,70133
+ardupilot_methodic_configurator/configuration_steps_ArduCopter.json,sha256=UXIijcKNM6ig8hHZiERHL5uCqQPsoucOKZCXGOJQhJo,70936
 ardupilot_methodic_configurator/configuration_steps_ArduPlane.json,sha256=sZ2eNwhH7iwcZ5c37dvH8D_ST8eSUNophQm-yICTkJo,69045
 ardupilot_methodic_configurator/configuration_steps_Heli.json,sha256=7zzQfMrqLgoobAnKSNOLexQuntEYclbI0xBLp69sQXo,68914
 ardupilot_methodic_configurator/configuration_steps_Rover.json,sha256=3ypq0_i7IZJ3hqNRePqCh2N8GFVhbY14Ei0AOtzOlWQ,68823
@@ -41,8 +42,8 @@ ardupilot_methodic_configurator/data_model_vehicle_project_creator.py,sha256=Evy
 ardupilot_methodic_configurator/data_model_vehicle_project_opener.py,sha256=694-NPdLcjJ6uL4Fz_FBlvFFGAwVTi38Plio5ALJ-BI,7158
 ardupilot_methodic_configurator/extract_param_defaults.py,sha256=7irpBZl2tgqu5l766Jcr8yMZqrdP5tQCF7Ahh7gR9Xw,9832
 ardupilot_methodic_configurator/frontend_tkinter_autoresize_combobox.py,sha256=1lBF8oJncQEOlUT_6hY5xhz8wGJ4kYQKmiQeAsu_5d4,2716
-ardupilot_methodic_configurator/frontend_tkinter_base_window.py,sha256=w0lAtGqgIG7cG5A_myKIQ7btNE6dByohiuzu1KVOt34,16077
-ardupilot_methodic_configurator/frontend_tkinter_component_editor.py,sha256=OF9tj3QAd25Ai-ZcMxah9br5Pa0lfXIz32uXLOK1kUg,15496
+ardupilot_methodic_configurator/frontend_tkinter_base_window.py,sha256=Z0vUiH5O_WKry0It96v7AKjuMTnpdipxv65La9g8-9E,16118
+ardupilot_methodic_configurator/frontend_tkinter_component_editor.py,sha256=eiYAP_0NGx4qcrZov1aEIRXZEvPNfT2vE_Lx1FRgdKQ,15523
 ardupilot_methodic_configurator/frontend_tkinter_component_editor_base.py,sha256=jUHTLnk4RkoPN8OVmBCq_NMoV4UVMXo7xI2mrZPJmqw,33029
 ardupilot_methodic_configurator/frontend_tkinter_component_template_manager.py,sha256=BKc824yeNSz0LaqQwxHYM6jyXtXg8lJAmh_aRGPUHR8,8875
 ardupilot_methodic_configurator/frontend_tkinter_connection_selection.py,sha256=VPQGFyF-JYhB4FnsYMPCyPbAvcqn90xihcn1Z1FG2uI,17878
@@ -52,7 +53,7 @@ ardupilot_methodic_configurator/frontend_tkinter_flightcontroller_info.py,sha256
 ardupilot_methodic_configurator/frontend_tkinter_font.py,sha256=QMEQNlSMjo6SCb2tXyzVkwNRqfJ9OOA9lDtlK-Jk7jw,7733
 ardupilot_methodic_configurator/frontend_tkinter_motor_test.py,sha256=EYZVAxnM68UXRMRqdj6tqbQTBi1DXCtLie1UTgc-kAk,38478
 ardupilot_methodic_configurator/frontend_tkinter_pair_tuple_combobox.py,sha256=32PuQooNVgRacTFM27YGfcBQv1zJYhGXXHROPRdl1mE,16555
-ardupilot_methodic_configurator/frontend_tkinter_parameter_editor.py,sha256=lk-k7A6JESxQswig2xq1zaujyzJ_NT5vxVufRTd_DA0,42508
+ardupilot_methodic_configurator/frontend_tkinter_parameter_editor.py,sha256=GgBIAK13i3c0r3IDpry88vydwW80ML6gRY5Bmti0ORs,41557
 ardupilot_methodic_configurator/frontend_tkinter_parameter_editor_documentation_frame.py,sha256=nCrjx9q4D-iVg9ebe3dAMj0xHl4v_GmWWzy0tuNLBnE,7978
 ardupilot_methodic_configurator/frontend_tkinter_parameter_editor_table.py,sha256=XPgTyKcfd87bt15teoM_wLcuBYMMKQjUgOZgyzsVhbU,39016
 ardupilot_methodic_configurator/frontend_tkinter_progress_window.py,sha256=I3M93OTsV7CR_MDys5YOWnizvK8yFtOQf2I29JUw_KA,5117
@@ -60,7 +61,7 @@ ardupilot_methodic_configurator/frontend_tkinter_project_creator.py,sha256=ioRBO
 ardupilot_methodic_configurator/frontend_tkinter_project_opener.py,sha256=RbNTHMJdxkwMAByW8rmNpjQ2ANEDLMY_L_Z5vEfDbb4,9819
 ardupilot_methodic_configurator/frontend_tkinter_rich_text.py,sha256=C-DwALNgsZi2TssZ9206tITcbb5X5v-wQOWD8zIHBMk,3789
 ardupilot_methodic_configurator/frontend_tkinter_scroll_frame.py,sha256=IxYe05zzhd4lfa6oenGYqVNxXFjk10bIXlLPMqQQn-g,5320
-ardupilot_methodic_configurator/frontend_tkinter_show.py,sha256=-URnqHdoi1fDERpaUqYAzPbyFLUzugSpLa8-FZyiZ2g,5733
+ardupilot_methodic_configurator/frontend_tkinter_show.py,sha256=dfNbW0edWiqNWGfqACjcnD7uICF3lD9TMnbDQaE4sw0,5815
 ardupilot_methodic_configurator/frontend_tkinter_software_update.py,sha256=Lwj2Dk0C529S3dtkNVkFC_JdGxX_UeHEmVrlU54WxY0,4484
 ardupilot_methodic_configurator/frontend_tkinter_stage_progress.py,sha256=Hwf7dHwbQUZmHPctzv9toqyoLB14jOYKyHG0x7h8QtY,10140
 ardupilot_methodic_configurator/frontend_tkinter_template_overview.py,sha256=Vp6_hSMEkgSlixOD_b8mdUXSzSq_chfnTIh164mLeCc,18249
@@ -1447,18 +1448,18 @@ ardupilot_methodic_configurator/vehicle_templates/Rover/AION_R1/52_use_optical_f
 ardupilot_methodic_configurator/vehicle_templates/Rover/AION_R1/53_everyday_use.param,sha256=cmWo5YitMy2DF46ljT2inyQWoQa5WKy3iz3oKe6G2B4,557
 ardupilot_methodic_configurator/vehicle_templates/Rover/AION_R1/apm.pdef.xml,sha256=uJcmAzQ6o3tIOCVGZXq6wtWaMJdTcOVJfWix8qIMXs4,1796429
 ardupilot_methodic_configurator/vehicle_templates/Rover/AION_R1/vehicle_components.json,sha256=7TxmsfW_S9mpfQ5edOhzs0w3A-7i5YUMXYAuDoGT8xc,5768
-ardupilot_methodic_configurator-2.7.0.dist-info/licenses/LICENSE.md,sha256=9pLwN-5iuDanZz6Zm0V2sUrcjxyx87w1WwGV1r4Q5F4,35826
-ardupilot_methodic_configurator-2.7.0.dist-info/licenses/LICENSES/Apache-2.0.txt,sha256=m_Togr3XXo0Q_HiMU9Cnh-8PWc6tH-Hwh4wkCJb8hhA,10353
-ardupilot_methodic_configurator-2.7.0.dist-info/licenses/LICENSES/BSD-3-Clause.txt,sha256=6Nd7--lm3uRacqEuyyJ4NP9UjMfbob3GP6hibyOxcHs,1470
-ardupilot_methodic_configurator-2.7.0.dist-info/licenses/LICENSES/GPL-3.0-or-later.txt,sha256=-5gWaMGKJ54oX8TYP7oeg2zITdTapzyWl9PP0tispuA,34674
-ardupilot_methodic_configurator-2.7.0.dist-info/licenses/LICENSES/LGPL-3.0-or-later.txt,sha256=nUeUYpsPD58HjDfHv-sti_MT71zWdBgDij3e3a1JGyE,42402
-ardupilot_methodic_configurator-2.7.0.dist-info/licenses/LICENSES/MIT-CMU.txt,sha256=NmPnxSbDhJtpLqezuy_J6VvC3_4MDs-RX5ZZPdNK43g,1157
-ardupilot_methodic_configurator-2.7.0.dist-info/licenses/LICENSES/MIT.txt,sha256=MjA6uIe4yQtub-SeLwMcoBsjtRkOI2RH2U4ZRGhNx2k,1096
-ardupilot_methodic_configurator-2.7.0.dist-info/licenses/LICENSES/MPL-2.0.txt,sha256=kiHC-TYVm4RG0ykkn7TA8lvlEPRHODoPEzNqx5hWaKM,17099
-ardupilot_methodic_configurator-2.7.0.dist-info/licenses/LICENSES/PSF-2.0.txt,sha256=j5YgHA_eqI9Frr3lhzVZqmyR65psFGCSM4-PnA0Cfc4,2474
-ardupilot_methodic_configurator-2.7.0.dist-info/licenses/credits/CREDITS.md,sha256=TNrMvP82trH7zgDA-gMzXzPoRydpD-WY9MopdwQyflw,8425
-ardupilot_methodic_configurator-2.7.0.dist-info/METADATA,sha256=8TG9f-mbZPqsxnKoTay8L7JIm_NLXV330nfk3YgPSdc,32883
-ardupilot_methodic_configurator-2.7.0.dist-info/WHEEL,sha256=_zCd3N1l69ArxyTb8rzEoP9TpbYXkqRFSNOD5OuxnTs,91
-ardupilot_methodic_configurator-2.7.0.dist-info/entry_points.txt,sha256=Ymzhv3OMn_WEeBVIfldx4ceUSjGxJtaCSbZH4Y9F-dg,918
-ardupilot_methodic_configurator-2.7.0.dist-info/top_level.txt,sha256=StTz5FGzOEMSseJ1vR6azxeDQOIDg3Cq5B1_dgMjrAI,32
-ardupilot_methodic_configurator-2.7.0.dist-info/RECORD,,
+ardupilot_methodic_configurator-2.7.1.dist-info/licenses/LICENSE.md,sha256=9pLwN-5iuDanZz6Zm0V2sUrcjxyx87w1WwGV1r4Q5F4,35826
+ardupilot_methodic_configurator-2.7.1.dist-info/licenses/LICENSES/Apache-2.0.txt,sha256=m_Togr3XXo0Q_HiMU9Cnh-8PWc6tH-Hwh4wkCJb8hhA,10353
+ardupilot_methodic_configurator-2.7.1.dist-info/licenses/LICENSES/BSD-3-Clause.txt,sha256=6Nd7--lm3uRacqEuyyJ4NP9UjMfbob3GP6hibyOxcHs,1470
+ardupilot_methodic_configurator-2.7.1.dist-info/licenses/LICENSES/GPL-3.0-or-later.txt,sha256=-5gWaMGKJ54oX8TYP7oeg2zITdTapzyWl9PP0tispuA,34674
+ardupilot_methodic_configurator-2.7.1.dist-info/licenses/LICENSES/LGPL-3.0-or-later.txt,sha256=nUeUYpsPD58HjDfHv-sti_MT71zWdBgDij3e3a1JGyE,42402
+ardupilot_methodic_configurator-2.7.1.dist-info/licenses/LICENSES/MIT-CMU.txt,sha256=NmPnxSbDhJtpLqezuy_J6VvC3_4MDs-RX5ZZPdNK43g,1157
+ardupilot_methodic_configurator-2.7.1.dist-info/licenses/LICENSES/MIT.txt,sha256=MjA6uIe4yQtub-SeLwMcoBsjtRkOI2RH2U4ZRGhNx2k,1096
+ardupilot_methodic_configurator-2.7.1.dist-info/licenses/LICENSES/MPL-2.0.txt,sha256=kiHC-TYVm4RG0ykkn7TA8lvlEPRHODoPEzNqx5hWaKM,17099
+ardupilot_methodic_configurator-2.7.1.dist-info/licenses/LICENSES/PSF-2.0.txt,sha256=j5YgHA_eqI9Frr3lhzVZqmyR65psFGCSM4-PnA0Cfc4,2474
+ardupilot_methodic_configurator-2.7.1.dist-info/licenses/credits/CREDITS.md,sha256=5IpN1DQO8dsbw8bI2UwtVoQFocM8BVinr6O4XV36AqE,8573
+ardupilot_methodic_configurator-2.7.1.dist-info/METADATA,sha256=Ai5nfcijuThpOoohUcHIABpRg5wFmQqOHn-bTSAGmAU,32883
+ardupilot_methodic_configurator-2.7.1.dist-info/WHEEL,sha256=_zCd3N1l69ArxyTb8rzEoP9TpbYXkqRFSNOD5OuxnTs,91
+ardupilot_methodic_configurator-2.7.1.dist-info/entry_points.txt,sha256=Ymzhv3OMn_WEeBVIfldx4ceUSjGxJtaCSbZH4Y9F-dg,918
+ardupilot_methodic_configurator-2.7.1.dist-info/top_level.txt,sha256=StTz5FGzOEMSseJ1vR6azxeDQOIDg3Cq5B1_dgMjrAI,32
+ardupilot_methodic_configurator-2.7.1.dist-info/RECORD,,

{ardupilot_methodic_configurator-2.7.0.dist-info → ardupilot_methodic_configurator-2.7.1.dist-info}/licenses/credits/CREDITS.md

@@ -91,3 +91,4 @@ These books helped shape this software:
 - [Clean Architecture by Robert C. Martin](https://www.oreilly.com/library/view/clean-architecture/9780134494272/)
 - [Modern Software Engineering by David Farley](https://www.oreilly.com/library/view/modern-software-engineering/9780137314942/)
 - [The DevOps Handbook by Gene Kim, Patrick Debois, John Willis, and Jez Humble](https://www.oreilly.com/library/view/the-devsecops-handbook/9781098182281/)
+- [Fundamentals of Software Architecture by Mark Richards, Neal Ford](https://www.oreilly.com/library/view/fundamentals-of-software/9781492043447/)