Add explicit casts for a few ops that assumed it could implicitly cast to/from

float and int for all types T. (The assumption doesn't hold for Eigen::half.)

These were all ops I could find in CPU implementation; probably, some remain
for GPU (and some other problems remain before we can turn on Eigen::half for
all of them).
Change: 118464463

1 files changed, 2 insertions, 2 deletions

diff --git a/tensorflow/core/kernels/summary_op.cc b/tensorflow/core/kernels/summary_op.cc
index 9fd5a4a6fc..16e5b0a0ff 100644
--- a/tensorflow/core/kernels/summary_op.cc
+++ b/tensorflow/core/kernels/summary_op.cc
@@ -52,7 +52,7 @@ class SummaryScalarOp : public OpKernel {
     for (int i = 0; i < Ttags.size(); i++) {
       Summary::Value* v = s.add_value();
       v->set_tag(Ttags(i));
-      v->set_simple_value(Tvalues(i));
+      v->set_simple_value(T(Tvalues(i)));
     }
 
     Tensor* summary_tensor = nullptr;
@@ -92,7 +92,7 @@ class SummaryHistoOp : public OpKernel {
             errors::OutOfRange("Nan in summary histogram for: ", name()));
         break;
       }
-      histo.Add(v);
+      histo.Add(static_cast<double>(v));
     }
 
     Summary s;