opacus icon indicating copy to clipboard operation
opacus copied to clipboard

Missing argument in MNIST example

Open danperazzo opened this issue 3 years ago • 1 comments

📚 Documentation

Hello everyone, I have been reading the MNIST_README.md to run its example and I noticed an issue.

It mentions that, to run it, we have to use:

python mnist.py --device=cpu -n=15 --lr=.25 --sigma=1.3 -c=1.5 -sr=0.004

However, there is no argument sr on the code, and the results are too different from the reported ones. Thanks for the attention!

danperazzo avatar Oct 04 '22 13:10 danperazzo

Hey danperazzo,

Thanks for flagging this! Labelling is at bug, will investigate.

pierrestock avatar Oct 05 '22 15:10 pierrestock

'-sr' means poisson sampling rate, but the code use '-b' batch size, so just change -sr=0.004 to -b=240, for example.

xuefeng-xu avatar Nov 01 '22 09:11 xuefeng-xu

Fixed with #520

ffuuugor avatar Nov 08 '22 19:11 ffuuugor