fix: freezegun version

fix: py version

fix: env class

fix: ut issues

fix: timezone issue

fix: timezone issue

Author: Udayraj123

.github/workflows/pyright.yml

@@ -12,7 +12,7 @@ jobs:
 
     strategy:
       matrix:
-        python-version: ["3.9", "3.10", "3.11", "3.12", "3.13"]
+        python-version: ["3.11", "3.13"]
 
     steps:
       - uses: jakebailey/pyright-action@v2

.github/workflows/ruff.yml

@@ -12,7 +12,7 @@ jobs:
 
     strategy:
       matrix:
-        python-version: ["3.9", "3.10", "3.11", "3.12", "3.13"]
+        python-version: ["3.11", "3.13"]
 
     steps:
       - name: Checkout
@@ -31,7 +31,8 @@ jobs:
 
     strategy:
       matrix:
-        python-version: ["3.9", "3.10", "3.11", "3.12", "3.13"]
+        # Min-max supported versions
+        python-version: ["3.11", "3.13"]
 
     steps:
       - name: Checkout

.github/workflows/test.yml

@@ -30,6 +30,9 @@ jobs:
         run: |
           set -o pipefail
           uv run pytest --cov-fail-under=70 --junitxml=./.github-pytest/pytest-output.xml | tee .github-pytest/pytest-coverage.txt
+        env:
+          OMR_CHECKER_CONTAINER: true
+          TZ: Asia/Kolkata
 
       - name: Pytest coverage comment
         uses: MishaKav/pytest-coverage-comment@v1

pyproject.toml

@@ -32,7 +32,7 @@ dependencies = [
 dev = [
     "coverage==7.6.0",
     "flake8==7.0.0",
-    "freezegun==1.4.0",
+    "freezegun>=1.4.0",
     "mkdocs-material>=9.5.50",
     "Pillow==11.1.0",
     "pre-commit>=4.1.0",

src/__init__.py

@@ -1,17 +1,15 @@
 # https://docs.python.org/3/tutorial/modules.html#:~:text=The%20__init__.py,on%20the%20module%20search%20path.
 
 # Note: This import is added to the root __init__.py to adjust for perceived loading time
-import os
 from pathlib import Path
 
+from src.utils.env import env
 from src.utils.logger import logger
 
 # It takes a few seconds for the imports
 logger.info("Loading OMRChecker modules...")
 
-VIRTUAL_ENV = os.environ.get("VIRTUAL_ENV", "Not Present")
+logger.info(f"VIRTUAL_ENV: {env.VIRTUAL_ENV}")
 
-logger.info(f"VIRTUAL_ENV: {VIRTUAL_ENV}")
-
-if not Path(VIRTUAL_ENV).exists():
+if not Path(env.VIRTUAL_ENV).exists():
     logger.warning("Your virtual Environment doesn't exist at the path!")

src/algorithm/template/detection/bubbles_threshold/interpretation.py

@@ -1,5 +1,4 @@
 import math
-import os
 import random
 import re
 
@@ -16,6 +15,7 @@
     BubblesFieldInterpretationDrawing,
 )
 from src.algorithm.template.layout.field.base import Field
+from src.utils.env import env
 from src.utils.logger import logger
 
 
@@ -226,7 +226,7 @@ def get_global_threshold(
     def plot_for_global_threshold(
         plot_means_and_refs, plot_title, file_level_fallback_threshold, thr2
     ) -> None:
-        if os.environ.get("OMR_CHECKER_CONTAINER"):
+        if env.OMR_CHECKER_CONTAINER:
             return
         _, ax = plt.subplots()
         # TODO: move into individual utils

src/processors/AutoRotate.py

@@ -22,7 +22,7 @@ def __init__(self, *args, **kwargs) -> None:
         if not path.exists():
             msg = f"Reference image for AutoRotate not found at {path}"
             raise Exception(msg)
-        self.reference_image = cv2.imread(path, cv2.IMREAD_GRAYSCALE)
+        self.reference_image = cv2.imread(str(path), cv2.IMREAD_GRAYSCALE)
         self.marker_dimensions = self.options.get("markerDimensions", None)
         self.resized_reference = self.reference_image
         self.threshold = self.options.get("threshold", None)

src/tests/utils.py

@@ -33,6 +33,10 @@ def run_entry_point(input_path, output_dir) -> None:
         "outputMode": "default",
     }
     with freeze_time(FROZEN_TIMESTAMP):
+        import time
+
+        frozen_time = time.time()
+        assert frozen_time == 0, f"Frozen time is not 0: {frozen_time}"
         entry_point_for_args(args)
 
 

src/utils/env.py

@@ -0,0 +1,11 @@
+import os
+
+
+class Env:
+    # Container for all environment variables used in OMRChecker
+    CI = os.environ.get("CI")
+    OMR_CHECKER_CONTAINER = os.environ.get("OMR_CHECKER_CONTAINER", CI)
+    VIRTUAL_ENV = os.environ.get("VIRTUAL_ENV", "Not Present")
+
+
+env = Env()

src/utils/interaction.py

@@ -1,4 +1,3 @@
-import os
 from dataclasses import dataclass
 
 import cv2
@@ -7,14 +6,15 @@
 
 from src.utils.constants import WAIT_KEYS
 from src.utils.drawing import DrawingUtils
+from src.utils.env import env
 from src.utils.image import ImageUtils
 from src.utils.logger import logger
 
 
 @dataclass
 class ImageMetrics:
     # TODO: fix window metrics doesn't account for the doc/taskbar on macos
-    if os.environ.get("OMR_CHECKER_CONTAINER"):
+    if env.OMR_CHECKER_CONTAINER:
         monitor_window = Monitor(0, 0, 1000, 1000, 100, 100, "FakeMonitor", False)  # NOQA: FBT003
     else:
         monitor_window = get_monitors()[0]
@@ -134,7 +134,7 @@ def show(
         config=None,
     ) -> None:
         image_metrics = InteractionUtils.image_metrics
-        if os.environ.get("OMR_CHECKER_CONTAINER"):
+        if env.OMR_CHECKER_CONTAINER:
             return
         if image is None:
             logger.warning(f"'{name}' - NoneType image to show!")