Skip to content

refactor: Update Python version support and reorganize requirements files...#63

Merged
duartebarbosadev merged 2 commits intomainfrom
refactor-requirements-filess-ci-workflow
Nov 16, 2025
Merged

refactor: Update Python version support and reorganize requirements files...#63
duartebarbosadev merged 2 commits intomainfrom
refactor-requirements-filess-ci-workflow

Conversation

@duartebarbosadev
Copy link
Copy Markdown
Owner

...;Add smoke test functionality and update CI workflow for startup regression checks

This pull request introduces a new smoke test capability for CI and updates dependency handling and documentation to reflect current support and packaging requirements. The main changes are the addition of a headless smoke test to catch startup regressions, removal of unused dependencies (notably mediapipe and pyiqa), and an update to documentation clarifying Python version support.

CI and Testing Improvements

  • Added a headless smoke test step to the CI workflow (.github/workflows/ci.yml), which briefly launches the app in offscreen mode to catch startup regressions. This is triggered via the new --smoke-test and --smoke-delay-ms command-line arguments in src/main.py and is controlled by a new environment variable. [1] [2] [3]

Dependency and Packaging Updates

  • Removed mediapipe and pyiqa from the requirements and packaging scripts (requirements-cuda.txt, .github/workflows/release-build.yml) to reflect that these are no longer needed for builds or runtime. [1] [2] [3] [4] [5]

Documentation Updates

  • Updated the README to clarify that Python 3.12 is the supported and tested version, but newer interpreters may work. This removes the implication that mediapipe is required for interpreter choice.

…iles; Add smoke test functionality and update CI workflow for startup regression checks
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This pull request refactors dependency management, adds smoke test functionality for CI startup regression checks, and updates documentation to reflect current Python version support. The changes remove unused dependencies (mediapipe, pyiqa) that are no longer required by the application.

  • Added headless smoke test capability with --smoke-test and --smoke-delay-ms command-line arguments to catch startup regressions in CI
  • Reorganized requirements files (alphabetically) and removed unused dependencies (mediapipe, pyiqa, and transitive dependencies transformers, safetensors)
  • Updated documentation to clarify Python 3.12 as the tested version without referencing removed dependencies

Reviewed Changes

Copilot reviewed 5 out of 6 changed files in this pull request and generated no comments.

Show a summary per file
File Description
src/main.py Added smoke test command-line arguments and auto-exit timer for CI validation
requirements.txt Reorganized alphabetically, removed pyiqa and mediapipe, added openai as explicit dependency
requirements-cuda.txt Same reorganization as requirements.txt with onnxruntime-gpu instead of onnxruntime
README.md Updated Python version note to remove MediaPipe reference and clarify 3.12 support
.github/workflows/release-build.yml Removed PyInstaller hidden imports and data collection for removed packages
.github/workflows/ci.yml Added smoke test step to catch startup regressions in headless mode

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@duartebarbosadev duartebarbosadev merged commit f02d216 into main Nov 16, 2025
3 checks passed
@duartebarbosadev duartebarbosadev deleted the refactor-requirements-filess-ci-workflow branch November 16, 2025 04:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants