Saving to /tmp

21 views
Skip to first unread message

Ian Goodfellow

unread,
Jul 9, 2018, 1:14:10 PM7/9/18
to cleverhans dev
This PR ( https://github.com/tensorflow/cleverhans/pull/445 ) changes the save location for one of the tutorials from "/tmp" to "./train_dir". Does anyone remember why we were writing to /tmp before? Do we need to do that so the default is friendly to processes that don't have write access to the cleverhans install location or something like that? Because I don't remember a reason to use specifically "/tmp" I'll assume it's OK to change unless someone replies with a reason not to change it.

Nicolas Papernot

unread,
Jul 10, 2018, 6:31:03 AM7/10/18
to Ian Goodfellow, cleverhans dev
I can't remember of any reasons we would have selected /tmp instead of the tutorials working directory. The PR passed Travis tests so I don't see any reasons not to change it. 

On Mon, Jul 9, 2018 at 7:14 PM 'Ian Goodfellow' via cleverhans dev <cleverh...@googlegroups.com> wrote:
This PR ( https://github.com/tensorflow/cleverhans/pull/445 ) changes the save location for one of the tutorials from "/tmp" to "./train_dir". Does anyone remember why we were writing to /tmp before? Do we need to do that so the default is friendly to processes that don't have write access to the cleverhans install location or something like that? Because I don't remember a reason to use specifically "/tmp" I'll assume it's OK to change unless someone replies with a reason not to change it.

--
You received this message because you are subscribed to the Google Groups "cleverhans dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email to cleverhans-de...@googlegroups.com.
To post to this group, send email to cleverh...@googlegroups.com.
To view this discussion on the web visit https://groups.google.com/d/msgid/cleverhans-dev/CAG5SLK-Uax13ShLXkFnDAgUTOQjHmz1vFw9dOJUZymj%2B0rhHVg%40mail.gmail.com.
For more options, visit https://groups.google.com/d/optout.
Reply all
Reply to author
Forward
0 new messages