Commit 23975377 authored by Igor Saprykin's avatar Igor Saprykin Committed by TensorFlower Gardener
Browse files

De-flakify and re-enable tests in replicate_model_fn_test.py.

"Reduce metric variables" operation is a single operation across all metric variables, which means it is across all eval metrics.  Previously, an update op for every eval metric was conditioned on a copy of overall "reduce metric variables" op.  The latter was meant to be idempotent and thus the end result was supposed to be correct.

However, "reduce metric variables" op consists of a number of variable assignments and thus is not atomic.  If execution of two "reduce metric variables" ops interleaves, then the end result might come out to be incorrect.  This caused flakiness in replicate_model_fn_test.py.  To fix the problem, there is now a single copy of the "reduce metric variables" and every eval metric is associated with that single instance.

PiperOrigin-RevId: 175232016
parent 47f8f08f
Loading
Loading
Loading
Loading
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please to comment