fix: Use audio_in_sample_rate to deserialize data for TelnyxFrameSerializer
This commit is contained in:
@@ -9,9 +9,10 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
|
|||||||
|
|
||||||
### Fixed
|
### Fixed
|
||||||
|
|
||||||
- Fixed an issue with the `TwilioFrameSerializer` where
|
- Fixed an issue with the `TwilioFrameSerializer` and `TelnyxFrameSerializer`
|
||||||
`frame.audio_out_sample_rate` was incorrectly used in place of
|
where `frame.audio_out_sample_rate` was incorrectly used in place of
|
||||||
`frame.audio_in_sample_rate`.
|
`frame.audio_in_sample_rate`, which caused audio input detection to fail when
|
||||||
|
using different input and output sample rates.
|
||||||
|
|
||||||
## [0.0.55] - 2025-02-05
|
## [0.0.55] - 2025-02-05
|
||||||
|
|
||||||
|
|||||||
@@ -31,8 +31,8 @@ from pipecat.serializers.base_serializer import FrameSerializer, FrameSerializer
|
|||||||
|
|
||||||
class TelnyxFrameSerializer(FrameSerializer):
|
class TelnyxFrameSerializer(FrameSerializer):
|
||||||
class InputParams(BaseModel):
|
class InputParams(BaseModel):
|
||||||
telnyx_sample_rate: Optional[int] = None
|
telnyx_sample_rate: Optional[int] = None # Default Telnyx rate (8kHz)
|
||||||
sample_rate: Optional[int] = None
|
sample_rate: Optional[int] = None # Pipeline input rate
|
||||||
inbound_encoding: str = "PCMU"
|
inbound_encoding: str = "PCMU"
|
||||||
outbound_encoding: str = "PCMU"
|
outbound_encoding: str = "PCMU"
|
||||||
|
|
||||||
@@ -48,6 +48,9 @@ class TelnyxFrameSerializer(FrameSerializer):
|
|||||||
params.inbound_encoding = inbound_encoding
|
params.inbound_encoding = inbound_encoding
|
||||||
self._params = params
|
self._params = params
|
||||||
|
|
||||||
|
self._telnyx_sample_rate = 0 # Fixed rate for Telnyx (8kHz)
|
||||||
|
self._sample_rate = 0 # Pipeline input rate
|
||||||
|
|
||||||
self._resampler = create_default_resampler()
|
self._resampler = create_default_resampler()
|
||||||
|
|
||||||
@property
|
@property
|
||||||
@@ -55,13 +58,15 @@ class TelnyxFrameSerializer(FrameSerializer):
|
|||||||
return FrameSerializerType.TEXT
|
return FrameSerializerType.TEXT
|
||||||
|
|
||||||
async def setup(self, frame: StartFrame):
|
async def setup(self, frame: StartFrame):
|
||||||
|
# Configure rates for input path: Telnyx (8kHz encoded) -> Pipeline (PCM)
|
||||||
self._telnyx_sample_rate = self._params.telnyx_sample_rate or frame.audio_in_sample_rate
|
self._telnyx_sample_rate = self._params.telnyx_sample_rate or frame.audio_in_sample_rate
|
||||||
self._sample_rate = self._params.sample_rate or frame.audio_out_sample_rate
|
self._sample_rate = self._params.sample_rate or frame.audio_in_sample_rate
|
||||||
|
|
||||||
async def serialize(self, frame: Frame) -> str | bytes | None:
|
async def serialize(self, frame: Frame) -> str | bytes | None:
|
||||||
if isinstance(frame, AudioRawFrame):
|
if isinstance(frame, AudioRawFrame):
|
||||||
data = frame.audio
|
data = frame.audio
|
||||||
|
|
||||||
|
# Output: Convert PCM at frame's rate to 8kHz encoded for Telnyx
|
||||||
if self._params.inbound_encoding == "PCMU":
|
if self._params.inbound_encoding == "PCMU":
|
||||||
serialized_data = await pcm_to_ulaw(
|
serialized_data = await pcm_to_ulaw(
|
||||||
data, frame.sample_rate, self._telnyx_sample_rate, self._resampler
|
data, frame.sample_rate, self._telnyx_sample_rate, self._resampler
|
||||||
@@ -92,6 +97,7 @@ class TelnyxFrameSerializer(FrameSerializer):
|
|||||||
payload_base64 = message["media"]["payload"]
|
payload_base64 = message["media"]["payload"]
|
||||||
payload = base64.b64decode(payload_base64)
|
payload = base64.b64decode(payload_base64)
|
||||||
|
|
||||||
|
# Input: Convert Telnyx's 8kHz encoded audio to PCM at pipeline input rate
|
||||||
if self._params.outbound_encoding == "PCMU":
|
if self._params.outbound_encoding == "PCMU":
|
||||||
deserialized_data = await ulaw_to_pcm(
|
deserialized_data = await ulaw_to_pcm(
|
||||||
payload,
|
payload,
|
||||||
|
|||||||
Reference in New Issue
Block a user