Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Ddp kwargs #52

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Conversation

nbroad1881
Copy link

I made a custom model and I got this error:

RuntimeError: Expected to have finished reduction in the prior iteration before 
starting a new one. This error indicates that your module has parameters that 
were not used in producing loss. You can enable unused parameter detection by 
passing the keyword argument `find_unused_parameters=True` to 
`torch.nn.parallel.DistributedDataParallel`, 
and by making sure all `forward` function outputs participate in calculating loss. 

If you already have done the above, then the distributed data parallel module 
wasn't able to locate the output tensors in the return value of your module's 
`forward` function. Please include the loss function and the structure of the 
return value of `forward` of your module when reporting this issue (e.g. list, 
dict, iterable).

I found this issue in the accelerate repo which indicated that the solution was to do the following

from accelerate import DistributedDataParallelKwargs

ddp_kwargs = DistributedDataParallelKwargs(find_unused_parameters=True)
accelerator = Accelerator(kwargs_handlers=[ddp_kwargs])

I added another argument to Tez that allows the user to set that.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant