From 75f6861cb8d138ef7265bc7967ae665551a0bbaa Mon Sep 17 00:00:00 2001 From: Changming Sun Date: Thu, 30 Mar 2023 09:58:11 -0700 Subject: [PATCH] Skip DNNL's opset18 tests (#15275) ### Description DNNL EP doesn't support opset18 yet. So, let it skip such tests so that we could still test the other EPs. The models mentioned above are ONNX node tests that live in github.com/onnx/onnx --- onnxruntime/test/providers/cpu/model_tests.cc | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/onnxruntime/test/providers/cpu/model_tests.cc b/onnxruntime/test/providers/cpu/model_tests.cc index 1da491fe4f..658339d603 100644 --- a/onnxruntime/test/providers/cpu/model_tests.cc +++ b/onnxruntime/test/providers/cpu/model_tests.cc @@ -118,10 +118,12 @@ TEST_P(ModelTest, Run) { return; } - if (model_info->GetONNXOpSetVersion() == 10 && provider_name == "dnnl") { + if ((model_info->GetONNXOpSetVersion() == 10 || model_info->GetONNXOpSetVersion() >= 18) && provider_name == "dnnl") { // DNNL can run most of the model tests, but only part of // them is enabled here to save CI build time. - SkipTest(" dnnl doesn't support opset 10"); + std::ostringstream oss; + oss << " dnnl doesn't support opset " << model_info->GetONNXOpSetVersion(); + SkipTest(oss.str()); return; }