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
common
configs
datasets
models
modules
trainers
utils
__init__.py -rw-r--r-- 401 bytes
projects l--------- 11 bytes
version.py -rw-r--r-- 209 bytes

back to top