[GitHub] spidyDev commented on a change in pull request #10472: [MXNET-106] [ONNX_MXNet] Change parameter names in imported model

2018-04-09 Thread GitBox
spidyDev commented on a change in pull request #10472: [MXNET-106] [ONNX_MXNet] 
Change parameter names in imported model
URL: https://github.com/apache/incubator-mxnet/pull/10472#discussion_r180247510
 
 

 ##
 File path: tests/python-pytest/onnx/onnx_test.py
 ##
 @@ -214,8 +211,8 @@ def test_bvlc_reference_caffenet():
 # run test for each test file
 for input_data, output_data in zip(inputs, outputs):
 # create module
-mod = mx.mod.Module(symbol=sym, data_names=['input_0'], 
context=mx.cpu(), label_names=None)
-mod.bind(for_training=False, data_shapes=[('input_0', 
input_data.shape)], label_shapes=None)
+mod = mx.mod.Module(symbol=sym, data_names=['data_0'], 
context=mx.cpu(), label_names=None)
 
 Review comment:
   same as above


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] spidyDev commented on a change in pull request #10472: [MXNET-106] [ONNX_MXNet] Change parameter names in imported model

2018-04-09 Thread GitBox
spidyDev commented on a change in pull request #10472: [MXNET-106] [ONNX_MXNet] 
Change parameter names in imported model
URL: https://github.com/apache/incubator-mxnet/pull/10472#discussion_r180247544
 
 

 ##
 File path: tests/python-pytest/onnx/onnx_test.py
 ##
 @@ -214,8 +211,8 @@ def test_bvlc_reference_caffenet():
 # run test for each test file
 for input_data, output_data in zip(inputs, outputs):
 # create module
-mod = mx.mod.Module(symbol=sym, data_names=['input_0'], 
context=mx.cpu(), label_names=None)
-mod.bind(for_training=False, data_shapes=[('input_0', 
input_data.shape)], label_shapes=None)
+mod = mx.mod.Module(symbol=sym, data_names=['data_0'], 
context=mx.cpu(), label_names=None)
+mod.bind(for_training=False, data_shapes=[('data_0', 
input_data.shape)], label_shapes=None)
 
 Review comment:
   same as above


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] spidyDev commented on a change in pull request #10472: [MXNET-106] [ONNX_MXNet] Change parameter names in imported model

2018-04-09 Thread GitBox
spidyDev commented on a change in pull request #10472: [MXNET-106] [ONNX_MXNet] 
Change parameter names in imported model
URL: https://github.com/apache/incubator-mxnet/pull/10472#discussion_r180247473
 
 

 ##
 File path: tests/python-pytest/onnx/onnx_test.py
 ##
 @@ -192,8 +189,8 @@ def test_bvlc_googlenet():
 # run test for each test file
 for input_data, output_data in zip(inputs, outputs):
 # create module
-mod = mx.mod.Module(symbol=sym, data_names=['input_0'], 
context=mx.cpu(), label_names=None)
-mod.bind(for_training=False, data_shapes=[('input_0', 
input_data.shape)], label_shapes=None)
+mod = mx.mod.Module(symbol=sym, data_names=['data_0'], 
context=mx.cpu(), label_names=None)
 
 Review comment:
   Can we fetch the data_names from sym ?


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] spidyDev commented on a change in pull request #10472: [MXNET-106] [ONNX_MXNet] Change parameter names in imported model

2018-04-09 Thread GitBox
spidyDev commented on a change in pull request #10472: [MXNET-106] [ONNX_MXNet] 
Change parameter names in imported model
URL: https://github.com/apache/incubator-mxnet/pull/10472#discussion_r180247574
 
 

 ##
 File path: tests/python-pytest/onnx/onnx_test.py
 ##
 @@ -236,8 +233,8 @@ def test_bvlc_rcnn_ilsvrc13():
 # run test for each test file
 for input_data, output_data in zip(inputs, outputs):
 # create module
-mod = mx.mod.Module(symbol=sym, data_names=['input_0'], 
context=mx.cpu(), label_names=None)
-mod.bind(for_training=False, data_shapes=[('input_0', 
input_data.shape)], label_shapes=None)
+mod = mx.mod.Module(symbol=sym, data_names=['data_0'], 
context=mx.cpu(), label_names=None)
 
 Review comment:
   same as above


This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services