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

Issue 1913813004: Update SK_IMAGE_VERSION, run nanobench on its own image set (Closed)

Created:
4 years, 8 months ago by msarett
Modified:
4 years, 8 months ago
Reviewers:
scroggo, borenet
CC:
reviews_skia.org
Base URL:
https://skia.googlesource.com/skia.git@master
Target Ref:
refs/heads/master
Project:
skia
Visibility:
Public.

Description

Update SK_IMAGE_VERSION, run nanobench on its own image set I've added a new directory to the cloud (gs://chromium-skia-gm/skimage/v4/nanobench) which will allow us to run dm and nanobench on different sets of images. The dm image set will stay the same, and includes incomplete images, edge cases images, etc. The nanobench set is much smaller, and includes jpegs and pngs that we've found useful to bench in the past, plus a couple of each for the other formats. We can refine this set as we move forward (and may also want to monitor these images on perf.skia.org). For now the goal is to reduce nanobench runtime by not testing on many, many obscure images that are not particularly interesting to time. The actual change to use nanobench images is in: https://codereview.chromium.org/1917903003/ BUG=skia:5181 GOLD_TRYBOT_URL= https://gold.skia.org/search2?unt=true&query=source_type%3Dgm&master=false&issue=1913813004 Committed: https://skia.googlesource.com/skia/+/0b55664afc854b03082c467505b1a811721bcd7a

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -1 line) Patch
M SK_IMAGE_VERSION View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 18 (10 generated)
msarett
4 years, 8 months ago (2016-04-26 15:15:31 UTC) #5
commit-bot: I haz the power
Dry run: CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1913813004/1 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1913813004/1
4 years, 8 months ago (2016-04-26 15:17:46 UTC) #7
borenet
LGTM
4 years, 8 months ago (2016-04-26 15:19:52 UTC) #8
commit-bot: I haz the power
Dry run: Try jobs failed on following builders: Test-Ubuntu-GCC-ShuttleA-GPU-GTX660-x86_64-Release-Trybot on client.skia (JOB_FAILED, http://build.chromium.org/p/client.skia/builders/Test-Ubuntu-GCC-ShuttleA-GPU-GTX660-x86_64-Release-Trybot/builds/3160)
4 years, 8 months ago (2016-04-26 15:28:15 UTC) #10
commit-bot: I haz the power
Dry run: CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1913813004/1 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1913813004/1
4 years, 8 months ago (2016-04-26 19:47:33 UTC) #12
commit-bot: I haz the power
Dry run: This issue passed the CQ dry run.
4 years, 8 months ago (2016-04-26 20:06:55 UTC) #14
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1913813004/1 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1913813004/1
4 years, 8 months ago (2016-04-26 20:08:04 UTC) #16
commit-bot: I haz the power
4 years, 8 months ago (2016-04-26 20:09:00 UTC) #18
Message was sent while issue was closed.
Committed patchset #1 (id:1) as
https://skia.googlesource.com/skia/+/0b55664afc854b03082c467505b1a811721bcd7a

Powered by Google App Engine
This is Rietveld 408576698