[moco_tf] Fix BiasAdd test indent todo (#4120)
author박세희/On-Device Lab(SR)/Principal Engineer/삼성전자 <saehie.park@samsung.com>
Mon, 8 Jul 2019 00:11:36 +0000 (09:11 +0900)
committerGitHub Enterprise <noreply-CODE@samsung.com>
Mon, 8 Jul 2019 00:11:36 +0000 (09:11 +0900)
This will fix indentation of BiasAdd test from previous change

Signed-off-by: SaeHie Park <saehie.park@samsung.com>
contrib/moco-tf/src/Op/BiasAdd.test.cpp

index 93c230a..5fdf4e1 100644 (file)
@@ -131,44 +131,41 @@ TEST(TensorFlowImport, bias_add_01)
   }
   else
   {
-    // TODO fix indentation and remove clang switch
-    // clang-format off
-  // test 1.
-  // loco node : 1. ConstGen ------------------+-- 4. BiasAdd<Domain::Tensor> -- 5. Push
-  //             2. ConstGen - 3. BiasEncode -/
-  loco::Graph::NodeContext *loco_nodes = graph->nodes();
-
-  loco::Graph::InputContext *loco_inputs = graph->inputs();
-  ASSERT_EQ(loco_inputs->size(), 0);
-  ASSERT_EQ(loco_nodes->size(), 5);
-
-  int idx = 0;
-
-  loco::ConstGen *value = dynamic_cast<loco::ConstGen *>(loco_nodes->at(idx++));
-  loco::ConstGen *bias = dynamic_cast<loco::ConstGen *>(loco_nodes->at(idx++));
-  loco::BiasEncode *enc = dynamic_cast<loco::BiasEncode *>(loco_nodes->at(idx++));
-  loco::BiasAdd<loco::Domain::Tensor> *bias_add =
-      dynamic_cast<loco::BiasAdd<loco::Domain::Tensor> *>(loco_nodes->at(idx++));
-  loco::Push *push = dynamic_cast<loco::Push *>(loco_nodes->at(idx++));
-
-  ASSERT_NE(value, nullptr);
-  ASSERT_NE(bias, nullptr);
-  ASSERT_NE(enc, nullptr);
-  ASSERT_NE(bias_add, nullptr);
-  ASSERT_NE(push, nullptr);
-
-  // check their connection is all OK
-  ASSERT_TRUE(enc->input() == bias);
-  ASSERT_TRUE(bias_add->bias() == enc);
-  ASSERT_TRUE(bias_add->value() == value);
-  ASSERT_TRUE(push->from() == bias_add);
-
-  // test 2.
-  // attrs inside BiasAdd
-
-  // axis
-  ASSERT_EQ(bias_add->axis(), 3); // NHWC
-    // clang-format on
+    // test 1.
+    // loco node : 1. ConstGen ------------------+-- 4. BiasAdd<Domain::Tensor> -- 5. Push
+    //             2. ConstGen - 3. BiasEncode -/
+    loco::Graph::NodeContext *loco_nodes = graph->nodes();
+
+    loco::Graph::InputContext *loco_inputs = graph->inputs();
+    ASSERT_EQ(loco_inputs->size(), 0);
+    ASSERT_EQ(loco_nodes->size(), 5);
+
+    int idx = 0;
+
+    loco::ConstGen *value = dynamic_cast<loco::ConstGen *>(loco_nodes->at(idx++));
+    loco::ConstGen *bias = dynamic_cast<loco::ConstGen *>(loco_nodes->at(idx++));
+    loco::BiasEncode *enc = dynamic_cast<loco::BiasEncode *>(loco_nodes->at(idx++));
+    loco::BiasAdd<loco::Domain::Tensor> *bias_add =
+        dynamic_cast<loco::BiasAdd<loco::Domain::Tensor> *>(loco_nodes->at(idx++));
+    loco::Push *push = dynamic_cast<loco::Push *>(loco_nodes->at(idx++));
+
+    ASSERT_NE(value, nullptr);
+    ASSERT_NE(bias, nullptr);
+    ASSERT_NE(enc, nullptr);
+    ASSERT_NE(bias_add, nullptr);
+    ASSERT_NE(push, nullptr);
+
+    // check their connection is all OK
+    ASSERT_TRUE(enc->input() == bias);
+    ASSERT_TRUE(bias_add->bias() == enc);
+    ASSERT_TRUE(bias_add->value() == value);
+    ASSERT_TRUE(push->from() == bias_add);
+
+    // test 2.
+    // attrs inside BiasAdd
+
+    // axis
+    ASSERT_EQ(bias_add->axis(), 3); // NHWC
   }
 }
 
@@ -259,16 +256,12 @@ TEST(TensorFlowImport, bias_add_NCHW_axis)
   }
   else
   {
-    // TODO fix indentation and remove clang switch
-    // clang-format off
-
-  // testing axis value of biasAdd
-  loco::Graph::NodeContext *loco_nodes = graph->nodes();
-  loco::BiasAdd<loco::Domain::Tensor> *bias_add =
-      dynamic_cast<loco::BiasAdd<loco::Domain::Tensor> *>(loco_nodes->at(3));
-  ASSERT_NE(bias_add, nullptr);
+    // testing axis value of biasAdd
+    loco::Graph::NodeContext *loco_nodes = graph->nodes();
+    loco::BiasAdd<loco::Domain::Tensor> *bias_add =
+        dynamic_cast<loco::BiasAdd<loco::Domain::Tensor> *>(loco_nodes->at(3));
+    ASSERT_NE(bias_add, nullptr);
 
-  ASSERT_EQ(bias_add->axis(), 1); // NCHW
-    // clang-format on
+    ASSERT_EQ(bias_add->axis(), 1); // NCHW
   }
 }