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

Unified Diff: experimental/skpdiff/main.cpp

Issue 19826002: add --csv parameter to skpdiff to dump all scores in a csv file. We can run it with all skps, and h… (Closed) Base URL: http://skia.googlecode.com/svn/trunk/
Patch Set: Created 7 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « experimental/skpdiff/SkDiffContext.cpp ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: experimental/skpdiff/main.cpp
===================================================================
--- experimental/skpdiff/main.cpp (revision 10232)
+++ experimental/skpdiff/main.cpp (working copy)
@@ -32,6 +32,7 @@
DEFINE_string2(patterns, p, "", "Use two patterns to compare images: <baseline> <test>");
DEFINE_string2(output, o, "skpdiff_output.json", "Writes the output of these diffs to output: <output>");
DEFINE_bool(jsonp, true, "Output JSON with padding");
+DEFINE_string(csv, "", "Writes the output of these diffs to a csv file");
#if SK_SUPPORT_OPENCL
/// A callback for any OpenCL errors
@@ -169,6 +170,13 @@
}
}
+ if (!FLAGS_csv.isEmpty()) {
+ if (1 != FLAGS_csv.count()) {
+ SkDebugf("csv flag expects one argument: <csv file>\n");
+ return 1;
+ }
+ }
+
SkDiffContext ctx;
ctx.setDiffers(chosenDiffers);
@@ -188,5 +196,10 @@
ctx.outputRecords(outputStream, FLAGS_jsonp);
}
+ if (!FLAGS_csv.isEmpty()) {
+ SkFILEWStream outputStream(FLAGS_csv[0]);
+ ctx.outputCsv(outputStream);
+ }
+
return 0;
}
« no previous file with comments | « experimental/skpdiff/SkDiffContext.cpp ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698