Skip to content

Commit

Permalink
Revert "Reland "Removing nondeterminism from partitioner eliminates o…
Browse files Browse the repository at this point in the history
…ne graph" (#6703)" (#6774)
  • Loading branch information
JackCaoG authored Mar 19, 2024
1 parent bd4420c commit 30a261a
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions test/dynamo/test_dynamo.py
Original file line number Diff line number Diff line change
Expand Up @@ -574,9 +574,10 @@ def test_resnet18(self):
# Graph 2: backward
# Graph 3: optimizer
# Graph 4: sync input for backward
# Graph 5, 6: PyTorch has updated the number of captured by resnet
# Graph 5: sync input for backward (TODO(JackCaoG) understand why there are two graphs)
# Graph 6, 7: PyTorch has updated the number of captured by resnet
# (https://github.com/pytorch/pytorch/pull/117434)
self.assertEqual(met.metric_data('CompileTime')[0], 6)
self.assertEqual(met.metric_data('CompileTime')[0], 7)
# We execute 4 graphs per step (+ 1 for SGD) when optimizer is enabled.
self.assertEqual(met.metric_data('ExecuteTime')[0], sample_count * 4 + 1)
# one for each forward, backward and optimizer
Expand Down

0 comments on commit 30a261a

Please sign in to comment.