Skip to content

Commit 7987146

Browse files
committed
chore: cleanup comments
1 parent fc074bd commit 7987146

2 files changed

Lines changed: 0 additions & 10 deletions

File tree

.github/scripts/benchmark_formatter.py

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -148,8 +148,6 @@ def extract_two_numbers(tokens):
148148
if padding < 2:
149149
padding = 2 # minimum spacing
150150
# If header is wider than data (unlikely but possible), adjust diff_col_start
151-
# But for now let's trust max_content_width or just append
152-
153151
if len(stripped_header) < diff_col_start:
154152
new_header = stripped_header + " " * (diff_col_start - len(stripped_header))
155153
else:
@@ -191,10 +189,6 @@ def append_aligned(left_part, content):
191189
aligned = left_part + " "
192190

193191
# Ensure content doesn't exceed right boundary (visual check only, we don't truncate)
194-
# But users asked not to exceed header pipe.
195-
# Header pipe is at right_boundary.
196-
# Content starts at diff_col_start.
197-
# So content length should be <= right_boundary - diff_col_start
198192
return f"{aligned}{content}"
199193

200194
# Special handling for geomean when values missing or zero

.github/scripts/pytest_benchstat.py

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -196,10 +196,6 @@ def format_cell(val, std, rounds):
196196
g_pr = math.exp(sum(math.log(x) for x in pr_geo_input) / len(pr_geo_input))
197197
g_pr_str = f"{format_val(g_pr)}"
198198

199-
# If we had zeros, we might want to indicate it, but for now just print what we have
200-
# benchmark_formatter.py handles "n/a (has zero)" if it detects issues,
201-
# but here we output the calculated mean of non-zeroes or N/A.
202-
203199
print(f"{'geomean':<{w_name}} {g_base_str:<{w_val}} {g_pr_str:<{w_val}}")
204200

205201
# Print Footnotes

0 commit comments

Comments
 (0)