Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(228)

Side by Side Diff: scripts/slave/recipes/libvpx/android_unittests.py

Issue 1782723002: Use buildername rather than slavename for "bot" with perf dashboard. (Closed) Base URL: https://chromium.googlesource.com/chromium/tools/build.git@master
Patch Set: Allow recipe to specify bot name in skeleton point. Created 4 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
« no previous file with comments | « scripts/slave/recipe_modules/perf_dashboard/example.expected/win_use_mirror.json ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 # Copyright 2014 The Chromium Authors. All rights reserved. 1 # Copyright 2014 The Chromium Authors. All rights reserved.
2 # Use of this source code is governed by a BSD-style license that can be 2 # Use of this source code is governed by a BSD-style license that can be
3 # found in the LICENSE file. 3 # found in the LICENSE file.
4 4
5 from recipe_engine.types import freeze 5 from recipe_engine.types import freeze
6 6
7 DEPS = [ 7 DEPS = [
8 'depot_tools/git', 8 'depot_tools/git',
9 'recipe_engine/json', 9 'recipe_engine/json',
10 'recipe_engine/path', 10 'recipe_engine/path',
(...skipping 131 matching lines...) Expand 10 before | Expand all | Expand 10 after
142 # framesPerSecond: fps 142 # framesPerSecond: fps
143 points = [] 143 points = []
144 device = BUILDER_TO_DEVICE[buildername] 144 device = BUILDER_TO_DEVICE[buildername]
145 145
146 #TODO(martiniss) convert loop 146 #TODO(martiniss) convert loop
147 for i in data: 147 for i in data:
148 if i["type"] == "encode_perf_test": 148 if i["type"] == "encode_perf_test":
149 # Two data points for encoder tests, FPS and minPsnr 149 # Two data points for encoder tests, FPS and minPsnr
150 testname = "libvpx/encode/perf_test/fps/" + device + "/" 150 testname = "libvpx/encode/perf_test/fps/" + device + "/"
151 testname = testname + i["videoName"] + "_" + str(i["speed"]) 151 testname = testname + i["videoName"] + "_" + str(i["speed"])
152 p = api.perf_dashboard.get_skeleton_point(testname, 152 p = api.perf_dashboard.get_skeleton_point(
153 libvpx_revision_number, i["framesPerSecond"]) 153 testname,
154 libvpx_revision_number,
155 i["framesPerSecond"],
156 bot=api.m.properties["slavename"])
154 p['units'] = "fps" 157 p['units'] = "fps"
155 points.append(p) 158 points.append(p)
156 159
157 #minPsnr 160 #minPsnr
158 testname = "libvpx/encode/perf_test/minPsnr/" + device + "/" 161 testname = "libvpx/encode/perf_test/minPsnr/" + device + "/"
159 testname = testname + i["videoName"] + "_" + str(i["speed"]) 162 testname = testname + i["videoName"] + "_" + str(i["speed"])
160 p = api.perf_dashboard.get_skeleton_point(testname, 163 p = api.perf_dashboard.get_skeleton_point(
161 libvpx_revision_number, i["minPsnr"]) 164 testname,
165 libvpx_revision_number,
166 i["minPsnr"],
167 bot=api.m.properties["slavename"])
162 p['units'] = "dB" 168 p['units'] = "dB"
163 points.append(p) 169 points.append(p)
164 else: 170 else:
165 testname = "libvpx/decode/perf_test/" + device + "/" 171 testname = "libvpx/decode/perf_test/" + device + "/"
166 testname = testname + i["videoName"] + "_" + str(i["threadCount"]) 172 testname = testname + i["videoName"] + "_" + str(i["threadCount"])
167 p = api.perf_dashboard.get_skeleton_point(testname, 173 p = api.perf_dashboard.get_skeleton_point(
168 libvpx_revision_number, i["framesPerSecond"]) 174 testname,
175 libvpx_revision_number,
176 i["framesPerSecond"],
177 bot=api.m.properties["slavename"])
169 p['units'] = "fps" 178 p['units'] = "fps"
170 points.append(p) 179 points.append(p)
171 180
172 api.perf_dashboard.set_default_config() 181 api.perf_dashboard.set_default_config()
173 api.perf_dashboard.post(points) 182 api.perf_dashboard.post(points)
174 183
175 def GenTests(api): 184 def GenTests(api):
176 # Right now we just support linux, but one day we will have mac and windows 185 # Right now we just support linux, but one day we will have mac and windows
177 # as well 186 # as well
178 yield ( 187 yield (
(...skipping 21 matching lines...) Expand all
200 "encodeTimeSecs": 56.277676, 209 "encodeTimeSecs": 56.277676,
201 "speed" : 5, 210 "speed" : 5,
202 "minPsnr" : 43.5, 211 "minPsnr" : 43.5,
203 "framesPerSecond": 317.976883, 212 "framesPerSecond": 317.976883,
204 "threadCount": 2, 213 "threadCount": 2,
205 "totalFrames": 17895, 214 "totalFrames": 17895,
206 "version": "v1.3.0-2045-g38c2d37", 215 "version": "v1.3.0-2045-g38c2d37",
207 "videoName": "vp90-2-bbb_640x360_tile_1x2_337kbps.webm" 216 "videoName": "vp90-2-bbb_640x360_tile_1x2_337kbps.webm"
208 } 217 }
209 ]))) 218 ])))
OLDNEW
« no previous file with comments | « scripts/slave/recipe_modules/perf_dashboard/example.expected/win_use_mirror.json ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698