Shorten names of gradle benchmarks
Removes noise from the names, should easily map back to the gradle tasks anyway
Jean-Marie: let me know if you think this will clash with future additions (e.g, we would like something like release be "default" instead of debug)
Change-Id: I3b430d6b4d4a9c3eeeee63373016de3cdcca9574
diff --git a/tools/test_gradle_benchmarks.py b/tools/test_gradle_benchmarks.py
index 6720c4a..a32b2a5 100755
--- a/tools/test_gradle_benchmarks.py
+++ b/tools/test_gradle_benchmarks.py
@@ -44,16 +44,18 @@
cleanCommand = ""
env = {}
- def __init__(self, displayName, benchmarkDir, moduleName, buildCommand, cleanCommand):
+ def __init__(self, displayName, benchmarkDir, moduleName, buildCommand,
+ cleanCommand):
self.displayName = displayName
- self.rootDirPath = os.path.join(BENCHMARKS_ROOT_DIR, benchmarkDir.split(os.sep)[0])
+ self.rootDirPath = os.path.join(BENCHMARKS_ROOT_DIR,
+ benchmarkDir.split(os.sep)[0])
self.appPath = os.path.join(BENCHMARKS_ROOT_DIR, benchmarkDir)
self.moduleName = moduleName
self.buildCommand = buildCommand
self.cleanCommand = cleanCommand
self.env = os.environ.copy()
- self.env["ANDROID_HOME"] = os.path.join(utils.REPO_ROOT, 'third_party', 'benchmarks',
- 'android-sdk')
+ self.env["ANDROID_HOME"] = os.path.join(utils.REPO_ROOT, 'third_party',
+ 'benchmarks', 'android-sdk')
def RunGradle(self, command, tool, desugarMode):
@@ -83,7 +85,9 @@
def Clean(self):
# tools and desugar mode not relevant for clean
- return self.RunGradle(self.cleanCommand, self.Tools.D8, self.DesugarMode.D8_DESUGARING)
+ return self.RunGradle(self.cleanCommand,
+ self.Tools.D8,
+ self.DesugarMode.D8_DESUGARING)
def EnsurePresence(self):
EnsurePresence(self.rootDirPath, self.displayName)
@@ -126,17 +130,25 @@
taskName = commaSplit[1][(len(benchmark.moduleName) + 1):]
# Just a temporary assumption.
- # This means we have submodules, so we'll need to check their configuration
- # so that the right r8/d8 is taken. For now it shouldn't be the case.
+ # This means we have submodules, so we'll need to check their
+ # configuration so that the right r8/d8 is taken. For now it shouldn't
+ # be the case.
assert taskName.find(':') == -1, taskName
- # Output example:
- # SantaTracker-transformClassesWithDexBuilderForDevelopmentDebug(RunTimeRaw): 748 ms
-
- golemBenchmarkValue = benchmark.displayName + '-' + taskName + '(RunTimeRaw): '
if TaskFilter(taskName):
- print('{}(RunTimeRaw): {} ms'
- .format(benchmark.displayName + '-' + taskName, commaSplit[2]))
+ # taskName looks like:
+ # transformClassesWithDexBuilderForDevelopmentDebug
+ # we don't want unimportant information in UI, so we strip it down to:
+ # ClassesDexBuilderDevelopment
+ # Output example:
+ # SantaTracker-ClassesDexBuilderDevelopment(RunTimeRaw): 748 ms
+ assert taskName.startswith('transform')
+ taskName = taskName[len('transform'):]
+ taskName = taskName.replace('With', '')
+ taskName = taskName.replace('For', '')
+ taskName = taskName.replace('Default', '')
+ benchmarkName = benchmark.displayName + '-' + taskName
+ print('{}(RunTimeRaw): {} ms'.format(benchmarkName, commaSplit[2]))
def Main():
@@ -158,17 +170,20 @@
Benchmark('Maps',
'gradle-java-1.6',
':maps',
- [':maps:assembleDebug', '--settings-file', 'settings.gradle.maps'],
+ [':maps:assembleDebug', '--settings-file',
+ 'settings.gradle.maps'],
['clean']),
Benchmark('Music2',
'gradle-java-1.6',
':music2Old',
- [':music2Old:assembleDebug', '--settings-file', 'settings.gradle.music2Old'],
+ [':music2Old:assembleDebug', '--settings-file',
+ 'settings.gradle.music2Old'],
['clean']),
Benchmark('Velvet',
'gradle-java-1.6',
':velvet',
- [':velvet:assembleDebug', '--settings-file', 'settings.gradle.velvet'],
+ [':velvet:assembleDebug', '--settings-file',
+ 'settings.gradle.velvet'],
['clean']),
Benchmark('SantaTracker',
'santa-tracker',
