From ada5ea8fe7b9ea72fafe5a18e9f9d281a2617616 Mon Sep 17 00:00:00 2001
From: "Dr.-Ing. Amilcar do Carmo Lucas" <amilcar.lucas@iav.de>
Date: Mon, 15 Apr 2024 00:14:51 +0200
Subject: [PATCH] IMPROVEMENT: reorder the command line parameters in a logical
 way, connection first, directory afterwards

---
 .../ardupilot_methodic_configurator.py        | 20 +++++++++----------
 1 file changed, 10 insertions(+), 10 deletions(-)

diff --git a/MethodicConfigurator/ardupilot_methodic_configurator.py b/MethodicConfigurator/ardupilot_methodic_configurator.py
index 31a23c4..67cc1ab 100755
--- a/MethodicConfigurator/ardupilot_methodic_configurator.py
+++ b/MethodicConfigurator/ardupilot_methodic_configurator.py
@@ -54,6 +54,16 @@ def argument_parser():
                         default="",
                         help='MAVLink connection string to the flight controller. Defaults to autodetection'
                         )  # pylint: disable=R0801
+    parser.add_argument('-r', '--reboot-time',
+                        type=int,
+                        default=7,
+                        help='Flight controller reboot time. '
+                        'Default is %(default)s')  # pylint: disable=R0801
+    parser.add_argument('-t', '--vehicle-type',
+                        choices=['AP_Periph', 'AntennaTracker', 'ArduCopter', 'ArduPlane',
+                                 'ArduSub', 'Blimp', 'Heli', 'Rover', 'SITL'],
+                        default='ArduCopter',
+                        help='The type of the vehicle. Defaults to ArduCopter')  # pylint: disable=R0801
     parser.add_argument('--vehicle-dir',
                         type=str,
                         default=os_getcwd(),
@@ -69,16 +79,6 @@ def argument_parser():
                         default='INFO',
                         choices=['DEBUG', 'INFO', 'WARNING', 'ERROR', 'CRITICAL'],
                         help='Logging level (default is INFO).')  # pylint: disable=R0801
-    parser.add_argument('-r', '--reboot-time',
-                        type=int,
-                        default=7,
-                        help='Flight controller reboot time. '
-                        'Default is %(default)s')  # pylint: disable=R0801
-    parser.add_argument('-t', '--vehicle-type',
-                        choices=['AP_Periph', 'AntennaTracker', 'ArduCopter', 'ArduPlane',
-                                 'ArduSub', 'Blimp', 'Heli', 'Rover', 'SITL'],
-                        default='ArduCopter',
-                        help='The type of the vehicle. Defaults to ArduCopter')  # pylint: disable=R0801
     parser.add_argument('-v', '--version',
                         action='version',
                         version=f'%(prog)s {VERSION}',