fix on-the-fly installs in clone and pip installed repo (#2205)

Author: mikel-brostrom

boxmot/configs/trackers/bytetrack.yaml

@@ -9,7 +9,7 @@ track_thresh:
   range: [0.4, 0.6]
 
 track_buffer:
-  type: randint
+  type: qrandint
   default: 30  # from the default parameters
   range: [10, 61, 10]  # step size of 10, upper bound exclusive
 

boxmot/configs/trackers/deepocsort.yaml

@@ -4,7 +4,7 @@ det_thresh:
   range: [0.3, 0.6]
 
 max_age:
-  type: randint
+  type: qrandint
   default: 30  # from the default parameters
   range: [10, 61, 10]  # step size of 10, upper bound exclusive
 

boxmot/configs/trackers/hybridsort.yaml

@@ -24,7 +24,7 @@ use_custom_kf:
   options: [True, False]
 
 longterm_bank_length:
-  type: randint
+  type: qrandint
   default: 30
   range: [10, 61, 10]  # step size 10
 

boxmot/utils/__init__.py

@@ -11,6 +11,14 @@
 from loguru import logger
 
 ROOT = Path(__file__).resolve().parents[2]
+
+# If running from a cloned repository, prefer the local path
+# This handles the case where the package is installed in site-packages
+# but the user is running from the source root and expects local data/configs.
+_local_root = Path.cwd()
+if (_local_root / "pyproject.toml").is_file() and (_local_root / "boxmot").is_dir():
+    ROOT = _local_root
+
 DATA = ROOT / "data"
 TOML = ROOT / "pyproject.toml"
 

boxmot/utils/checks.py

@@ -8,7 +8,7 @@
 from packaging.requirements import Requirement
 
 # Replace this import with your logger, or use logging.getLogger(__name__)
-from boxmot.utils import logger as LOGGER
+from boxmot.utils import logger as LOGGER, ROOT
 
 REQUIREMENTS_FILE = Path("requirements.txt")
 
@@ -21,8 +21,7 @@ class RequirementsChecker:
       - Check/install a list of requirement specifiers (e.g., ["yolox", "onnx>=1.15"])
       - Read and install from a requirements.txt
       - Install a uv dependency *group* (requires uv)
-      - Install a project *extra* (PEP 621 optional-dependencies) via uv or pip
-      - Falls back to pip if uv is not available
+      - Install a project *extra* (PEP 621 optional-dependencies) via uv
       - Backward-compatible alias: `cmds` == `extra_args`
     """
 
@@ -35,7 +34,6 @@ def __init__(
         """
         self.group = group
         self.requirements_file = requirements_file
-        self._uv_available = shutil.which("uv") is not None
 
     # ---------- public API ----------
 
@@ -84,25 +82,27 @@ def sync_extra(
     ):
         """
         Install a project *extra* (PEP 621 optional-dependencies).
-        - From source checkout + uv available: uv pip install ".[extra]"
-        - From source checkout w/o uv:        python -m pip install ".[extra]"
-        - From PyPI install:                  python -m pip install "boxmot[extra]"
+        - From source checkout + uv available: uv pip install -e ".[extra]"
+        - From PyPI install:                  uv pip install "boxmot[extra]"
         """
         if not extra:
             raise ValueError("Extra name must be provided (e.g. 'openvino', 'export').")
         LOGGER.warning(f"Installing extra '{extra}'...")
 
-        in_source = Path("pyproject.toml").is_file()
+        # Check if we are running from a source install (editable)
+        # ROOT is the package root. If pyproject.toml exists there, it's an editable install.
+        root_pyproject = ROOT / "pyproject.toml"
+        
         cmd: list[str]
 
-        if in_source:
-            if self._uv_available:
-                cmd = ["uv", "pip", "install", "--no-cache-dir", f".[{extra}]"]
-            else:
-                cmd = [sys.executable, "-m", "pip", "install", f".[{extra}]"]
+        if root_pyproject.is_file():
+            # Editable install detected or running from source root
+            # We use ROOT to point to the source directory
+            target = f"{ROOT}[{extra}]"
+            cmd = ["uv", "pip", "install", "--no-cache-dir", "-e", target]
         else:
             # Installed from PyPI: install the published dist extra
-            cmd = [sys.executable, "-m", "pip", "install", f"boxmot[{extra}]"]
+            cmd = ["uv", "pip", "install", f"boxmot[{extra}]"]
 
         if extra_args:
             cmd.extend(extra_args)
@@ -120,17 +120,14 @@ def _install_packages(
         self, packages: Sequence[str], extra_args: Optional[Sequence[str]] = None
     ):
         """
-        Install an explicit list of requirement specifiers with uv (if present) or pip.
+        Install an explicit list of requirement specifiers with uv.
         """
         try:
             LOGGER.warning(
                 f"\nMissing or mismatched packages: {', '.join(packages)}\n"
                 "Attempting installation..."
             )
-            if self._uv_available:
-                cmd = ["uv", "pip", "install", "--no-cache-dir"]
-            else:
-                cmd = ["pip", "install"]
+            cmd = ["uv", "pip", "install", "--no-cache-dir"]
 
             if extra_args:
                 cmd += list(extra_args)
@@ -140,4 +137,4 @@ def _install_packages(
             LOGGER.info("All missing packages were installed successfully.")
         except Exception as e:
             LOGGER.error(f"Failed to install packages: {e}")
-            raise RuntimeError(f"Failed to install packages: {e}")
+            raise RuntimeError(f"Failed to install packages: {e}")