fix pylint

Author: zhijxu-MS

tests/test_optimizers.py

@@ -348,7 +348,8 @@ def test_duplicated_node_is_graph_output(self):
         )
 
         model_proto = helper.make_model(graph, producer_name="onnx-tests")
-        self.run_merge_duplicated_nodes_compare(["value1", "value2"], {"X": np.random.randn(5, 5).astype(np.float32)}, model_proto,
+        self.run_merge_duplicated_nodes_compare(["value1", "value2"],
+                                                {"X": np.random.randn(5, 5).astype(np.float32)}, model_proto,
                                                 op_type="Add", remaining_op_num=2)
 
     def test_duplicated_different_output_length(self):

tf2onnx/optimizer/identity_optimizer.py

@@ -14,7 +14,7 @@
 log = logging.getLogger("tf2onnx.optimizer.identity_optimizer")
 
 
-# pylint: disable=logging-not-lazy,unused-argument,missing-docstring,unused-variable
+# pylint: disable=logging-not-lazy,unused-argument,missing-docstring,unused-variable,arguments-differ
 
 
 class IdentityOptimizer(GraphOptimizerBase):

tf2onnx/optimizer/merge_duplicated_nodes_optimizer.py

@@ -15,12 +15,11 @@
 
 # pylint: disable=logging-not-lazy,unused-argument,missing-docstring
 
+_KeyToGroupNodes = namedtuple("key", "type input")
 
 class MergeDuplicatedNodesOptimizer(GraphOptimizerBase):
     """Remove duplicate nodes.
     """
-    _KeyToGroupNodes = namedtuple("key", "type input")
-
     def __init__(self, debug=False):
         super(MergeDuplicatedNodesOptimizer, self).__init__(debug)
         # optimizer should have name and log property
@@ -47,10 +46,11 @@ def _merge_duplicated_nodes(self, graph):
                 continue
             self._del_nodes_if_duplicated(nodes_group, graph)
 
-    def _group_nodes_by_type_inputs(self, graph):
+    @staticmethod
+    def _group_nodes_by_type_inputs(graph):
         res = defaultdict(list)
         for node in graph.get_nodes():
-            res[self._KeyToGroupNodes(node.type, tuple(node.input))].append(node)
+            res[_KeyToGroupNodes(node.type, tuple(node.input))].append(node)
         return res
 
     def _del_nodes_if_duplicated(self, nodes_group, graph):

tf2onnx/optimizer/transpose_optimizer.py

@@ -17,7 +17,7 @@
 log = logging.getLogger("tf2onnx.optimizer.transpose_optimizer")
 
 
-# pylint: disable=logging-not-lazy,unused-argument,missing-docstring
+# pylint: disable=logging-not-lazy,unused-argument,missing-docstring,abstract-method
 # FIXME:
 # pylint: disable=unused-variable