Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit d8930fbf authored by Matías Hernández's avatar Matías Hernández Committed by Automerger Merge Worker
Browse files

Merge "Tweak RateEstimator's alpha value" into udc-qpr-dev am: 0e5f622c am: 7f63c535 

Original change: https://googleplex-android-review.googlesource.com/c/platform/frameworks/base/+/23583156



Change-Id: Ib5c67c677ab59fa50e084c8c9d93b411aab07f69
Signed-off-by: default avatarAutomerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
parents 6560ff1e 7f63c535

services/core/java/com/android/server/notification/RateEstimator.java

+1 −1
Original line number Diff line number Diff line
@@ -23,7 +23,7 @@ package com.android.server.notification; 
 * {@hide}

 */

class RateEstimator {

    private static final double RATE_ALPHA = 0.5;

    private static final double RATE_ALPHA = 0.7;

    private static final double MINIMUM_DT = 0.0005;



    private Long mLastEventTime;

services/tests/uiservicestests/src/com/android/server/notification/RateEstimatorTest.java

+7 −7
Original line number Diff line number Diff line
@@ -138,22 +138,22 @@ public class RateEstimatorTest extends UiServiceTestCase { 


    @Test

    public void testEstimateCatchesUpQuickly() {

        long nextEventTime = postEvents(mTestStartTime, 10, 20); // 20 events at 100Hz

        long nextEventTime = postEvents(mTestStartTime, 100, 30); // 30 events at 10Hz



        final float firstBurstRate = mEstimator.getRate(nextEventTime);

        assertThat(firstBurstRate).isWithin(10f).of(100);

        assertThat(firstBurstRate).isWithin(2f).of(10);



        nextEventTime += HOURS.toMillis(3); // 3 hours later...

        nextEventTime = postEvents(nextEventTime, 10, 20); // same burst of 20 events at 100Hz

        nextEventTime = postEvents(nextEventTime, 100, 30); // same burst of 30 events at 10Hz



        // Catching up. Rate is not yet 100, since we had a long period of inactivity...

        // Catching up. Rate is not yet 10, since we had a long period of inactivity...

        float secondBurstRate = mEstimator.getRate(nextEventTime);

        assertThat(secondBurstRate).isWithin(10f).of(60);

        assertThat(secondBurstRate).isWithin(1f).of(6);



        // ... but after a few more events, we are there.

        nextEventTime = postEvents(nextEventTime, 10, 10); // 10 more events at 100Hz

        nextEventTime = postEvents(nextEventTime, 100, 10); // 10 more events at 10Hz

        secondBurstRate = mEstimator.getRate(nextEventTime);

        assertThat(secondBurstRate).isWithin(10f).of(100);

        assertThat(secondBurstRate).isWithin(1f).of(10);

    }



    /** @return the next event time. */