Skip to content

fix(gradium): report the configured model name from TTS.model and STT.model - #7814

Merged
longcw merged 1 commit into
livekit:mainfrom
hhugo-gradium:fix/gradium-model-name
Oct 21, 2026
Merged

longcw merged 1 commit into
livekit:mainfrom
hhugo-gradium:fix/gradium-model-name

Conversation

@hhugo-gradium

Copy link
Copy Markdown
Contributor

Summary

TTS.model and STT.model returned the literal "unknown", so TTS and STT metrics and traces carried no model name even though model_name is always known at construction. Both now return the configured model_name.

Testing

  • New tests/test_plugin_gradium_model_name.py (unit category).
  • ruff, ruff format --check and mypy --strict clean on the plugin.

Split out of #7672.

🤖 Generated with Claude Code

@CLAassistant

CLAassistant commented Oct 14, 2026 •

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

@hhugo-gradium
hhugo-gradium force-pushed the fix/gradium-model-name branch from fc908b7 to dc0cdd5 Compare October 14, 2026 11:34
@hhugo-gradium
hhugo-gradium marked this pull request as ready for review October 14, 2026 11:36
@hhugo-gradium
hhugo-gradium requested a review from a team as a code owner October 14, 2026 11:36

@devin-ai-integration devin-ai-integration Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no bugs or issues to report.

Devin Review

@hhugo-gradium

Copy link
Copy Markdown
Contributor Author

cc @longcw

@hhugo-gradium

Copy link
Copy Markdown
Contributor Author

@longcw, I think the ci failure is unrelated

….model

Both properties returned the literal "unknown", so TTS and STT metrics
and traces carried no model name even though `model_name` is always
known.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
@hhugo-gradium
hhugo-gradium force-pushed the fix/gradium-model-name branch from dc0cdd5 to a11754f Compare October 20, 2026 16:14
@longcw
longcw merged commit 56bc9e5 into livekit:main Oct 21, 2026
15 checks passed
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.

3 participants