Skip to content

Commit f535c05

Browse files
XuehaiPanfacebook-github-bot
authored andcommitted
fix functools.reduce() function with None as initial (#116398)
Summary: The `initial` argument in `functools.reduce` can be `None`. ```python initial_missing = object() def reduce(function, iterable, initial=initial_missing, /): it = iter(iterable) if initial is initial_missing: value = next(it) else: value = initial for element in it: value = function(value, element) return value ``` Reference: - python/cpython#102759 X-link: pytorch/pytorch#116398 Approved by: https://github.com/Skylion007 Reviewed By: izaitsevfb, jeanschmidt Differential Revision: D52424773 fbshipit-source-id: 36d3a6f5f5786350fe8c325773deb135ccbd48e9
1 parent cfe8ebd commit f535c05

File tree

1 file changed

+9
-2
lines changed
  • userbenchmark/dynamo/dynamobench/_dynamo

1 file changed

+9
-2
lines changed

userbenchmark/dynamo/dynamobench/_dynamo/testing.py

Lines changed: 9 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -244,7 +244,14 @@ def normalize_gm(gm_str) -> str:
244244
return remove_trailing_space(strip_comment(gm_str))
245245

246246

247-
def standard_test(self, fn, nargs, expected_ops=None, expected_ops_dynamic=None):
247+
def standard_test(
248+
self,
249+
fn,
250+
nargs,
251+
expected_ops=None,
252+
expected_ops_dynamic=None,
253+
expected_frame_count=1,
254+
):
248255
if not config.assume_static_by_default and expected_ops_dynamic is not None:
249256
expected_ops = expected_ops_dynamic
250257

@@ -265,7 +272,7 @@ def standard_test(self, fn, nargs, expected_ops=None, expected_ops_dynamic=None)
265272
self.assertTrue(same(val1b, correct1))
266273
self.assertTrue(same(val2a, correct2))
267274
self.assertTrue(same(val2b, correct2))
268-
self.assertEqual(actual.frame_count, 1)
275+
self.assertEqual(actual.frame_count, expected_frame_count)
269276
if expected_ops is not None:
270277
self.assertEqual(actual.op_count, expected_ops)
271278

0 commit comments

Comments
 (0)