Disable tensorflow profiler by default

bargavj / EvaluatingDPML

This project's goal is to evaluate the privacy leakage of differentially private machine learning models.

MIT License

129 stars 49 forks source link

Disable tensorflow profiler by default #8

Closed jonahweissman closed 5 years ago

jonahweissman commented 5 years ago

Given that several people have encountered problems with the tensorflow ProfilerHook that I don't know how to fix, I think it makes the most sense to leave the functionality around, but disable it by default.

If you think it's a good idea, I could add a comment warning about the errors that some users get.

bargavj commented 5 years ago

Its a good idea to set the logging to FALSE by default.