diff options
| author |  Hal Canary <halcanary@google.com> | 2018-01-05 14:00:28 -0500 |
|---|---|---|
| committer |  Skia Commit-Bot <skia-commit-bot@chromium.org> | 2018-01-07 03:07:46 +0000 |
| commit | 3de24bf20cd9bff6ecbe8d262b596658642ab7c3 (patch) | |
| tree | 58595de14f50fbdc8d16b69dfa6aaed5f18ae8cc /tools/skqp | |
| parent | 1b3bd9cfc5a4c6ede24ec410e643f585a475fad2 (diff) | |
SkQP: dump csv report
Change-Id: I7485ed3315acdc3dd3b795a42846ed3deedb6b25
Reviewed-on: https://skia-review.googlesource.com/91403
Reviewed-by: Derek Sollenberger <djsollen@google.com>
Commit-Queue: Hal Canary <halcanary@google.com>
Diffstat (limited to 'tools/skqp')
| -rw-r--r-- | tools/skqp/gm_knowledge.cpp | 23 |
1 files changed, 21 insertions, 2 deletions
diff --git a/tools/skqp/gm_knowledge.cpp b/tools/skqp/gm_knowledge.cpp index fb2134f954..7670aa9ba7 100644 --- a/tools/skqp/gm_knowledge.cpp +++ b/tools/skqp/gm_knowledge.cpp @@ -24,7 +24,6 @@ #include "SkPngEncoder.h" #include "SkStream.h" - #include "skqp_asset_manager.h" #define PATH_MAX_PNG "max.png" @@ -170,6 +169,10 @@ float Check(const uint32_t* pixels, ++badPixelCount; } } + if (badness == 0) { + std::lock_guard<std::mutex> lock(gMutex); + gErrors.push_back(Run{SkString(backend), SkString(name), 0, 0}); + } if (report_directory_path && badness > 0 && report_directory_path[0] != '\0') { sk_mkdir(report_directory_path); if (!backend) { @@ -209,6 +212,20 @@ float Check(const uint32_t* pixels, } bool MakeReport(const char* report_directory_path) { + std::lock_guard<std::mutex> lock(gMutex); + { + SkFILEWStream csvOut(SkOSPath::Join(report_directory_path, "out.csv").c_str()); + if (!csvOut.isValid()) { + return false; + } + for (const Run& run : gErrors) { + SkString line = SkStringPrintf("\"%s\",\"%s\",%d,%d\n", + run.fBackend.c_str(), run.fGM.c_str(), + run.fMaxerror, run.fBadpixels); + csvOut.write(line.c_str(), line.size()); + } + } + SkFILEWStream out(SkOSPath::Join(report_directory_path, PATH_REPORT).c_str()); if (!out.isValid()) { return false; @@ -226,12 +243,14 @@ bool MakeReport(const char* report_directory_path) { "<body>\n" "<h1>SkQP Report</h1>\n" "<hr>\n"); - std::lock_guard<std::mutex> lock(gMutex); for (const Run& run : gErrors) { const SkString& backend = run.fBackend; const SkString& gm = run.fGM; int maxerror = run.fMaxerror; int badpixels = run.fBadpixels; + if (maxerror == 0 && badpixels == 0) { + continue; + } SkString rdir = SkOSPath::Join(backend.c_str(), gm.c_str()); SkString text = SkStringPrintf( "<h2>%s</h2>\n" |