opacus icon indicating copy to clipboard operation
opacus copied to clipboard

Include batch_size as an argument, instead of sample_rate

Open ashkan-software opened this issue 3 years ago • 1 comments

Summary: Other examples in Opacus follow this logic for arguments related to batch size:

  • batch_size and batch_size_test

CIFAR-10 still uses the sample_rate * len(train_data).

This diff updates that, to aslo make it clearer (someone was confused here)

Differential Revision: D38799564

ashkan-software avatar Aug 17 '22 20:08 ashkan-software

This pull request was exported from Phabricator. Differential Revision: D38799564

facebook-github-bot avatar Aug 17 '22 20:08 facebook-github-bot

This pull request was exported from Phabricator. Differential Revision: D38799564

facebook-github-bot avatar Aug 23 '22 20:08 facebook-github-bot

This pull request was exported from Phabricator. Differential Revision: D38799564

facebook-github-bot avatar Aug 25 '22 19:08 facebook-github-bot

This pull request was exported from Phabricator. Differential Revision: D38799564

facebook-github-bot avatar Sep 01 '22 15:09 facebook-github-bot

This pull request was exported from Phabricator. Differential Revision: D38799564

facebook-github-bot avatar Sep 01 '22 16:09 facebook-github-bot