Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

DIOS-6236: Show Target Bitrate in Stats #202

Merged
merged 12 commits into from
Jul 25, 2024

Conversation

sheiladoherty-dolby
Copy link
Contributor

  • Added Target Bitrate to stats
  • Refactored SingleStreamView, SingleStreamViewModel, StatisticsInfoView & StatsInfoViewModel so that the StatisticsInfoView is no longer being reinitialized on every stat change by moving listeners to StatsInfoViewModel
  • Added published value to VideoTracksManager to publish the targetBitrate values for all the sourceIDs.

Simulator Screenshot - iPhone 15 Pro - 2024-07-23 at 15 28 00

Copy link
Contributor

@aravind-raveendran aravind-raveendran left a comment

Choose a reason for hiding this comment

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

LGTM, a few minor questions

@aravind-raveendran
Copy link
Contributor

May be we should follow squash commits on PRs to keep the history much cleaner !!

bitrate for source

cleanup

cleanup

cleanup

cleanup

Remove observable object
@sheiladoherty-dolby sheiladoherty-dolby merged commit 0389bcd into main Jul 25, 2024
2 checks passed
@sheiladoherty-dolby sheiladoherty-dolby deleted the feature/targetBitrate branch August 8, 2024 22:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants