FrameLab v1.1.0: Basler/backend support, audit (complexity + bug fixes + tests), dependency pinning

.github/workflows/tests.yml

@@ -0,0 +1,59 @@
+name: CI
+
+on:
+  push:
+    branches: [ main, dev-v1.1.0 ]
+  pull_request:
+    branches: [ main ]
+
+jobs:
+  test:
+    runs-on: ubuntu-latest
+
+    steps:
+    - uses: actions/checkout@v4
+
+    # MediaPipe (PoseLandmarker) links libGLESv2.so.2 even on the CPU delegate,
+    # and OpenCV needs libGL — neither ships on the bare runner.
+    - name: Install system libraries (MediaPipe/OpenCV need GL)
+      run: |
+        sudo apt-get update
+        sudo apt-get install -y libgl1 libegl1 libgles2
+
+    - name: Set up Python
+      uses: actions/setup-python@v5
+      with:
+        python-version: '3.10'
+        cache: pip
+        cache-dependency-path: requirements.txt
+
+    - name: Install dependencies
+      run: |
+        python -m pip install --upgrade pip
+        pip install -r requirements.txt
+
+    - name: Run tests
+      run: |
+        python tests/run_tests.py
+
+  lint:
+    runs-on: ubuntu-latest
+
+    steps:
+    - uses: actions/checkout@v4
+
+    - name: Set up Python
+      uses: actions/setup-python@v5
+      with:
+        python-version: '3.10'
+        cache: pip
+        cache-dependency-path: requirements-dev.txt
+
+    - name: Install dependencies
+      run: |
+        python -m pip install --upgrade pip
+        pip install -r requirements-dev.txt
+
+    - name: Run pylint
+      run: |
+        pylint $(git ls-files '*.py')

.gitignore

@@ -136,3 +136,7 @@ dmypy.json
 # Cython debug symbols
 cython_debug/
 
+# Local runtime data
+settings.json
+db_athlete/
+

.markdownlint.json

@@ -0,0 +1,7 @@
+{
+  "MD013": false,
+  "MD033": false,
+  "MD041": false,
+  "MD024": { "siblings_only": true },
+  "MD029": false
+}

.pre-commit-config.yaml

@@ -0,0 +1,20 @@
+# Pre-commit hooks for FrameLab.
+#
+# One-time setup (inside your venv):
+#     pip install -r requirements-dev.txt
+#     pre-commit install
+#
+# Run manually against everything:
+#     pre-commit run --all-files
+#
+# The pylint hook uses the project's .pylintrc and the pylint installed in
+# your environment (language: system), so it matches the CI lint job.
+repos:
+  - repo: local
+    hooks:
+      - id: pylint
+        name: pylint
+        entry: pylint
+        language: system
+        types: [python]
+        require_serial: true

.pylintrc

@@ -0,0 +1,59 @@
+[MAIN]
+# Use multiple processes to speed up pylint
+jobs=1
+
+# Allow loading of all C extensions (cv2, mediapipe, dearpygui)
+unsafe-load-any-extension=yes
+
+[MESSAGES CONTROL]
+disable=
+    C0114, # missing-module-docstring
+    C0115, # missing-class-docstring
+    C0116, # missing-function-docstring
+    C0301, # line-too-long (DearPyGUI calls are naturally long)
+    R0902, # too-many-instance-attributes (Camera class legitimately large)
+    R0903, # too-few-public-methods
+    R0913, # too-many-arguments
+    R0914, # too-many-locals
+    R0912, # too-many-branches
+    R0915, # too-many-statements
+    R0801, # duplicate-code (camera init duplicated between main.py and layout.py)
+    E1101, # no-member (false positives with cv2, dpg, mediapipe C extensions)
+    W0718, # broad-exception-caught (used intentionally in cleanup paths)
+    W0719, # broad-exception-raised
+    W1203, # logging-fstring-interpolation (project uses f-strings consistently)
+    W0511, # fixme/todo
+    C0415, # import-outside-toplevel (deferred imports used to avoid circular deps)
+    W0603, # global-statement (singleton/cache pattern used consistently)
+    W0613, # unused-argument (DearPyGUI callbacks use a fixed (sender, app_data, user_data) signature)
+    R0917, # too-many-positional-arguments (same rationale as R0913 above)
+    R0911, # too-many-return-statements (dispatch-style functions)
+    R1702, # too-many-nested-blocks (UI/loop code; same family as R0912/R0915 above)
+    R0904, # too-many-public-methods (Settings is a large config facade; see R0902)
+    R0401, # cyclic-import (gui deferred-import cycle; broken at runtime via function-local imports)
+
+[FORMAT]
+max-line-length=120
+indent-string='    '
+
+[BASIC]
+# Allow short variable names common in image processing,
+# plus cv2.VideoCapture-compatibility method aliases (intentional camelCase).
+good-names=i,j,k,x,y,w,h,dx,dy,f,e,ex,_,dt,p1,p2,p3,v1,v2,db,isOpened,getBackendName
+
+# gui/state.py uses module-level names as intentionally-mutable shared state,
+# not constants, so don't enforce UPPER_CASE on module-level assignments.
+const-naming-style=any
+
+[DESIGN]
+max-args=10
+max-locals=25
+
+[IMPORTS]
+# Known third-party modules
+known-third-party=cv2,numpy,dearpygui,mediapipe,certifi
+
+[TYPECHECK]
+# Ignore imports that can't be resolved (C extensions)
+ignored-modules=cv2,dearpygui.dearpygui,mediapipe
+ignored-classes=cv2.VideoCapture,cv2.VideoWriter