search

cleverhans-lab / cleverhans

An adversarial example library for constructing attacks, building defenses, and benchmarking both
MIT License
6.21k stars 1.39k forks source link

Switch mnist dataset mirror to a more reliable one #1201

Closed tejuafonja closed 3 years ago

tejuafonja commented 3 years ago

The current mnist dataset mirror used in the cleverhans_v3 sometimes sporadically return 503. This issue was first noted in 2018 (#496) and considered resolved since everything seem to be working fine since then. Yesterday, @orlp reported this issue again. I looked into it and it's quite sporadic, returning a 503 every once in a while. I thought it best to switch the url once and for all. Since we're using the "https://storage.googleapis.com/cvdf-datasets/mnist/" in our jax tutorial, I wanted to stay consistent with that and changed all referenced " http://yann.lecun.com/exdb/mnist/" to "https://storage.googleapis.com/cvdf-datasets/mnist/". This is the same url referenced in the tf datasets class.

npapernot commented 3 years ago

You will need to resolve conflicts: are you working from the last commit? Could be easier to open a new PR if that’s not the case since this is 2 line diff

tejuafonja commented 3 years ago

Thanks, Nicolas - now resolved.

You will need to resolve conflicts: are you working from the last commit? Could be easier to open a new PR if that’s not the case since this is 2 line diff