kinemotion 0.42.0__tar.gz → 0.43.0__tar.gz

{kinemotion-0.42.0 → kinemotion-0.43.0}/CHANGELOG.md

@@ -7,6 +7,24 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 <!-- version list -->
 
+## v0.43.0 (2025-12-11)
+
+### Continuous Integration
+
+- Remove b ad workflow
+  ([`7584229`](https://github.com/feniix/kinemotion/commit/7584229682eb94ddf13a6becac7ad31b96d62f0f))
+
+### Features
+
+- Optimize debug video generation and improve code quality
+  ([`81ce698`](https://github.com/feniix/kinemotion/commit/81ce698d5a0fa8dff6c20c6870a7bee52080cef0))
+
+### Testing
+
+- Optimize test suite execution speed
+  ([`26ea210`](https://github.com/feniix/kinemotion/commit/26ea21072745ef606405b5f6f5277dd422636baf))
+
+
 ## v0.42.0 (2025-12-10)
 
 ### Documentation

{kinemotion-0.42.0 → kinemotion-0.43.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: kinemotion
-Version: 0.42.0
+Version: 0.43.0
 Summary: Video-based kinematic analysis for athletic performance
 Project-URL: Homepage, https://github.com/feniix/kinemotion
 Project-URL: Repository, https://github.com/feniix/kinemotion

{kinemotion-0.42.0 → kinemotion-0.43.0}/justfile

@@ -1,3 +1,14 @@
+# Check python sanity
+python-check:
+    @echo "Running pyright..."
+    uv run pyright
+    (cd backend && uv run pyright)
+    @echo "Running ruff check and format..."
+    uv run ruff check --fix
+    uv run ruff format
+    (cd backend && uv run ruff check --fix)
+    (cd backend && uv run ruff format)
+
 # Clean generated and temporary files (preserves .claude/, .cursor/, .mcp.json)
 clean:
     @echo "Cleaning generated files..."

{kinemotion-0.42.0 → kinemotion-0.43.0}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "kinemotion"
-version = "0.42.0"
+version = "0.43.0"
 description = "Video-based kinematic analysis for athletic performance"
 readme = "README.md"
 requires-python = ">=3.10,<3.13"

{kinemotion-0.42.0 → kinemotion-0.43.0}/src/kinemotion/api.py

@@ -1,4 +1,4 @@
-"""Public API for programmatic use of kinemotion analysis."""
+"Public API for programmatic use of kinemotion analysis."
 
 import time
 from collections.abc import Callable
@@ -37,7 +37,6 @@ from .core.smoothing import smooth_landmarks, smooth_landmarks_advanced
 from .core.timing import PerformanceTimer
 from .core.video_io import VideoProcessor
 from .dropjump.analysis import (
-    ContactState,
     compute_average_foot_position,
     detect_ground_contact,
     find_contact_phases,
@@ -370,105 +369,15 @@ def _convert_timer_to_stage_names(
         "json_serialization": "JSON serialization",
         "debug_video_generation": "Debug video generation",
         "debug_video_reencode": "Debug video re-encoding",
+        "frame_rotation": "Frame rotation",
+        "debug_video_resize": "Debug video resizing",
+        "debug_video_copy": "Debug video frame copy",
+        "debug_video_draw": "Debug video drawing",
+        "debug_video_write": "Debug video encoding",
     }
     return {mapping.get(k, k): v for k, v in timer_metrics.items()}
 
 
-def _generate_dropjump_outputs(
-    metrics: DropJumpMetrics,
-    json_output: str | None,
-    output_video: str | None,
-    frames: list,
-    smoothed_landmarks: list,
-    contact_states: list[ContactState],
-    video: VideoProcessor,
-    verbose: bool,
-    timer: PerformanceTimer | None = None,
-) -> None:
-    """Generate JSON and debug video outputs if requested.
-
-    Args:
-        metrics: Calculated drop jump metrics
-        json_output: Optional path for JSON output
-        output_video: Optional path for debug video
-        frames: List of video frames
-        smoothed_landmarks: Smoothed landmark sequence
-        contact_states: Ground contact state for each frame
-        video: Video processor with dimensions and fps
-        verbose: Print progress messages
-        timer: Optional PerformanceTimer for measuring operations
-    """
-    # Save JSON if requested
-    if json_output:
-        import json
-
-        if timer:
-            with timer.measure("json_serialization"):
-                output_path = Path(json_output)
-                metrics_dict = metrics.to_dict()
-                json_str = json.dumps(metrics_dict, indent=2)
-                output_path.write_text(json_str)
-        else:
-            output_path = Path(json_output)
-            metrics_dict = metrics.to_dict()
-            json_str = json.dumps(metrics_dict, indent=2)
-            output_path.write_text(json_str)
-
-        if verbose:
-            print(f"Metrics written to: {json_output}")
-
-    # Generate debug video if requested
-    if output_video:
-        if verbose:
-            print(f"Generating debug video: {output_video}")
-
-        if timer:
-            with timer.measure("debug_video_generation"):
-                with DebugOverlayRenderer(
-                    output_video,
-                    video.width,
-                    video.height,
-                    video.display_width,
-                    video.display_height,
-                    video.fps,
-                ) as renderer:
-                    for i, frame in enumerate(frames):
-                        annotated = renderer.render_frame(
-                            frame,
-                            smoothed_landmarks[i],
-                            contact_states[i],
-                            i,
-                            metrics,
-                            use_com=False,
-                        )
-                        renderer.write_frame(annotated)
-            # Capture re-encoding duration separately
-            with timer.measure("debug_video_reencode"):
-                pass  # Re-encoding happens in context manager __exit__
-        else:
-            with DebugOverlayRenderer(
-                output_video,
-                video.width,
-                video.height,
-                video.display_width,
-                video.display_height,
-                video.fps,
-            ) as renderer:
-                for i, frame in enumerate(frames):
-                    annotated = renderer.render_frame(
-                        frame,
-                        smoothed_landmarks[i],
-                        contact_states[i],
-                        i,
-                        metrics,
-                        use_com=False,
-                    )
-                    renderer.write_frame(annotated)
-
-        if verbose:
-            print(f"Debug video saved: {output_video}")
-
-
 @dataclass
 class DropJumpVideoResult:
     """Result of processing a single drop jump video."""
@@ -740,6 +649,7 @@ def process_dropjump_video(
                             video.display_width,
                             video.display_height,
                             video.fps,
+                            timer=timer,
                         ) as renderer:
                             for i, frame in enumerate(frames):
                                 annotated = renderer.render_frame(
@@ -762,6 +672,7 @@ def process_dropjump_video(
                         video.display_width,
                         video.display_height,
                         video.fps,
+                        timer=timer,
                     ) as renderer:
                         for i, frame in enumerate(frames):
                             annotated = renderer.render_frame(
@@ -843,7 +754,7 @@ def process_dropjump_video(
                     dur_ms = duration * 1000
                     print(f"{stage:.<40} {dur_ms:>6.0f}ms ({percentage:>5.1f}%)")
                 total_ms = total_time * 1000
-                print(f"{'Total':.>40} {total_ms:>6.0f}ms (100.0%)")
+                print(f"{('Total'):.>40} {total_ms:>6.0f}ms (100.0%)")
                 print()
                 print("Analysis complete!")
 
@@ -994,95 +905,6 @@ class CMJVideoResult:
     processing_time: float = 0.0
 
 
-def _generate_cmj_outputs(
-    output_video: str | None,
-    json_output: str | None,
-    metrics: CMJMetrics,
-    frames: list,
-    smoothed_landmarks: list,
-    video_width: int,
-    video_height: int,
-    video_display_width: int,
-    video_display_height: int,
-    video_fps: float,
-    verbose: bool,
-    timer: PerformanceTimer | None = None,
-) -> None:
-    """Generate JSON and debug video outputs for CMJ analysis.
-
-    Args:
-        output_video: Optional path for debug video output
-        json_output: Optional path for JSON output
-        metrics: Calculated CMJ metrics
-        frames: List of video frames
-        smoothed_landmarks: Smoothed landmark sequence
-        video_width: Video width in pixels
-        video_height: Video height in pixels
-        video_display_width: Display width considering aspect ratio
-        video_display_height: Display height considering aspect ratio
-        video_fps: Video frames per second
-        verbose: Print progress messages
-        timer: Optional PerformanceTimer for measuring operations
-    """
-    if json_output:
-        import json
-
-        if timer:
-            with timer.measure("json_serialization"):
-                output_path = Path(json_output)
-                metrics_dict = metrics.to_dict()
-                json_str = json.dumps(metrics_dict, indent=2)
-                output_path.write_text(json_str)
-        else:
-            output_path = Path(json_output)
-            metrics_dict = metrics.to_dict()
-            json_str = json.dumps(metrics_dict, indent=2)
-            output_path.write_text(json_str)
-
-        if verbose:
-            print(f"Metrics written to: {json_output}")
-
-    if output_video:
-        if verbose:
-            print(f"Generating debug video: {output_video}")
-
-        if timer:
-            with timer.measure("debug_video_generation"):
-                with CMJDebugOverlayRenderer(
-                    output_video,
-                    video_width,
-                    video_height,
-                    video_display_width,
-                    video_display_height,
-                    video_fps,
-                ) as renderer:
-                    for i, frame in enumerate(frames):
-                        annotated = renderer.render_frame(
-                            frame, smoothed_landmarks[i], i, metrics
-                        )
-                        renderer.write_frame(annotated)
-            # Capture re-encoding duration separately
-            with timer.measure("debug_video_reencode"):
-                pass  # Re-encoding happens in context manager __exit__
-        else:
-            with CMJDebugOverlayRenderer(
-                output_video,
-                video_width,
-                video_height,
-                video_display_width,
-                video_display_height,
-                video_fps,
-            ) as renderer:
-                for i, frame in enumerate(frames):
-                    annotated = renderer.render_frame(
-                        frame, smoothed_landmarks[i], i, metrics
-                    )
-                    renderer.write_frame(annotated)
-
-        if verbose:
-            print(f"Debug video saved: {output_video}")
-
-
 def process_cmj_video(
     video_path: str,
     quality: str = "balanced",
@@ -1340,6 +1162,7 @@ def process_cmj_video(
                             video.display_width,
                             video.display_height,
                             video.fps,
+                            timer=timer,  # Passing timer here too
                         ) as renderer:
                             for i, frame in enumerate(frames):
                                 annotated = renderer.render_frame(
@@ -1357,6 +1180,7 @@ def process_cmj_video(
                         video.display_width,
                         video.display_height,
                         video.fps,
+                        timer=timer,  # Passing timer here too
                     ) as renderer:
                         for i, frame in enumerate(frames):
                             annotated = renderer.render_frame(
@@ -1433,7 +1257,7 @@ def process_cmj_video(
                     dur_ms = duration * 1000
                     print(f"{stage:.<40} {dur_ms:>6.0f}ms ({percentage:>5.1f}%)")
                 total_ms = total_time * 1000
-                print(f"{'Total':.>40} {total_ms:>6.0f}ms (100.0%)")
+                print(f"{('Total'):.>40} {total_ms:>6.0f}ms (100.0%)")
                 print()
 
                 print(f"\nJump height: {metrics.jump_height:.3f}m")

{kinemotion-0.42.0 → kinemotion-0.43.0}/src/kinemotion/core/debug_overlay_utils.py

@@ -9,6 +9,8 @@ from pathlib import Path
 import cv2
 import numpy as np
 
+from .timing import PerformanceTimer
+
 
 def create_video_writer(
     output_path: str,
@@ -104,6 +106,7 @@ class BaseDebugOverlayRenderer:
         display_width: int,
         display_height: int,
         fps: float,
+        timer: PerformanceTimer | None = None,
     ):
         """
         Initialize overlay renderer.
@@ -115,16 +118,30 @@ class BaseDebugOverlayRenderer:
             display_width: Display width (considering SAR)
             display_height: Display height (considering SAR)
             fps: Frames per second
+            timer: Optional PerformanceTimer for measuring operations
         """
         self.output_path = output_path
         self.width = width
         self.height = height
-        self.display_width = display_width
-        self.display_height = display_height
+        self.timer = timer
+
+        # Optimize debug video resolution: Cap max dimension to 720p
+        # Reduces software encoding time on single-core Cloud Run instances.
+        # while keeping sufficient quality for visual debugging.
+        max_dimension = 720
+        if max(display_width, display_height) > max_dimension:
+            scale = max_dimension / max(display_width, display_height)
+            # Ensure dimensions are even for codec compatibility
+            self.display_width = int(display_width * scale) // 2 * 2
+            self.display_height = int(display_height * scale) // 2 * 2
+        else:
+            self.display_width = display_width
+            self.display_height = display_height
+
         # Duration of ffmpeg re-encoding (0.0 if not needed)
         self.reencode_duration_s = 0.0
         self.writer, self.needs_resize, self.used_codec = create_video_writer(
-            output_path, width, height, display_width, display_height, fps
+            output_path, width, height, self.display_width, self.display_height, fps
         )
 
     def write_frame(self, frame: np.ndarray) -> None:
@@ -148,13 +165,29 @@ class BaseDebugOverlayRenderer:
 
         # Resize to display dimensions if needed (to handle SAR)
         if self.needs_resize:
-            frame = cv2.resize(
-                frame,
-                (self.display_width, self.display_height),
-                interpolation=cv2.INTER_LANCZOS4,
-            )
+            if self.timer:
+                with self.timer.measure("debug_video_resize"):
+                    frame = cv2.resize(
+                        frame,
+                        (self.display_width, self.display_height),
+                        interpolation=cv2.INTER_LINEAR,
+                    )
+            else:
+                frame = cv2.resize(
+                    frame,
+                    (self.display_width, self.display_height),
+                    interpolation=cv2.INTER_LINEAR,
+                )
 
-        write_overlay_frame(self.writer, frame, self.display_width, self.display_height)
+        if self.timer:
+            with self.timer.measure("debug_video_write"):
+                write_overlay_frame(
+                    self.writer, frame, self.display_width, self.display_height
+                )
+        else:
+            write_overlay_frame(
+                self.writer, frame, self.display_width, self.display_height
+            )
 
     def close(self) -> None:
         """Release video writer and re-encode if possible."""

{kinemotion-0.42.0 → kinemotion-0.43.0}/src/kinemotion/core/video_io.py

@@ -189,15 +189,27 @@ class VideoProcessor:
             return None
 
         # Apply rotation if video has rotation metadata
-        if self.rotation == -90 or self.rotation == 270:
-            # -90 degrees = rotate 90 degrees clockwise
-            frame = cv2.rotate(frame, cv2.ROTATE_90_CLOCKWISE)
-        elif self.rotation == 90 or self.rotation == -270:
-            # 90 degrees = rotate 90 degrees counter-clockwise
-            frame = cv2.rotate(frame, cv2.ROTATE_90_COUNTERCLOCKWISE)
-        elif self.rotation == 180 or self.rotation == -180:
-            # 180 degrees rotation
-            frame = cv2.rotate(frame, cv2.ROTATE_180)
+        if self.timer:
+            with self.timer.measure("frame_rotation"):
+                if self.rotation == -90 or self.rotation == 270:
+                    # -90 degrees = rotate 90 degrees clockwise
+                    frame = cv2.rotate(frame, cv2.ROTATE_90_CLOCKWISE)
+                elif self.rotation == 90 or self.rotation == -270:
+                    # 90 degrees = rotate 90 degrees counter-clockwise
+                    frame = cv2.rotate(frame, cv2.ROTATE_90_COUNTERCLOCKWISE)
+                elif self.rotation == 180 or self.rotation == -180:
+                    # 180 degrees rotation
+                    frame = cv2.rotate(frame, cv2.ROTATE_180)
+        else:
+            if self.rotation == -90 or self.rotation == 270:
+                # -90 degrees = rotate 90 degrees clockwise
+                frame = cv2.rotate(frame, cv2.ROTATE_90_CLOCKWISE)
+            elif self.rotation == 90 or self.rotation == -270:
+                # 90 degrees = rotate 90 degrees counter-clockwise
+                frame = cv2.rotate(frame, cv2.ROTATE_90_COUNTERCLOCKWISE)
+            elif self.rotation == 180 or self.rotation == -180:
+                # 180 degrees rotation
+                frame = cv2.rotate(frame, cv2.ROTATE_180)
 
         return frame
 

{kinemotion-0.42.0 → kinemotion-0.43.0}/src/kinemotion/dropjump/debug_overlay.py

@@ -138,42 +138,53 @@ class DebugOverlayRenderer(BaseDebugOverlayRenderer):
         Returns:
             Frame with debug overlay
         """
-        annotated = frame.copy()
-
-        # Draw landmarks
-        if landmarks:
-            if use_com:
-                self._draw_com_visualization(annotated, landmarks, contact_state)
-            else:
-                self._draw_foot_visualization(annotated, landmarks, contact_state)
-
-        # Draw contact state
-        state_color = (
-            (0, 255, 0) if contact_state == ContactState.ON_GROUND else (0, 0, 255)
-        )
-        cv2.putText(
-            annotated,
-            f"State: {contact_state.value}",
-            (10, 30),
-            cv2.FONT_HERSHEY_SIMPLEX,
-            1,
-            state_color,
-            2,
-        )
-
-        # Draw frame number
-        cv2.putText(
-            annotated,
-            f"Frame: {frame_idx}",
-            (10, 70),
-            cv2.FONT_HERSHEY_SIMPLEX,
-            0.7,
-            (255, 255, 255),
-            2,
-        )
-
-        # Draw phase labels
-        if metrics:
-            self._draw_phase_labels(annotated, frame_idx, metrics)
+        if self.timer:
+            with self.timer.measure("debug_video_copy"):
+                annotated = frame.copy()
+        else:
+            annotated = frame.copy()
+
+        def _draw_overlays() -> None:
+            # Draw landmarks
+            if landmarks:
+                if use_com:
+                    self._draw_com_visualization(annotated, landmarks, contact_state)
+                else:
+                    self._draw_foot_visualization(annotated, landmarks, contact_state)
+
+            # Draw contact state
+            state_color = (
+                (0, 255, 0) if contact_state == ContactState.ON_GROUND else (0, 0, 255)
+            )
+            cv2.putText(
+                annotated,
+                f"State: {contact_state.value}",
+                (10, 30),
+                cv2.FONT_HERSHEY_SIMPLEX,
+                1,
+                state_color,
+                2,
+            )
+
+            # Draw frame number
+            cv2.putText(
+                annotated,
+                f"Frame: {frame_idx}",
+                (10, 70),
+                cv2.FONT_HERSHEY_SIMPLEX,
+                0.7,
+                (255, 255, 255),
+                2,
+            )
+
+            # Draw phase labels
+            if metrics:
+                self._draw_phase_labels(annotated, frame_idx, metrics)
+
+        if self.timer:
+            with self.timer.measure("debug_video_draw"):
+                _draw_overlays()
+        else:
+            _draw_overlays()
 
         return annotated