wx44wx/pipecat
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Keep model/language in LiveOptions at construction time so apply_update's bidirectional sync is sufficient; simplify _build_live_options to only add sample_rate

Browse Source
This commit is contained in:
Paul Kompfner
2026-02-26 09:32:52 -05:00
parent e21e8585f0
commit 3c20eda8bf
2 changed files with 6 additions and 16 deletions
Download Patch File Download Diff File Expand all files Collapse all files

11
src/pipecat/services/deepgram/stt.py
View File

@@ -262,10 +262,9 @@ class DeepgramSTTService(STTService):
if "language" in merged_dict and isinstance(merged_dict["language"], Language):
merged_dict["language"] = merged_dict["language"].value
# Extract model/language for top-level STTSettings fields; everything
# else lives inside LiveOptions.
model = merged_dict.pop("model", None)
language = merged_dict.pop("language", None)
# Sync model/language to top-level STTSettings fields
model = merged_dict.get("model")
language = merged_dict.get("language")
settings = DeepgramSTTSettings(
model=model, language=language, live_options=LiveOptions(**merged_dict)
@@ -380,10 +379,6 @@ class DeepgramSTTService(STTService):
A fully-populated ``LiveOptions`` ready for the Deepgram SDK.
"""
opts: dict[str, Any] = self._settings.live_options.to_dict()
# Overlay model/language from top-level settings and sample_rate from service.
opts["model"] = self._settings.model
opts["language"] = self._settings.language
opts["sample_rate"] = self.sample_rate
return LiveOptions(**opts)

11
src/pipecat/services/deepgram/stt_sagemaker.py
View File

@@ -247,10 +247,9 @@ class DeepgramSageMakerSTTService(STTService):
if "language" in merged_dict and isinstance(merged_dict["language"], Language):
merged_dict["language"] = merged_dict["language"].value
# Extract model/language for top-level STTSettings fields; everything
# else lives inside LiveOptions.
model = merged_dict.pop("model", None)
language = merged_dict.pop("language", None)
# Sync model/language to top-level STTSettings fields
model = merged_dict.get("model")
language = merged_dict.get("language")
settings = DeepgramSageMakerSTTSettings(
model=model, language=language, live_options=LiveOptions(**merged_dict)
@@ -344,10 +343,6 @@ class DeepgramSageMakerSTTService(STTService):
A fully-populated ``LiveOptions`` ready for the Deepgram SDK.
"""
opts: dict[str, Any] = self._settings.live_options.to_dict()
# Overlay model/language from top-level settings and sample_rate from service.
opts["model"] = self._settings.model
opts["language"] = self._settings.language
opts["sample_rate"] = self.sample_rate
return LiveOptions(**opts)