cmd/tools/fast: tighten benchmark table layout

Author: medvednikov

cmd/tools/fast/README.md

@@ -160,11 +160,11 @@ export-and-push is their replacement.)
 |-------------|----------------------------------|----------------------------------|
 | `v -o v.c`  | `vprod -o v.c cmd/v`             | self-compile to C                |
 | `v -o v`    | `vprod -o v cmd/v`               | self-compile to a binary         |
+| `V lines / s` | derived                        | V lines per second (`v -o v.c`)  |
+| `V lines`   | `-stats`                          | number of V source lines         |
 | `v hello.v` | `vprod examples/hello_world.v`   | compile a tiny program           |
 | `v.c size`  | –                                | size of the generated `v.c`      |
 | scan/parse/check/cgen | `-show-timings`        | per-stage compiler times (min)   |
-| V lines     | `-stats`                         | number of V source lines         |
-| lines/s     | derived                          | V lines per second (`v -o v.c`)  |
 
 Wall-clock timings take `max_samples` measurements after a couple of warmups and
 discard the slowest ones to cut noise (see the constants in `fast.v`).

cmd/tools/fast/fast.v

@@ -62,10 +62,10 @@ fn git(dir string, subcmd string) string {
 	return lexec('git -C ${os.quoted_path(dir)} ${subcmd}')
 }
 
-// short_hash abbreviates a commit hash for display, without panicking on the
-// 7-char hashes that `import` accepts (a plain `[..8]` slice would).
+// short_hash abbreviates a commit hash to 6 characters for display, without
+// panicking on shorter hashes that `import` accepts.
 fn short_hash(h string) string {
-	return if h.len > 8 { h[..8] } else { h }
+	return if h.len > 6 { h[..6] } else { h }
 }
 
 // exe_name returns the platform-specific executable file name (V and its

cmd/tools/fast/models.v

@@ -414,7 +414,6 @@ struct Delta {
 // here in V (server side), so the template and the browser stay dumb.
 struct Row {
 mut:
-	num         int
 	timestamp   string
 	commit_hash string
 	commit_url  string
@@ -461,9 +460,8 @@ fn build_rows(list []Benchmark) []Row {
 		// the list is newest-first, so the older commit is the next element
 		prev := if i + 1 < list.len { list[i + 1] } else { b }
 		rows << Row{
-			num:         list.len - i
-			timestamp:   b.commit_date.format()
-			commit_hash: b.commit_hash
+			timestamp:   b.commit_date.custom_format('MMM D HH:mm')
+			commit_hash: short_hash(b.commit_hash)
 			commit_url:  'https://github.com/vlang/v/commit/${b.commit_hash}'
 			message:     b.message
 			v_c:         b.v_c_ms

cmd/tools/fast/templates/index.html

@@ -78,8 +78,7 @@
 	}
 	th.left, td.left { text-align: left; }
 	tbody tr:hover { background: #1c202b; }
-	td.num { color: var(--muted); font-size: 11px; }
-	td.msg { max-width: 460px; overflow: hidden; text-overflow: ellipsis; color: var(--fg); }
+	td.msg { max-width: 368px; overflow: hidden; text-overflow: ellipsis; color: var(--fg); }
 	td.commit a { color: var(--accent); text-decoration: none; }
 	.cell { position: relative; }
 	.diff {
@@ -135,24 +134,22 @@ <h2 id="chart-title">Self-compilation time (ms), oldest → newest</h2>
 		<div class="tablewrap">
 		<table>
 			<tr>
-				<th></th>
 				<th class="left">Timestamp</th>
 				<th class="left">Commit</th>
 				<th class="left">Message</th>
 				<th>v -o v.c</th>
 				<th>v -o v</th>
+				<th>V lines / s</th>
+				<th>V lines</th>
 				<th>v hello.v</th>
 				<th>v.c size</th>
 				<th>scan</th>
 				<th>parse</th>
 				<th>check</th>
 				<th>cgen</th>
-				<th>V lines</th>
-				<th>lines/s</th>
 			</tr>
 			@for row in rows
 			<tr>
-				<td class="num">${row.num}</td>
 				<td class="left">${row.timestamp}</td>
 				<td class="left commit"><a href="${row.commit_url}" target="_blank">${row.commit_hash}</a></td>
 				<td class="left msg" title="${row.message}">${row.message}</td>
@@ -166,6 +163,8 @@ <h2 id="chart-title">Self-compilation time (ms), oldest → newest</h2>
 					<span class="diff ${row.d_v_self.cls}">${row.d_v_self.text}</span>
 					@end
 				</td>
+				<td>${row.lines_per_s}</td>
+				<td>${row.vlines}</td>
 				<td class="cell">${row.hello}ms
 					@if row.d_hello.text != ''
 					<span class="diff ${row.d_hello.cls}">${row.d_hello.text}</span>
@@ -176,8 +175,6 @@ <h2 id="chart-title">Self-compilation time (ms), oldest → newest</h2>
 				<td>${row.parse}ms</td>
 				<td>${row.check}ms</td>
 				<td>${row.cgen}ms</td>
-				<td>${row.vlines}</td>
-				<td>${row.lines_per_s}</td>
 			</tr>
 			@end
 		</table>