Remove argsparse from hailo

Author: jrrodri

abraia/hailo/camera_utils.py

@@ -486,6 +486,7 @@ def _get_usb_video_devices_windows():
     hailo_logger.debug(f"USB video devices found on Windows: {usb_video_devices}")
     return usb_video_devices
 
+
 def main():
     hailo_logger.debug("Running main() to check for USB cameras.")
     usb_video_devices = get_usb_video_devices()

abraia/hailo/config_manager.py

@@ -35,8 +35,6 @@
 
 from __future__ import annotations
 
-import argparse
-import sys
 import yaml
 from dataclasses import dataclass
 from functools import lru_cache
@@ -1578,71 +1576,3 @@ def _print_result(name: str, result, error=None, indent=2):
     else:
         print("\n  🎉 All functions passed!")
         return True
-
-
-def main():
-
-    """CLI entry point."""
-    parser = argparse.ArgumentParser(
-        description="Hailo Apps Configuration Manager",
-        formatter_class=argparse.RawDescriptionHelpFormatter,
-        epilog="""
-Examples:
-  python -m hailo_apps.config.config_manager --dry-run
-  python -m hailo_apps.config.config_manager --test-all
-  python -m hailo_apps.config.config_manager --list-apps
-  python -m hailo_apps.config.config_manager --show-models detection hailo8
-  python -m hailo_apps.config.config_manager --show-paths
-        """,
-    )
-
-    parser.add_argument(
-        "--dry-run",
-        action="store_true",
-        help="Validate all configuration files and show summary",
-    )
-    parser.add_argument(
-        "--list-apps", action="store_true", help="List all available applications"
-    )
-    parser.add_argument(
-        "--show-models",
-        nargs=2,
-        metavar=("APP", "ARCH"),
-        help="Show models for an app and architecture",
-    )
-    parser.add_argument(
-        "--show-paths",
-        action="store_true",
-        help="Show configuration file paths",
-    )
-    parser.add_argument(
-        "--test-all",
-        action="store_true",
-        help="Test all API functions and print their outputs",
-    )
-
-    args = parser.parse_args()
-
-    if args.dry_run:
-        success = _dry_run()
-        sys.exit(0 if success else 1)
-    elif args.test_all:
-        success = _test_all_functions()
-        sys.exit(0 if success else 1)
-    elif args.list_apps:
-        _list_apps()
-    elif args.show_models:
-        _show_models(args.show_models[0], args.show_models[1])
-    elif args.show_paths:
-        _print_header("Configuration File Paths")
-        print(f"  Repo Root: {ConfigPaths.repo_root()}")
-        print(f"  Main Config: {ConfigPaths.main_config()}")
-        print(f"  Resources Config: {ConfigPaths.resources_config()}")
-        print(f"  Test Definition: {ConfigPaths.test_definition_config()}")
-        print(f"  Test Control: {ConfigPaths.test_control_config()}")
-    else:
-        parser.print_help()
-
-
-if __name__ == "__main__":
-    main()

abraia/hailo/core.py

@@ -1,5 +1,5 @@
 from __future__ import annotations
-"""Core helpers: arch detection, parser, buffer utils, model resolution."""
+"""Core helpers: arch detection, buffer utils, model resolution."""
 
 import os
 import queue
@@ -9,8 +9,6 @@
 from typing import Optional, Tuple
 from dataclasses import dataclass
 from dotenv import load_dotenv
-from . import parser as common_parser
-import argparse
 
 from .defines import (
     DEFAULT_DOTENV_PATH,
@@ -75,31 +73,6 @@ def load_environment(env_file=DEFAULT_DOTENV_PATH, required_vars=None) -> bool:
     return True
 
 
-def get_base_parser():
-    """Proxy to the shared base parser implementation."""
-    return common_parser.get_base_parser()
-
-
-def get_pipeline_parser():
-    """Proxy to the shared pipeline parser implementation."""
-    return common_parser.get_pipeline_parser()
-
-
-def get_standalone_parser():
-    """Proxy to the shared standalone parser implementation."""
-    return common_parser.get_standalone_parser()
-
-
-def get_default_parser():
-    """Legacy proxy preserved for backward compatibility."""
-    return common_parser.get_default_parser()
-
-
-def configure_multi_model_hef_path(parser):
-    """Proxy to configure --hef-path for multi-model apps."""
-    return common_parser.configure_multi_model_hef_path(parser)
-
-
 def get_resource_path(
     pipeline_name: str, resource_type: str, arch: str | None = None, model: str | None = None
 ) -> Path | None:
@@ -790,7 +763,7 @@ def _map_app_to_resource_group(app_name: str) -> str:
 # =============================================================================
 # Handle and Resolve Common Args
 # =============================================================================
-def handle_and_resolve_args(args: argparse.Namespace, APP_NAME: str, multi_hef: bool = False, using_onnx_pp=False) -> None:
+def handle_and_resolve_args(args, APP_NAME: str, multi_hef: bool = False, using_onnx_pp=False) -> None:
     """
     Handle common CLI argument logic for Hailo applications.
 
@@ -805,7 +778,7 @@ def handle_and_resolve_args(args: argparse.Namespace, APP_NAME: str, multi_hef:
     - This helper is intended mainly for standalone applications.
 
     Args:
-        args: Parsed argparse.Namespace from the application
+        args: Parsed args from the application
         APP_NAME: The application name for model/input resolution
         using_onnx_pp: Whether the app uses ONNX postprocessing
     """

abraia/hailo/download_resources.py

@@ -16,7 +16,6 @@
 - Force-redownload capability
 """
 
-import argparse
 import os
 import sys
 import tempfile
@@ -1560,149 +1559,3 @@ def list_models_for_arch(
     if include_extra:
         total_msg += f", {len(extra_models)} extra model(s)"
     print(total_msg)
-
-
-# =============================================================================
-# CLI Entry Point
-# =============================================================================
-
-def main():
-    parser = argparse.ArgumentParser(
-        description="Download Hailo resources (models, videos, images, configs)",
-        formatter_class=argparse.RawDescriptionHelpFormatter,
-        epilog="""
-Examples:
-  # Download default resources for detected architecture
-  %(prog)s
-
-  # Download all models (default + extra) for all apps
-  %(prog)s --all
-
-  # Download resources for a specific app
-  %(prog)s --group detection
-
-  # Download a specific resource (strict targeted mode)
-  %(prog)s --group detection --resource-type model --resource-name yolov8m
-  %(prog)s --group detection --resource-type image --resource-name bus.jpg
-  %(prog)s --group detection --resource-type video --resource-name example.mp4
-
-  # Download for a specific architecture
-  %(prog)s --arch hailo10h
-
-  # Preview what would be downloaded (dry run)
-  %(prog)s --dry-run
-
-  # Force re-download existing files
-  %(prog)s --force
-
-  # Download gen-ai app (auto-includes gen-ai models)
-  %(prog)s --group vlm_chat --arch hailo10h
-
-  # Include gen-ai apps in bulk download (downloads ALL apps including VLM, LLM, Whisper)
-  %(prog)s --all --include-gen-ai --arch hailo10h
-"""
-    )
-    
-    parser.add_argument(
-        "--all",
-        action="store_true",
-        help="Download all models (default + extra) for all apps"
-    )
-    parser.add_argument(
-        "--config",
-        type=str,
-        default=DEFAULT_RESOURCES_CONFIG_PATH,
-        help="Path to config file"
-    )
-    parser.add_argument(
-        "--arch",
-        type=str,
-        default=None,
-        choices=["hailo8", "hailo8l", "hailo10h"],
-        help="Hailo architecture override"
-    )
-    parser.add_argument(
-        "--group",
-        type=str,
-        default=None,
-        help="Group/app name to download resources for (e.g., detection, vlm_chat)"
-    )
-    parser.add_argument(
-        "--resource-name",
-        type=str,
-        default=None,
-        help="Specific resource name to download"
-    )
-    parser.add_argument(
-        "--resource-type",
-        type=str,
-        default=None,
-        choices=sorted(RESOURCE_TYPES),
-        help="Type of the resource specified by --resource-name (model/image/video/onnx)"
-    )
-    parser.add_argument(
-        "--list-models",
-        action="store_true",
-        help="List all available models for the detected/selected architecture"
-    )
-    parser.add_argument(
-        "--dry-run",
-        action="store_true",
-        help="Show what would be downloaded without actually downloading"
-    )
-    parser.add_argument(
-        "--force",
-        action="store_true",
-        help="Force re-download even if files already exist"
-    )
-    parser.add_argument(
-        "--no-parallel",
-        action="store_true",
-        help="Disable parallel downloads (download sequentially)"
-    )
-    parser.add_argument(
-        "--include-gen-ai",
-        action="store_true",
-        help="Include gen-ai apps in bulk downloads (not needed with --group, gen-ai models are auto-included when group has them)"
-    )
-    
-    args = parser.parse_args()
-    
-    load_environment()
-    
-    # List models and exit
-    if args.list_models:
-        list_models_for_arch(
-            resource_config_path=args.config,
-            arch=args.arch,
-            include_extra=True
-        )
-        return
-    
-    # Validate targeted mode arguments
-    if args.resource_name and not args.resource_type:
-        parser.error("--resource-name requires --resource-type")
-
-    if args.resource_type and not args.resource_name:
-        parser.error("--resource-type requires --resource-name")
-
-
-    # Download resources
-    download_resources(
-        resource_config_path=args.config,
-        arch=args.arch,
-        group=args.group,
-        all_models=args.all,
-        resource_name=args.resource_name,
-        resource_type=args.resource_type,
-        dry_run=args.dry_run,
-        force=args.force,
-        parallel=not args.no_parallel,
-        include_gen_ai=args.include_gen_ai
-    )
-    
-    hailo_logger.info("Resource download completed.")
-
-
-if __name__ == "__main__":
-    main()

abraia/hailo/hailo_logger.py

@@ -266,45 +266,6 @@ def get_logger(name: str) -> logging.Logger:
     return logging.getLogger(name)
 
 
-def add_logging_cli_args(parser: Any) -> None:
-    """Add --log-level/--debug/--log-file flags to an argparse parser.
-
-    Typical usage:
-
-        from hailo_logger import add_logging_cli_args, init_logging, level_from_args
-
-        parser = argparse.ArgumentParser()
-        add_logging_cli_args(parser)
-        args = parser.parse_args()
-        init_logging(level=level_from_args(args), log_file=args.log_file)
-    """
-    parser.add_argument(
-        "--log-level",
-        default=os.getenv("HAILO_LOG_LEVEL", "INFO"),
-        choices=[k.lower() for k in _LEVELS.keys()],
-        help="Logging level (default: %(default)s or $HAILO_LOG_LEVEL).",
-    )
-    parser.add_argument(
-        "--debug",
-        action="store_true",
-        help="Shortcut for DEBUG log level (overrides --log-level).",
-    )
-    parser.add_argument(
-        "--log-file",
-        default=os.getenv("HAILO_LOG_FILE"),
-        help="Optional log file path (also respects $HAILO_LOG_FILE).",
-    )
-
-
-def level_from_args(args: Any) -> str:
-    """Resolve level string from argparse args."""
-    return (
-        "DEBUG"
-        if getattr(args, "debug", False)
-        else str(getattr(args, "log_level", "INFO")).upper()
-    )
-
-
 # Auto-configuration: set HAILO_LOG_AUTOCONFIG=1 to call init_logging() at import time.
 # Level comes from HAILO_LOG_LEVEL env var (defaults to INFO).
 # Noisy loggers are NOT suppressed when level is set via env var.