From 29434d3d993f027f7c9be3ce7b749827221591af Mon Sep 17 00:00:00 2001 From: Pranov Duggasani Date: Wed, 17 Jul 2019 14:32:47 -0700 Subject: [PATCH 1/2] Added continue statement to as that is what is in the original python file and got rid off cast error --- src/TensorFlowNET.Core/Framework/meta_graph.py.cs | 2 +- src/TensorFlowNET.Core/Train/Optimizer.cs | 4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/src/TensorFlowNET.Core/Framework/meta_graph.py.cs b/src/TensorFlowNET.Core/Framework/meta_graph.py.cs index 9e40bb46..58051ae6 100644 --- a/src/TensorFlowNET.Core/Framework/meta_graph.py.cs +++ b/src/TensorFlowNET.Core/Framework/meta_graph.py.cs @@ -134,7 +134,7 @@ namespace Tensorflow } break; default: - throw new NotImplementedException("import_scoped_meta_graph_with_return_elements"); + continue; } } } diff --git a/src/TensorFlowNET.Core/Train/Optimizer.cs b/src/TensorFlowNET.Core/Train/Optimizer.cs index 6c13ac20..ef042e8f 100644 --- a/src/TensorFlowNET.Core/Train/Optimizer.cs +++ b/src/TensorFlowNET.Core/Train/Optimizer.cs @@ -199,8 +199,8 @@ namespace Tensorflow if (!tf.context.executing_eagerly()) { - var train_op = ops.get_collection_ref(ops.GraphKeys.TRAIN_OP) as List; - if (!train_op.Contains(apply_updates)) + var train_op = ops.get_collection_ref(ops.GraphKeys.TRAIN_OP) as List; + if (train_op != null && train_op.Contains(apply_updates)) train_op.Add(apply_updates); } From edd5f51f25c95e111ae21d208df89e0728b37442 Mon Sep 17 00:00:00 2001 From: Pranov Duggasani Date: Wed, 17 Jul 2019 14:57:35 -0700 Subject: [PATCH 2/2] Added Console.Write instead of ignoring exception --- src/TensorFlowNET.Core/Framework/meta_graph.py.cs | 1 + 1 file changed, 1 insertion(+) diff --git a/src/TensorFlowNET.Core/Framework/meta_graph.py.cs b/src/TensorFlowNET.Core/Framework/meta_graph.py.cs index 58051ae6..447b7835 100644 --- a/src/TensorFlowNET.Core/Framework/meta_graph.py.cs +++ b/src/TensorFlowNET.Core/Framework/meta_graph.py.cs @@ -134,6 +134,7 @@ namespace Tensorflow } break; default: + Console.WriteLine("import_scoped_meta_graph_with_return_elements"); continue; } }