fork main to develop#169
Closed
thestumonkey wants to merge 215 commits into
Closed
Conversation
Added version selection from the conversation page, delete version and the queuemanagement page # Conflicts: # backends/advanced/src/advanced_omi_backend/main.py
commit 3ba5506 Merge: dbf37c7 8ffd97a Author: Stuart Alexander <stu@theawesome.co.uk> Date: Wed Sep 24 20:03:29 2025 +0100 Merge branch 'AnkushMalaker:main' into pytest commit 8ffd97a Author: Ankush Malaker <43288948+AnkushMalaker@users.noreply.github.com> Date: Wed Sep 24 09:25:11 2025 +0530 fix faiss issue in speaker recognition cpu commit dbf37c7 Author: stu alexander <thestumonkey@gmail.com> Date: Sat Sep 20 19:02:22 2025 +0100 Fixed update user commit 14845a6 Author: stu alexander <thestumonkey@gmail.com> Date: Sat Sep 20 19:02:03 2025 +0100 Added robot test suite
refactored to use a service model for transcription, and got it working with rq
Declutterd main, moved all processors to RQ. Streaming and batch mode enabled on audio recording
streaming segments should now be able to identify the speaker. Removed a bunch of dead code
Tidied up dead code, added timeline to job page so we can see time taken, got speaker recognition working on conversation trigger
- Added `start.sh` and `start-workers.sh` to `.dockerignore`. - Removed unused `COMPUTE_MODE` arguments from `docker-compose.yml` and `Dockerfile`. - Updated `pyproject.toml` to require `pandas>=2.0.0` and added new dependencies: `faiss-cpu`, `torch`, and `torchaudio`. - Modified `download-pyannote.py` to use `token` instead of `use_auth_token` for model downloads. - Refactored imports and logging in `download-pyannote.py` and `websocket_wrapper.py` for better clarity.
…anges Update Docker configuration and speaker recognition dependencies
…eet. Update Docker configurations to accept CUDA version as a build argument in speaker recognition and ASR services. Enhance dependency management in pyproject.toml for speaker recognition and ASR services with optional CUDA versions.
…kwell feat: Remove unused Dockerfiles and pyproject files for blackwell gpus
Updates robot-tests.yml to post test results as PR comments while maintaining GitHub Pages deployment: - Parse test results from output.xml - Post formatted comment with pass/fail summary table - Include direct links to GitHub Pages reports (report.html, log.html) - Include links to downloadable artifacts - Keep existing GitHub Pages deployment for live reports 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
This commit simplifies the robot-tests GitHub workflow by following the same pattern as integration-tests.yml - delegating all logic to a local test runner script. Changes: - Add tests/run-robot-tests.sh (mirrors backends/advanced/run-test.sh) - Handles environment setup and service orchestration - Delegates test execution to Makefile - Supports CLEANUP_CONTAINERS and OUTPUTDIR configuration - Provides colored output and comprehensive error handling - Simplify .github/workflows/robot-tests.yml - Reduce from 413 to ~150 lines - Remove inline Docker setup, service orchestration - Just calls ./run-robot-tests.sh with API keys - Update tests/README.md - Add Quick Start section documenting new script - Explain configuration options - Show local development workflow Benefits: - Consistency with integration-tests workflow pattern - Single source of truth for test logic (script, not YAML) - Same script works locally and in CI - Easier debugging with CLEANUP_CONTAINERS=false - Simpler maintenance 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
…ion creation - Increase from 200 to 300 chunks to ensure enough speech detected in CI - Helps ensure open_conversation job is reliably created before checking
The refactored version had several issues: - Missing Docker build cache management (slower builds) - Less visibility into worker validation - Reduced debugging output - Container cleanup issues Restoring the original workflow that had all tests passing.
- Escape template literal syntax to prevent YAML parsing errors - Indent template literal content properly - Add .secrets to .gitignore - Add ACT_GUIDE.md with instructions for local GitHub Actions testing The YAML parser was interpreting ** at start of lines as YAML anchors. Escaping the backticks and $ symbols fixes this.
This helps create test fixtures so we can omit fixture folder from test clears
# Conflicts: # .github/workflows/robot-tests.yml
- env: must come BEFORE run: in GitHub Actions YAML - Previous commit had them in wrong order causing secrets to be empty - Removed unnecessary verification and export steps
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
5 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.