Onnx: test data for shrink_test is of incorrect shape

Created on 19 Feb 2019  路  4Comments  路  Source: onnx/onnx

Both onnx/backend/test/data/simple/test_shrink/test_data_set_0/input_0.pb and onnx/backend/test/data/simple/test_shrink/test_data_set_0/output_0.pb have shape [5] whereas the graph input and output in onnx/backend/test/data/simple/test_shrink/model.onnx has shape [1,5].

In test_shrink_soft and test_shrink_hard, the test input data, test output data, the graph input defined in the model, the graph output defined in the model all have consistent shape of [5].

Tagging @zrphercule as fyi

All 4 comments

Thanks for raising this, I will look into it once I get back to office tomorrow.

Hi @hariharans29 , I have checked this operator. When dealing with model test and node test, we use totally different test cases and data, that is why the input/output of a node test could be very different from the input/output of a model test. So long as a node test/model test is consistent itself, then it is fine.
After checking the code, it seems we need to modify the test data of model test. Sounds decent?

Again, thanks for your issue.

Hi @zrphercule

1) Agreed. Sorry for the confusion - I did not mean to say the test cases between model test and node test are required to be the same. So long as they are consistent within the individual test itself, it is fine.

2) Agreed. You can modify the shape of the test data to make it consistent with graph input/output.

Thanks for the quick turnaround.

Closed due to the merging of #1825

Was this page helpful?
0 / 5 - 0 ratings