New issue
Advanced search Search tips

Issue 602242 link

Starred by 1 user

Issue metadata

Status: WontFix
Owner:
Closed: Apr 2016
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 2
Type: Bug-Regression



Sign in to add a comment

A zero-to-nonzero to 50% regression in webrtc_perf_tests at 12258:12259

Project Member Reported by minyue@chromium.org, Apr 11 2016

Issue description

Comment 2 by peah@chromium.org, Apr 15 2016

This change is so small so it is expected that the changes in the code being done could have caused it. Therefore I'd say it is fine and expected.

Comment 3 by peah@chromium.org, Apr 15 2016

Status: WontFix (was: Assigned)
Closing this bug.

Comment 4 by minyue@webrtc.org, Apr 15 2016

I cannot agree that the change is so small, it is 50%. But you know better.

Comment 5 by peah@chromium.org, Apr 15 2016

Sorry, I probably should have been more verbose.
What I meant was that the absolute value of the changes was so small. You are definitely right that in percent it is high.

But you should take into account that the failing tests are the worst case call duration of the render side in the audio processing module when no processing at all is done. 

The code in the CL that caused this degradation reintroduces the resampling of a 48 kHz render signal to 16 kHz that was present before March 17 so the call duration increase is not an increase if you look longer back in time.
And regardless of that, the increase is very small compared to the complexity in the module when actual processing is done.

Sign in to add a comment