diff options
| author | cmtice <cmtice@google.com> | 2013-12-06 15:46:49 -0800 |
|---|---|---|
| committer | chrome-internal-fetch <chrome-internal-fetch@google.com> | 2013-12-10 22:42:42 +0000 |
| commit | 9847df92a2b5f76ccddc4bf10288819712a8ca47 (patch) | |
| tree | e36fbfd5361e0895c4254abdd2f8d7147d36e756 /crosperf/settings_factory.py | |
| parent | d80a960b62d89bc6f83d2f5fd4d599408f2ec7f9 (diff) | |
| download | toolchain-utils-9847df92a2b5f76ccddc4bf10288819712a8ca47.tar.gz | |
Use default results for Telemetry tests; add option to use all results.
When running telemetry_Crosperf, use the json file of report defaults to
determine which results to actually include in the report. This CL
also adds a flag to Crospref, --show_all_results=True, that allows the
user to override the defaults and generate a report showing all the values.
BUG=None
TEST=Tested this both with and without the --show_all_results flag. It
works as expected for me.
Change-Id: I91364572ebdf3d54bb8b408fbd5a2580923f6829
Reviewed-on: https://chrome-internal-review.googlesource.com/149457
Reviewed-by: Yunlian Jiang <yunlian@google.com>
Commit-Queue: Caroline Tice <cmtice@google.com>
Tested-by: Caroline Tice <cmtice@google.com>
Diffstat (limited to 'crosperf/settings_factory.py')
| -rw-r--r-- | crosperf/settings_factory.py | 12 |
1 files changed, 10 insertions, 2 deletions
diff --git a/crosperf/settings_factory.py b/crosperf/settings_factory.py index 05647c96..84c615c9 100644 --- a/crosperf/settings_factory.py +++ b/crosperf/settings_factory.py @@ -43,8 +43,12 @@ class BenchmarkSettings(Settings): self.AddField(TextField("suite", default="pyauto", description="The type of the benchmark")) self.AddField(BooleanField("use_test_that", default=False, - description="(experimental) Whether to use the" + description="Whether to use the" " new test_that script for running the test.")) + self.AddField(BooleanField("show_all_results", default=False, + description="When running Telemetry tests, " + "whether to all the results, instead of just " + "the default (summary) results.")) class LabelSettings(Settings): @@ -134,8 +138,12 @@ class GlobalSettings(Settings): description="Whether to disable the email to " "user after crosperf finishes.")) self.AddField(BooleanField("use_test_that", default=False, - description="(experimental) Whether to use the " + description="Whether to use the " "new test_that script for running the test.")) + self.AddField(BooleanField("show_all_results", default=False, + description="When running Telemetry tests, " + "whether to all the results, instead of just " + "the default (summary) results.")) self.AddField(TextField("share_users", default="", description="Who's cache data you want to " "use. It accepts multiple users seperated by \",\"")) |