Skip to content

perf(voice): make _TTS_PIPELINE_DEPTH env-configurable; default 2 (#6811) - #6812

Merged
mrveiss merged 1 commit into
Dev_new_guifrom
issue-6811
May 4, 2026
Merged

mrveiss merged 1 commit into
Dev_new_guifrom
issue-6811

Conversation

@mrveiss

@mrveiss mrveiss commented May 4, 2026

Copy link
Copy Markdown
Owner

Summary

_TTS_PIPELINE_DEPTH was hardcoded to 3 (#6752). Pocket TTS calls run via loop.run_in_executor(None, _run_synthesis, ...) and serialize on the GIL/model lock — depth>2 only adds queue contention without true parallelism.

This PR:

Verification

$ python3 -c '... resolver tests ...'
Invalid AUTOBOT_TTS_PIPELINE_DEPTH='banana'; using 2
All resolver cases pass

Edge cases tested:

  • default → 2
  • '1' → 1 (min)
  • '8' → 8 (max)
  • '99' → 8 (upper clamp)
  • '0' → 1 (lower clamp)
  • '-5' → 1
  • 'banana' → 2 + warning

python -m py_compile passes.

Test plan

Closes

Closes #6811

Related: #6810 (single-WS unification), #6752 (original jitter-absorption introducing depth=3)

)

Pocket TTS calls run via run_in_executor and serialize on the GIL/model
lock — depth>2 only adds queue contention without true parallelism.
Lowering default from 3 to 2 keeps one chunk synthesizing ahead while the
client plays the previous chunk (the actual jitter-absorption goal of
#6752) without piling up backlog.

AUTOBOT_TTS_PIPELINE_DEPTH overrides the default, clamped to 1..8 to
prevent ops typos OOMing the worker. Invalid values log a warning and
fall back to 2.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@github-actions

github-actions Bot commented May 4, 2026

Copy link
Copy Markdown
Contributor

⚠️ SSOT Configuration Compliance: Violations Found

Metric Count
Total Violations 1
SSOT Violations (high priority) 1
Other Violations 0

⚠️ 1 values have SSOT config equivalents!

These should be replaced with SSOT config imports:

Python:

from src.config.ssot_config import config
# Use: config.vm.main, config.port.backend, config.backend_url

TypeScript:

import config from '@/config/ssot-config'
// Use: config.vm.main, config.port.backend, config.backendUrl

📖 See SSOT_CONFIG_GUIDE.md for documentation.

@mrveiss
mrveiss merged commit b508487 into Dev_new_gui May 4, 2026
2 of 5 checks passed
@mrveiss
mrveiss deleted the issue-6811 branch May 4, 2026 09:51
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.

1 participant