Skip to content

Commit

Permalink
[Caffe2] Skip some tests to unbreak CI (pytorch#7804)
Browse files Browse the repository at this point in the history
* Skip some tests to unbreak CI

* Pass the opset_version to run_node

* Remove the stale check_graph call, caffe2_net_to_onnx_model will invoke check_model
  • Loading branch information
houseroad authored and bddppq committed May 24, 2018
1 parent fdabc02 commit f9633b9
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 2 deletions.
3 changes: 2 additions & 1 deletion caffe2/python/onnx/backend.py
Original file line number Diff line number Diff line change
Expand Up @@ -207,7 +207,8 @@ def dummy_name(cls):
# opset_version if you don't want this to version.
@classmethod
def run_node(cls, node, inputs, device='CPU', opset_version=_known_opset_version, outputs_info=None):
super(Caffe2Backend, cls).run_node(node, inputs, device=device, outputs_info=outputs_info)
super(Caffe2Backend, cls).run_node(node, inputs, device=device,
outputs_info=outputs_info, opset_version=opset_version)

device_option = get_device_option(Device(device))
ws = Workspace()
Expand Down
1 change: 0 additions & 1 deletion caffe2/python/onnx/frontend.py
Original file line number Diff line number Diff line change
Expand Up @@ -265,7 +265,6 @@ def caffe2_net_to_onnx_graph(cls,
for name in predict_net.external_output
if name in all_output)

checker.check_graph(graph_def)
return graph_def

@classmethod
Expand Down
2 changes: 2 additions & 0 deletions caffe2/python/onnx/tests/onnx_backend_test.py
Original file line number Diff line number Diff line change
Expand Up @@ -35,6 +35,7 @@
'|test_simple_rnn.*' # Seems simple RNN case has some problem
'|test_gru.*' # Seems GRU case has some problem
'|test_upsample.*' # Upsample is redesigned in ONNX
'|test_prelu.*' # PRelu is not compliant with ONNX yet
'|test_operator_repeat.*' # Tile is not compliant with ONNX yet
'|test_.*pool_.*same.*' # Does not support pool same.
')')
Expand All @@ -43,6 +44,7 @@
backend_test.exclude('(test_cast_.*'
'|test_Conv1d_.*cuda'
'|test_Conv3d_groups_cuda'
'|test_rnn_seq_length'
'|test_operator_add.*_cuda'
'|test_operator_lstm_cuda'
'|test_operator_rnn.*_cuda)')
Expand Down

0 comments on commit f9633b9

Please sign in to comment.