Merge pull request #6455 from jepler/require-multi-occur

tannewt · web-flow · commit 5c9768965c2a · 2022-06-03T14:58:02.000-07:00
Only check the savings if a word occurs at least twice
diff --git a/py/makeqstrdata.py b/py/makeqstrdata.py
@@ -400,7 +400,8 @@ def est_net_savings(s, occ):
         # words[] array.
 
         scores = sorted(
-            ((s, -est_net_savings(s, occ)) for (s, occ) in counter.items()), key=lambda x: x[1]
+            ((s, -est_net_savings(s, occ)) for (s, occ) in counter.items() if occ > 1),
+            key=lambda x: x[1],
         )
 
         # Pick the one with the highest score.  The score must be negative.

Original file line number	Diff line number	Diff line change
`@@ -400,7 +400,8 @@ def est_net_savings(s, occ):`
`400`	`400`	`# words[] array.`
`401`	`401`
`402`	`402`	`scores = sorted(`
`403`		`- ((s, -est_net_savings(s, occ)) for (s, occ) in counter.items()), key=lambda x: x[1]`
	`403`	`+ ((s, -est_net_savings(s, occ)) for (s, occ) in counter.items() if occ > 1),`
	`404`	`+ key=lambda x: x[1],`
`404`	`405`	`)`
`405`	`406`
`406`	`407`	`# Pick the one with the highest score. The score must be negative.`