Skip to content

Commit dde7058

Browse files
masnesralpytorchmergebot
authored andcommitted
Fix test broken by D73809989 (pytorch#153413)
Summary: I forgot to remove this unused field in D73809989. Test Plan: `buck test 'fbcode//mode/opt' fbcode//caffe2/test:fbonly -- --exact 'caffe2/test:fbonly - test_compilation_metrics_logger_in_sync (caffe2.test.fb.test_fb.TestFBOnly)'` Pull Request resolved: pytorch#153413 Approved by: https://github.com/c00w
1 parent 216e28f commit dde7058

File tree

2 files changed

+0
-3
lines changed

2 files changed

+0
-3
lines changed

test/dynamo/test_utils.py

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -310,7 +310,6 @@ def test_dynamo_timed(self, mock_time, mock_time_ns):
310310
'compliant_custom_ops': set(),
311311
'config_inline_inbuilt_nn_modules': False,
312312
'config_suppress_errors': False,
313-
'cuda_synchronize_time_us': None,
314313
'cuda_version': None,
315314
'cudagraph_skip_reason': None,
316315
'distributed_ephemeral_timeout_us': None,
@@ -402,7 +401,6 @@ def test_dynamo_timed(self, mock_time, mock_time_ns):
402401
'compliant_custom_ops': None,
403402
'config_inline_inbuilt_nn_modules': None,
404403
'config_suppress_errors': None,
405-
'cuda_synchronize_time_us': None,
406404
'cuda_version': None,
407405
'cudagraph_skip_reason': None,
408406
'distributed_ephemeral_timeout_us': None,

torch/_dynamo/utils.py

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1245,7 +1245,6 @@ class CompilationMetrics:
12451245
runtime_cudagraphify_time_us: Optional[int] = None
12461246
runtime_triton_autotune_time_us: Optional[int] = None
12471247
dynamo_compile_time_before_restart_us: Optional[int] = None
1248-
cuda_synchronize_time_us: Optional[int] = None # TODO: instrument
12491248
distributed_ephemeral_timeout_us: Optional[int] = None
12501249
structured_logging_overhead_us: Optional[int] = None
12511250
remote_fx_graph_cache_get_time_us: Optional[int] = None

0 commit comments

Comments
 (0)