skipped failing testcase (MathOpTest.Clip_Default_int64) #860
Conversation
There was a problem hiding this comment.
Pull Request Overview
This PR temporarily skips the MathOpTest.Clip_Default_int64 test case for the OpenVINO execution provider due to a known issue tracked as CVS-176933.
- Adds a conditional skip for the test when OpenVINO provider is available
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| } | ||
|
|
||
| if (DefaultOpenVINOExecutionProvider().get() != nullptr) { | ||
| GTEST_SKIP() << "Skipping because of the following error: error: Expected equality of these values: cur_expected[i] Which is: 11 cur_actual[i] Which is: 0"; |
There was a problem hiding this comment.
The error message contains a redundant 'error:' prefix at the beginning. Consider removing it for consistency with the existing skip message above (line 86).
| GTEST_SKIP() << "Skipping because of the following error: error: Expected equality of these values: cur_expected[i] Which is: 11 cur_actual[i] Which is: 0"; | |
| GTEST_SKIP() << "Skipping because of the following error: Expected equality of these values: cur_expected[i] Which is: 11 cur_actual[i] Which is: 0"; |
ankitm3k
left a comment
ankitm3k
left a comment
There was a problem hiding this comment.
This is how you disable the test @RajeevSekar , rebuild & test again
test.Run(OpTester::ExpectResult::kExpectSuccess, "", {kTensorrtExecutionProvider, kOpenVINOExecutionProvider});
preetha-intel
left a comment
preetha-intel
left a comment
There was a problem hiding this comment.
Disabling the test case until the op failure is fixed in OV.
Test Case is temporarily skipped until the issue is fixed in Openvino [CVS-176933].