From ef7d75baec1b0b3861acef52f1973bbe379ae881 Mon Sep 17 00:00:00 2001 From: Justin Lebar Date: Fri, 3 Mar 2017 17:03:22 -0800 Subject: Fix code that ignores tensorflow::Status. Change: 149170109 --- tensorflow/core/kernels/debug_ops.h | 12 +++++++++--- 1 file changed, 9 insertions(+), 3 deletions(-) (limited to 'tensorflow/core/kernels/debug_ops.h') diff --git a/tensorflow/core/kernels/debug_ops.h b/tensorflow/core/kernels/debug_ops.h index bfa163b292..bd42997732 100644 --- a/tensorflow/core/kernels/debug_ops.h +++ b/tensorflow/core/kernels/debug_ops.h @@ -94,9 +94,11 @@ class DebugIdentityOp : public OpKernel { void Compute(OpKernelContext* context) override { if (!debug_urls_.empty()) { + // TODO(b/32704451): Don't just ignore the ::tensorflow::Status object! DebugIO::PublishDebugTensor(tensor_name_, "DebugIdentity", context->input(0), - Env::Default()->NowMicros(), debug_urls_); + Env::Default()->NowMicros(), debug_urls_) + .IgnoreError(); } context->set_output(0, context->input(0)); @@ -144,8 +146,10 @@ class DebugNanCountOp : public OpKernel { output_tensor->vec()(0) = nan_count; if (!debug_urls_.empty()) { + // TODO(b/32704451): Don't just ignore the ::tensorflow::Status object! DebugIO::PublishDebugTensor(tensor_name_, "DebugNanCount", *output_tensor, - Env::Default()->NowMicros(), debug_urls_); + Env::Default()->NowMicros(), debug_urls_) + .IgnoreError(); } } @@ -256,9 +260,11 @@ class DebugNumericSummaryOp : public OpKernel { output_tensor->vec()(11) = variance; if (!debug_urls_.empty()) { + // TODO(b/32704451): Don't just ignore the ::tensorflow::Status object! DebugIO::PublishDebugTensor(tensor_name_, "DebugNumericSummary", *output_tensor, Env::Default()->NowMicros(), - debug_urls_); + debug_urls_) + .IgnoreError(); } } -- cgit v1.2.3