Fix initializer for CumSum. (apache#9)

gigiblender · Jan 31, 2023 · 2e9941b · 2e9941b
1 parent 957f266
commit 2e9941b
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 6 deletions.
diff --git a/python/tvm/relax/frontend/onnx_frontend.py b/python/tvm/relax/frontend/onnx_frontend.py
@@ -454,8 +454,7 @@ class CumSum(OnnxOpConverter):
     def _impl_v13(cls, bb, inputs, attr):
         assert getattr(attr, "reverse", 0) == 0, "reverse is not supported yet"
         if len(inputs) > 1:
-            # axis = int(infer_value(inputs[1], params).numpy())
-            axis = inputs[1]
+            axis = int(inputs[1].data.numpy())
         else:
             axis = None
         return bb.emit_te(

diff --git a/tests/python/relax/frontend/test_onnx_frontend.py b/tests/python/relax/frontend/test_onnx_frontend.py
@@ -37,7 +37,7 @@
 
 
 def generate_random_inputs(
-    model: ModelProto, inputs: Dict[str, np.array] = None
+    model: ModelProto, inputs: Optional[Dict[str, np.array]] = None
 ) -> Dict[str, np.array]:
     input_values = {}
     # Iterate through model inputs and extract their shape.
@@ -559,13 +559,13 @@ def test_cumsum():
         "cumsum_test",
         inputs=[
             helper.make_tensor_value_info("x", TensorProto.FLOAT, shape),
-            helper.make_tensor_value_info("axis", TensorProto.INT64, ()),
         ],
+        initializer=[helper.make_tensor("axis", TensorProto.INT64, (), [1])],
         outputs=[helper.make_tensor_value_info("y", TensorProto.FLOAT, shape)],
     )
 
     model = helper.make_model(graph, producer_name="cumsum_test")
-    check_correctness(model, {"axis": [1]})
+    check_correctness(model)
 
 
 if __name__ == "__main__":
@@ -585,6 +585,7 @@ def test_cumsum():
     test_conv()
     test_pow()
     test_erf()
+    test_cumsum()
 
     # TODO, still has issues
     # test_reshape()
@@ -594,4 +595,3 @@ def test_cumsum():
     test_transpose()
     test_unsqueeze()
     # test_shape()
-    # test_cumsum()  # need axis as int