facebookresearch / facebookresearch/fairscale

Checkpoint wrapper doesn't support dict return type

Open
#802 2 comments 0 reactions 0 assignees View on GitHub
Dominant language
Python
Stars
3.4k
Forks
293
PR merge metrics
No merged PRs in 30d

Description

## 🐛 Bug

I have been trying to wrap some modules which return dict output type with checkpoint_wrapper. It looks like it only returns keys back.

## To Reproduce

```
from fairscale.nn import checkpoint_wrapper
from torch import nn

class A(nn.Module):
def forward(self, x):
return {"x": 1, "y": 2}

a = checkpoint_wrapper(A())
print(a(None))
## Prints out ("x", "y")
```

## Expected behavior

I should get dict back as it is.

## Environment

fairscale version: 0.3.7

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.