benchmarks icon indicating copy to clipboard operation
benchmarks copied to clipboard

Published 20 hours ago • verified publisher icon tensorflow

Make sure only chief worker can add global_step

Open anpark opened this issue 6 years ago • 1 comments

https://github.com/tensorflow/benchmarks/blob/2389369f6b5c9d3241676a728b450e47482966c0/scripts/tf_cnn_benchmarks/benchmark_cnn.py#L1579 why not change global_step update op for #199 ? Make sure only chief worker can add global_step like tf.train.SyncReplicasOptimizer

@alsrgv @reedwm

anpark avatar Jun 13 '18 09:06 anpark

That does sound like a better approach, but I didn't write the global_step code so I'm not sure. @zheng-xq is there a reason not to have only the chief update global_step?

reedwm avatar Jun 15 '18 18:06 reedwm