summaryrefslogtreecommitdiffstats
path: root/Tools/scripts/summarize_stats.py
diff options
context:
space:
mode:
authorMichael Droettboom <mdboom@gmail.com>2023-10-31 00:10:07 (GMT)
committerGitHub <noreply@github.com>2023-10-31 00:10:07 (GMT)
commit9495bcaf599e0961b45447b8fab7f8446945ba1a (patch)
treeb47a02e532d4eda937a65557f4615bc0d65ff131 /Tools/scripts/summarize_stats.py
parent84b4533e8446cbff3325fffe939c87f7120a3ffd (diff)
downloadcpython-9495bcaf599e0961b45447b8fab7f8446945ba1a.zip
cpython-9495bcaf599e0961b45447b8fab7f8446945ba1a.tar.gz
cpython-9495bcaf599e0961b45447b8fab7f8446945ba1a.tar.bz2
gh-111277: In summarize_stats.py, don't fail fast on invalid ratios (#111278)
Diffstat (limited to 'Tools/scripts/summarize_stats.py')
-rw-r--r--Tools/scripts/summarize_stats.py8
1 files changed, 5 insertions, 3 deletions
diff --git a/Tools/scripts/summarize_stats.py b/Tools/scripts/summarize_stats.py
index 0527d4be..360b7c7 100644
--- a/Tools/scripts/summarize_stats.py
+++ b/Tools/scripts/summarize_stats.py
@@ -421,8 +421,6 @@ class Ratio:
self.num = num
self.den = den
self.percentage = percentage
- if den == 0 and num != 0:
- raise ValueError("Invalid denominator")
def __float__(self):
if self.den == 0:
@@ -433,7 +431,11 @@ class Ratio:
return self.num / self.den
def markdown(self) -> str:
- if self.den == 0 or self.den is None:
+ if self.den is None:
+ return ""
+ elif self.den == 0:
+ if self.num != 0:
+ return f"{self.num:,} / 0 !!"
return ""
elif self.percentage:
return f"{self.num / self.den:,.01%}"