Fix keras metric.result_state when the metric variables are sharded variable.

ckkuang · tensorflower-gardener · commit 434d29d27763 · 2021-06-24T11:02:04.000-07:00
PiperOrigin-RevId: 381292911
diff --git a/keras/distribute/sharded_variable_test.py b/keras/distribute/sharded_variable_test.py
@@ -108,22 +108,28 @@ def __init__(self):
 
   def test_keras_metrics(self):
     with self.strategy.scope():
+      fp = keras.metrics.FalsePositives(thresholds=[0.2, 0.5, 0.7, 0.8])
       auc = keras.metrics.AUC(num_thresholds=10)
 
     @tf.function
     def update():
+      fp.update_state([0., 1., 0., 0.], [0., 0., 0.3, 0.9])
       auc.update_state([0, 0, 1, 1], [0, 0.5, 0.3, 0.9])
 
     @tf.function
     def reset():
+      fp.reset_state()
       auc.reset_state()
 
     update()
     self.assertEqual(auc.result(), 0.75)
+    self.assertAllEqual(fp.result(), [2., 1., 1., 1.])
     reset()
     self.assertEqual(auc.result(), 0.0)
+    self.assertAllEqual(fp.result(), [0., 0., 0., 0.])
 
     self.assertTrue(hasattr(auc.true_positives, 'variables'))
+    self.assertTrue(hasattr(fp.accumulator, 'variables'))
 
   def test_saved_model(self):
 
diff --git a/keras/metrics.py b/keras/metrics.py
@@ -1038,9 +1038,9 @@ def result(self):
     return tf.convert_to_tensor(result)
 
   def reset_state(self):
-    num_thresholds = len(to_list(self.thresholds))
-    backend.batch_set_value(
-        [(v, np.zeros((num_thresholds,))) for v in self.variables])
+    backend.batch_set_value([
+        (v, np.zeros(v.shape.as_list())) for v in self.variables
+    ])
 
   def get_config(self):
     config = {'thresholds': self.init_thresholds}
@@ -3175,8 +3175,9 @@ def result(self):
 
   def reset_state(self):
     if self._built:
-      backend.batch_set_value(
-          [(v, np.zeros(self._shape.as_list())) for v in self.variables])
+      backend.batch_set_value([
+          (v, np.zeros(v.shape.as_list())) for v in self.variables
+      ])
 
 
 @keras_export('keras.metrics.BinaryCrossentropy')
