Commit 1414d14
Re-land D66465376 (#2637)
Summary:
Re-land diff D66465376
NOTE: use jit.ignore on the forward function to get rid of jit script error with `TensorDict`
```
def test_td_scripting(self) -> None:
class TestModule(torch.nn.Module):
torch.jit.ignore # <----- test fails without this ignore
def forward(self, x: Union[TensorDict, KeyedJaggedTensor]) -> torch.Tensor:
if isinstance(x, TensorDict):
keys = list(x.keys())
return torch.cat([x[key]._values for key in keys], dim=0)
else:
return x._values
m = TestModule()
gm = torch.fx.symbolic_trace(m)
jm = torch.jit.script(gm)
values = torch.tensor([0, 1, 2, 3, 2, 3, 4])
kjt = KeyedJaggedTensor.from_offsets_sync(
keys=["f1", "f2", "f3"],
values=values,
offsets=torch.tensor([0, 2, 2, 3, 4, 5, 7]),
)
torch.testing.assert_allclose(jm(kjt), values)
```
Differential Revision: D664603921 parent f059a49 commit 1414d14
4 files changed
Lines changed: 28 additions & 2 deletions
File tree
- torchrec
- distributed
- modules
- sparse
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
97 | 97 | | |
98 | 98 | | |
99 | 99 | | |
| 100 | + | |
| 101 | + | |
| 102 | + | |
| 103 | + | |
| 104 | + | |
| 105 | + | |
| 106 | + | |
| 107 | + | |
100 | 108 | | |
101 | 109 | | |
102 | 110 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
98 | 98 | | |
99 | 99 | | |
100 | 100 | | |
| 101 | + | |
| 102 | + | |
| 103 | + | |
| 104 | + | |
| 105 | + | |
| 106 | + | |
| 107 | + | |
101 | 108 | | |
102 | 109 | | |
103 | 110 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
21 | 21 | | |
22 | 22 | | |
23 | 23 | | |
| 24 | + | |
| 25 | + | |
| 26 | + | |
| 27 | + | |
| 28 | + | |
| 29 | + | |
| 30 | + | |
| 31 | + | |
24 | 32 | | |
25 | 33 | | |
26 | 34 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
49 | 49 | | |
50 | 50 | | |
51 | 51 | | |
52 | | - | |
| 52 | + | |
53 | 53 | | |
54 | | - | |
| 54 | + | |
| 55 | + | |
| 56 | + | |
| 57 | + | |
55 | 58 | | |
56 | 59 | | |
57 | 60 | | |
| |||
0 commit comments