Skip to content

Commit

Permalink
fix: fixed variable name mismatch in for loop (#408)
Browse files Browse the repository at this point in the history
## 📝 Summary
Fixed the variable name mismatch between the `for` loop and the `print`
statement, where `ms` was used in the loop but `msg` was referenced in
the print statement. This led to a `NameError: name 'msg' is not
defined`.

## 💡 Motivation and Context
This fix resolves the error caused by the variable name inconsistency,
ensuring that the code runs without any issues.

## ✅ I have completed the following steps:

* [x] Run `make lint`
* [x] Run `make test`
* [ ] Added tests (if applicable)
  • Loading branch information
defiberrys authored Feb 10, 2025
1 parent 28d3e66 commit b6a1486
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion scripts/ci/criterion-get-changes.py
Original file line number Diff line number Diff line change
Expand Up @@ -137,5 +137,5 @@ def estimate_to_str(estimates_data):
print("</tbody>")
else:
print("Benchmark \t Mean \t Status \t Significant")
for full_id, url, mean_fmt, ms, is_significant in change_data:
for full_id, url, mean_fmt, msg, is_significant in change_data:
print(f"{full_id} \t {mean_fmt}% \t {msg} \t {is_significant}")

0 comments on commit b6a1486

Please sign in to comment.