The order of the parameters is reversed. The two parameter (save_every, total_epochs)positions should be interchanged.
RuleNHao opened this issue · comments
tianph commented
tianph commented
It took me hours to find the bug. This is an unforgettable experience.
tianph commented
And it also need to use sampler.set_epoch()
method at the beginning of each epoch.
Steven Liu commented
cc: @suraj813
Suraj Subramanian commented
@RuleNHao thanks for catching this!
set_epoch()
is an important suggestion to ensure random shuffling; thank you for bringing this up, I'll update the example to include this.