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

[telemetry] Add telemetry to SbPlayerWriteSamples #4097

Merged
merged 1 commit into from
Sep 30, 2024

Conversation

borongc
Copy link
Contributor

@borongc borongc commented Sep 9, 2024

Adding additional Media Telemetry to cover SbPlayerWriteSamples.

The rough range of each function:

  1. When batched write is enabled (Media.MaxAudioSamplesPerWrite=256, [media] Limit max number of buffers to write with audio write ahead #3271):
    1. SbPlayerWriteSamples()/audio: 1us to 98us
    2. SbPlayerWriteSamples()/video: 1us to 3.4ms
  2. When batched write is disabled (Media.MaxAudioSamplesPerWrite=1, [media] Limit max number of buffers to write with audio write ahead #3271):
    1. SbPlayerWriteSamples()/audio: 9us to 65.47ms
    2. SbPlayerWriteSamples()/video: 8us to 28.07ms

b/303328238

@borongc borongc force-pushed the telemetry_sbplayer_write_samples branch 2 times, most recently from 75d2d7c to 9d21512 Compare September 10, 2024 15:18
cobalt/media/base/metrics_provider.cc Outdated Show resolved Hide resolved
tools/metrics/histograms/metadata/cobalt/histograms.xml Outdated Show resolved Hide resolved
@borongc borongc force-pushed the telemetry_sbplayer_write_samples branch from 9d21512 to eaecf86 Compare September 10, 2024 16:02
@borongc borongc force-pushed the telemetry_sbplayer_write_samples branch 3 times, most recently from 7f9fbc4 to 5293281 Compare September 28, 2024 14:39
Adding additional Media Telemetry to cover SbPlayerWriteSamples.

The rough range of each function:
1. When batched write is enabled (Media.MaxAudioSamplesPerWrite=256):
  1. SbPlayerWriteSamples()/audio: 1us to 98us
  2. SbPlayerWriteSamples()/video: 1us to 3.4ms
2. When batched write is disabled (Media.MaxAudioSamplesPerWrite=1):
  1. SbPlayerWriteSamples()/audio: 9us to 65.47ms
  2. SbPlayerWriteSamples()/video: 8us to 28.07ms

b/303328238
@borongc borongc force-pushed the telemetry_sbplayer_write_samples branch from 5293281 to 09ae263 Compare September 30, 2024 19:10
@borongc borongc merged commit d79a729 into youtube:25.lts.1+ Sep 30, 2024
300 of 301 checks passed
@borongc borongc deleted the telemetry_sbplayer_write_samples branch September 30, 2024 20:16
borongc added a commit that referenced this pull request Oct 15, 2024
borongc added a commit that referenced this pull request Oct 15, 2024
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