Revision 3ba4725e067b439ef7997b9063e3986ea938a3b4 authored by Amanpreet Singh on 19 March 2021, 22:17:27 UTC, committed by Facebook GitHub Bot on 19 March 2021, 22:18:53 UTC
Summary:
The calculation of length for multidataloader used get_batch_size to
divide total dataset length which is incorrect as get_batch_size is per
device rather than global batch size. This PR fixes it and fixes
snapshot interval calculation as well. This in turn:
- Fixes max_epochs support
- ETA
- All other things dependent on this

Pull Request resolved: https://github.com/facebookresearch/mmf/pull/820

Test Plan: Tested locally with MVLT runs

Reviewed By: ytsheng, madian9

Differential Revision: D27184804

Pulled By: apsdehal

fbshipit-source-id: d18aeb52b7e280f33c3b9e7509dafa123ed1c629
1 parent 08fb370
History
File Mode Size
datasets
models
zoo
defaults.yaml -rw-r--r-- 13.6 KB

back to top