Skip to content

Conversation

@aconchillo
Copy link
Contributor

@aconchillo aconchillo commented Sep 25, 2025

Please describe the changes in your PR. If it is addressing an issue, please reference that as well.

This PR deprecates DailyUpdateRemoteParticipantsFrame. Developers can create their own custom frame and add a custom processor after the base output transport, or use on of the new events like, for example, on_after_push_frame in the output transport.

The main reason is to avoid having to support one frame per function and the corresponding handling of that frame.

@codecov
Copy link

codecov bot commented Sep 25, 2025

Codecov Report

❌ Patch coverage is 0% with 6 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
src/pipecat/transports/daily/transport.py 0.00% 6 Missing ⚠️
Files with missing lines Coverage Δ
src/pipecat/transports/daily/transport.py 0.00% <0.00%> (ø)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@aconchillo aconchillo force-pushed the aleix/deprecate-daily-update-remote-participants-frame branch from 4899649 to 85ea1a3 Compare September 25, 2025 20:45
CHANGELOG.md Outdated
Comment on lines 17 to 19
future version. Create your own custom frame and use a custom processor to
handle it or use, for example, `on_after_push_frame` event instead in the
output transport.
Copy link
Member

Choose a reason for hiding this comment

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

wording nit

Suggested change
future version. Create your own custom frame and use a custom processor to
handle it or use, for example, `on_after_push_frame` event instead in the
output transport.
future version. Create your own custom frame and handle it in the
`@transport.output().event_handler("on_after_push_frame")` event handler
or a custom processor.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thank you. Done!

warnings.simplefilter("always")
warnings.warn(
"DailyUpdateRemoteParticipantsFrame is deprecated and will be removed in a future version."
"Create your own custom frame and use a custom processor to handle it or use, for example, "
Copy link
Member

Choose a reason for hiding this comment

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

same wording as above nit

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Updated.

@aconchillo aconchillo force-pushed the aleix/deprecate-daily-update-remote-participants-frame branch from 85ea1a3 to cf2c9d3 Compare September 30, 2025 23:25
@aconchillo aconchillo force-pushed the aleix/deprecate-daily-update-remote-participants-frame branch from cf2c9d3 to 92d3be8 Compare September 30, 2025 23:26
@aconchillo aconchillo merged commit feae3b6 into main Sep 30, 2025
4 of 5 checks passed
@aconchillo aconchillo deleted the aleix/deprecate-daily-update-remote-participants-frame branch September 30, 2025 23:27
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.

4 participants