Skip to content

Commit

Permalink
Fix off-by-one error in proof summary count (#207)
Browse files Browse the repository at this point in the history
We were claiming one more successful proof than we actually had for
successful execution of "print_tool_versions" was also counted as a
proof.
  • Loading branch information
tautschnig committed Jun 25, 2024
1 parent 9db3f07 commit a5daf89
Showing 1 changed file with 2 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -86,13 +86,13 @@ def _get_status_and_proof_summaries(run_dict):
count_statuses = {}
proofs = [["Proof", "Status"]]
for proof_pipeline in run_dict["pipelines"]:
if proof_pipeline["name"] == "print_tool_versions":
continue
status_pretty_name = proof_pipeline["status"].title().replace("_", " ")
try:
count_statuses[status_pretty_name] += 1
except KeyError:
count_statuses[status_pretty_name] = 1
if proof_pipeline["name"] == "print_tool_versions":
continue
proofs.append([proof_pipeline["name"], status_pretty_name])
statuses = [["Status", "Count"]]
for status, count in count_statuses.items():
Expand Down

0 comments on commit a5daf89

Please sign in to comment.