Skip to content

Commit

Permalink
Fix a bad merge from D24154657 - second try
Browse files Browse the repository at this point in the history
Summary: This removes a line that reset the dummy batch in begin_epoch. In actuality the batch is reset in get_{train,valid}_iterator, and this line was resetting it to None unnecessarily.

Reviewed By: joshim5, jhcross

Differential Revision: D24157057

fbshipit-source-id: 59ac68327094ceff70f66d7b471fa810997fe84e
  • Loading branch information
Dmitriy Genzel authored and facebook-github-bot committed Oct 7, 2020
1 parent 63aca3f commit b880744
Showing 1 changed file with 0 additions and 3 deletions.
3 changes: 0 additions & 3 deletions fairseq/trainer.py
Original file line number Diff line number Diff line change
Expand Up @@ -413,9 +413,6 @@ def begin_epoch(self, epoch):
# task specific setup per epoch
self.task.begin_epoch(epoch, self.get_model())

# reset dummy batch
self._dummy_batch = None

if self.tpu:
import torch_xla.core.xla_model as xm

Expand Down

0 comments on commit b880744

Please sign in to comment.