@@ -749,7 +749,7 @@ def _samples(runs: list[dict[str, Any]], target: str, source: str, key: str,
749749
750750def _fmt_range (values : list [float ], unit : str ) -> str :
751751 if not values :
752- return "— "
752+ return ", "
753753 lo , hi = min (values ), max (values )
754754 suffix = "x" if unit == "x" else " ns"
755755 if abs (hi - lo ) < 0.005 :
@@ -780,7 +780,7 @@ def render_markdown(report: dict[str, Any]) -> str:
780780 targets = report .get ("targetsRequested" ) or DEFAULT_TARGETS
781781 lines : list [str ] = []
782782
783- dirty = " **(working tree dirty — these numbers are not reproducible from the commit alone)**" \
783+ dirty = " **(working tree dirty, these numbers are not reproducible from the commit alone)**" \
784784 if report .get ("workingTreeDirty" ) else ""
785785 lines .append (f"_Generated by `benchmarks/cost_table.py` from "
786786 f"`{ report .get ('sourceFile' , 'the newest JSON' )} )
@@ -815,31 +815,31 @@ def render_markdown(report: dict[str, Any]) -> str:
815815 host = report ["host" ]
816816 warm = report .get ("benchmarkTestWarmup" , {})
817817 lines += [
818- f"- **commit** `{ report ['commit' ][:12 ]} — { report .get ('commitSubject' , '' )} { dirty } ,
818+ f"- **commit** `{ report ['commit' ][:12 ]} , { report .get ('commitSubject' , '' )} { dirty } ,
819819 f"- **branch** `{ report .get ('branch' , '?' )} { report .get ('worktree' , '?' )} ,
820820 f"- **captured** { report ['capturedAt' ]}
821- + (" — the first of several cuts; each one's own time is in the source table below"
821+ + (", the first of several cuts; each one's own time is in the source table below"
822822 if len (report .get ("sources" ) or []) > 1 else "" ),
823823 f"- **host** { host .get ('cpu' , '?' )} { host .get ('cores' , '?' )}
824824 f"{ host .get ('os' , '?' )} { host .get ('machine' , '' )} ,
825825 f"- **load average** { report .get ('loadAtStart' )} { report .get ('loadAtEnd' )}
826826 f"(ceiling { report ['loadCeiling' ]} { report .get ('loadGuard' )}
827- + (" — of the first cut only; per-cut figures below"
827+ + (", of the first cut only; per-cut figures below"
828828 if len (report .get ("sources" ) or []) > 1 else "" ),
829- f"- **runs per target** { report ['runsRequested' ]} — ranges below are min–max over them" ,
829+ f"- **runs per target** { report ['runsRequested' ]} , ranges below are min–max over them" ,
830830 f"- **`UpcallBoundaryCostTest`** warmup and iteration counts are self-reported per target "
831831 "in the table below (the test prints them)" ,
832832 f"- **`BenchmarkTest`** warmup, read from the source at this commit: "
833833 f"`WARMUP` = { warm .get ('WARMUP' , '?' )} { warm .get ('WARMUP_BULK' , '?' )}
834- "(the ≥4096-char string rows use the second — see that file's own caveat)" ,
834+ "(the ≥4096-char string rows use the second, see that file's own caveat)" ,
835835 "- **suite scope** full task, never `--tests`-filtered: narrowing the suite changes what is "
836836 "being measured" ,
837837 "" ,
838838 ]
839839
840840 sources = report .get ("sources" ) or []
841841 if len (sources ) > 1 :
842- lines += ["This table is merged from several cuts — which target came from which:" , "" ]
842+ lines += ["This table is merged from several cuts, which target came from which:" , "" ]
843843 lines += _table (
844844 ["cut" , "commit" , "captured" , "load at start" , "contaminated" , "targets" ],
845845 [[f"`{ s ['file' ]} , f"`{ s ['commit' ]} , str (s .get ("capturedAt" )),
@@ -875,16 +875,16 @@ def render_markdown(report: dict[str, Any]) -> str:
875875 device = sim
876876 wall = "/" .join (str (r .get ("wallSeconds" , "?" )) for r in mine )
877877 counts = (f"{ suite .get ('tests' , '?' )} { suite .get ('failures' , '?' )} { suite .get ('skipped' , '?' )}
878- if suite else "— " )
878+ if suite else ", " )
879879 rows .append ([
880880 f"`{ name } ,
881881 f"`{ target .task } if target else "?" ,
882882 status if not reason else f"**{ status } ,
883- device or ("— " if not (target and target .avd ) else "— " ),
884- platform_name or "— " ,
883+ device or (", " if not (target and target .avd ) else ", " ),
884+ platform_name or ", " ,
885885 counts ,
886- wall or "— " ,
887- conditions or "— " ,
886+ wall or ", " ,
887+ conditions or ", " ,
888888 ])
889889 lines += _table (
890890 ["target" , "task" , "status" , "device / simulator" , "platform, self-reported" ,
@@ -895,7 +895,7 @@ def render_markdown(report: dict[str, Any]) -> str:
895895 notes = [(n , r ) for n , r in notes if r ]
896896 if notes :
897897 lines += ["**Targets that did not produce numbers, and why:**" , "" ]
898- lines += [f"- `{ n } — { r } for n , r in notes ]
898+ lines += [f"- `{ n } , { r } for n , r in notes ]
899899 lines .append ("" )
900900
901901 # ---- upcall --------------------------------------------------------------------
@@ -913,7 +913,7 @@ def render_markdown(report: dict[str, Any]) -> str:
913913 lines += _table (["target" ] + [c for c , _ in UPCALL_COLUMNS ], rows )
914914 lines .append ("" )
915915
916- lines += ["Controls, from the same block — these are what say whether a row moved because the "
916+ lines += ["Controls, from the same block, these are what say whether a row moved because the "
917917 "boundary moved or because the whole host did:" , "" ]
918918 rows = []
919919 for name in targets :
0 commit comments