[MotionMark] Increase the warm-up time for each sub-test from 100ms to 1000ms

shallawa · shallawa · commit d35730664068 · 2020-12-17T23:43:57.000Z
https://bugs.webkit.org/show_bug.cgi?id=219984 Reviewed by Jon Lee. Ensure the warm-up time is enough for the system to finish any initialization work which may be triggered by the sub-test before starting the sampling. This should prevent bi-modality in the score of the sub-tests since the initial frame rate is crucial in deciding how much the complexity can grow. * MotionMark/developer.html: * MotionMark/resources/runner/motionmark.js: (this.clear): Canonical link: https://commits.webkit.org/232581@main git-svn-id: https://svn.webkit.org/repository/webkit/trunk@270959 268f45cc-cd09-0410-ab3c-d52691b4dbfc
diff --git a/PerformanceTests/ChangeLog b/PerformanceTests/ChangeLog
@@ -1,3 +1,19 @@
+2020-12-17  Said Abou-Hallawa  <said@apple.com>
+
+        [MotionMark] Increase the warm-up time for each sub-test from 100ms to 1000ms
+        https://bugs.webkit.org/show_bug.cgi?id=219984
+
+        Reviewed by Jon Lee.
+
+        Ensure the warm-up time is enough for the system to finish any initialization
+        work which may be triggered by the sub-test before starting the sampling.
+        This should prevent bi-modality in the score of the sub-tests since the
+        initial frame rate is crucial in deciding how much the complexity can grow.
+
+        * MotionMark/developer.html:
+        * MotionMark/resources/runner/motionmark.js:
+        (this.clear):
+
 2020-12-14  Antti Koivisto  <antti@apple.com>
 
         Add inline-block line layout microbench
diff --git a/PerformanceTests/MotionMark/developer.html b/PerformanceTests/MotionMark/developer.html
@@ -65,7 +65,7 @@ <h2>Options:</h2>
                         <form name="benchmark-options">
                             <ul>
                                 <li>
-                                    <label>Warmup length: <input type="number" id="warmup-length" value="100"> milliseconds</label>
+                                    <label>Warmup length: <input type="number" id="warmup-length" value="1000"> milliseconds</label>
                                 </li>
                                 <li>
                                     <label>Test length: <input type="number" id="test-interval" value="30"> seconds each</label>
diff --git a/PerformanceTests/MotionMark/resources/runner/motionmark.js b/PerformanceTests/MotionMark/resources/runner/motionmark.js
@@ -465,7 +465,7 @@ window.benchmarkController = {
         "kalman-process-error": 1,
         "kalman-measurement-error": 4,
         "time-measurement": "performance",
-        "warmup-length": 100
+        "warmup-length": 1000
     },
 
     initialize: function()
