Conversation
- Consolidated dependencies into a single setup-requirements.txt for Friend-Lite setup scripts. - Updated command paths in wizard.py to reference the new location of setup-requirements.txt. - Added support for PYTORCH_CUDA_VERSION in asr-services to enhance compatibility. - Introduced init.py for interactive configuration of the ASR service. - Removed outdated setup-requirements.txt files from advanced and speaker-recognition services. - Updated docker-compose.yml files to use the new PYTORCH_CUDA_VERSION variable.
…asr-services-setup Update ASR services setup with new dependencies and configuration
…se.yml for OpenMemory MCP - Added conditional start for Deepgram worker based on DEEPGRAM_API_KEY presence. - Updated shutdown and monitoring functions to handle AUDIO_STREAM_WORKER_PID safely. - Changed OpenMemory MCP service to use the official Docker image instead of a local build. - Removed deprecated init-cache.sh script and streamlined setup.sh for clarity.
- Introduced conditional starting of Parakeet worker based on PARAKEET_ASR_URL or OFFLINE_ASR_TCP_URI presence. - Updated variable names for clarity, changing AUDIO_STREAM_WORKER_PID to AUDIO_STREAM_DEEPGRAM_WORKER_PID and added AUDIO_STREAM_PARAKEET_WORKER_PID. - Improved shutdown and monitoring functions to handle new worker PIDs safely.
…ckend for all proxy configurations - Changed reverse proxy targets for WebSocket, API, Auth, Health, Readiness, Users, and Audio endpoints from friend-backend:8000 to chronicle-backend:8000. - Ensured consistency across all relevant sections of the Caddyfile.template.
- Changed the default network name in backends/advanced/docker-compose.yml from friend-network to chronicle-network. - Updated extras/openmemory-mcp/docker-compose.yml to include chronicle-network as an external network and added it to the services configuration.
…memory-cache Fix deepgram/parakeet workers bug
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the ✨ Finishing touches🧪 Generate unit tests (beta)
Comment |
|
This has an anti-pattern. Most of the codebase has the pattern of "first check CI env vars, then .env.test and fallback to .env". |
|
Feel free to update it and fix anti pattern, this fixes the broken tests issue, and I am changing the way env vars work with a later PR, so this will change later anyway |
|
Can I close this? Looks like it comments a big fun and also comments it's call() |
|
if the tests pass in deepgram without it then close. otherwise merge. |
2 participants
No description provided.