Skip to content

Commit aeb3ffb

Browse files
Shuangping Liufacebook-github-bot
authored andcommitted
Clean up tests for tracing (#3381)
Summary: Pull Request resolved: #3381 Clean up the unused dependencies in the unit tests of tracing. Also rename the file to `test_tracing.py`, as the upper directory already indicates this is train pipeline tests. Reviewed By: TroyGarden Differential Revision: D82658018 fbshipit-source-id: 3c22d6c08eef3f17c0653bf1c24669a3a85196fd
1 parent ef33c80 commit aeb3ffb

File tree

2 files changed

+1
-9
lines changed

2 files changed

+1
-9
lines changed

torchrec/distributed/train_pipeline/tests/test_train_pipeline_tracing.py renamed to torchrec/distributed/train_pipeline/tests/test_tracing.py

Lines changed: 1 addition & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -8,26 +8,19 @@
88
# pyre-strict
99

1010
import unittest
11-
from typing import List, Optional
11+
from typing import List
1212
from unittest.mock import MagicMock
1313

1414
import parameterized
1515

1616
import torch
17-
from torch import nn
18-
1917
from torchrec.distributed.train_pipeline.pipeline_context import TrainPipelineContext
20-
2118
from torchrec.distributed.train_pipeline.tracing import (
22-
_get_leaf_module_names,
2319
ArgInfo,
2420
ArgInfoStepFactory,
2521
CallArgs,
2622
NodeArgsHelper,
27-
PipelinedPostproc,
28-
Tracer,
2923
)
30-
from torchrec.distributed.types import NullShardedModuleContext, ShardedModule
3124
from torchrec.sparse.jagged_tensor import KeyedJaggedTensor
3225

3326

torchrec/distributed/train_pipeline/tracing.py

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,6 @@
77

88
# pyre-strict
99
import logging
10-
from dataclasses import dataclass
1110
from itertools import chain
1211
from typing import Any, Dict, List, Optional, Set, Tuple, Union
1312

0 commit comments

Comments
 (0)