Remove OOPIF benchmark |
|||||||
Issue descriptionThis is part of the effort to make Telemetry net simulation more realistic. Nasko: can you help triaging converting WPR archives used by oopif benchmarks to WPRGO? The instruction is in: bit.ly/wpr-go-migration
,
Aug 30 2017
Ping nasko@
,
Aug 30 2017
The OOPIF benchmark was one of the discussions for us yesterday and the conclusion is that we should just remove it, as it doesn't give us as much meaningful data anymore. Instead of converting the archives, I will just remove them. Is there any doc on what is needed to remove a benchmark and its associated WPR files?
,
Aug 30 2017
If you think we can just remove the benchmark, I can take this bug from now on :-)
,
Aug 30 2017
,
Aug 30 2017
Yes, I think it is best for us to remove it. Thanks!
,
Aug 30 2017
,
Aug 30 2017
The following revision refers to this bug: https://chromium.googlesource.com/chromium/src.git/+/fb6e6ca79e23843ea491d246e29c8a39908ee4e0 commit fb6e6ca79e23843ea491d246e29c8a39908ee4e0 Author: Ned Nguyen <nednguyen@google.com> Date: Wed Aug 30 23:25:11 2017 Remove oopif benchmarks This also remove the benchmarks' corresponding page_set, WPR archives. Bug: 756396 Change-Id: I8fea69361f4bd8416b88b3e55d1d895822440396 Reviewed-on: https://chromium-review.googlesource.com/644486 Reviewed-by: Charlie Reis <creis@chromium.org> Commit-Queue: Ned Nguyen <nednguyen@google.com> Cr-Commit-Position: refs/heads/master@{#498663} [modify] https://crrev.com/fb6e6ca79e23843ea491d246e29c8a39908ee4e0/testing/buildbot/chromium.perf.fyi.json [modify] https://crrev.com/fb6e6ca79e23843ea491d246e29c8a39908ee4e0/testing/buildbot/chromium.perf.json [modify] https://crrev.com/fb6e6ca79e23843ea491d246e29c8a39908ee4e0/tools/perf/benchmark.csv [delete] https://crrev.com/0af324ecfba2c9f45a8e58ed02da41e926694903/tools/perf/benchmarks/oopif.py [delete] https://crrev.com/0af324ecfba2c9f45a8e58ed02da41e926694903/tools/perf/page_sets/data/oopif_basic.json [delete] https://crrev.com/0af324ecfba2c9f45a8e58ed02da41e926694903/tools/perf/page_sets/data/oopif_basic_000.wpr.sha1 [delete] https://crrev.com/0af324ecfba2c9f45a8e58ed02da41e926694903/tools/perf/page_sets/oopif_basic_page_set.py
,
Aug 30 2017
,
Aug 31 2017
Issue 670256 has been merged into this issue. |
|||||||
►
Sign in to add a comment |
|||||||
Comment 1 by nedngu...@google.com
, Aug 17 2017