Skip to content

update test case for test_convtranspose_autopad_same#24477

Merged
fs-eire merged 2 commits intomicrosoft:mainfrom
fs-eire:disable-failed-convtranspose-test-case
Apr 22, 2025
Merged

update test case for test_convtranspose_autopad_same#24477
fs-eire merged 2 commits intomicrosoft:mainfrom
fs-eire:disable-failed-convtranspose-test-case

Conversation

@fs-eire
Copy link
Contributor

@fs-eire fs-eire commented Apr 19, 2025

Description

It looks like the test case test_convtranspose_autopad_same in ONNX opset14(v1.9) is generated incorrectly: onnx/onnx#3440

The problem is already fixed in opset15+(v1.10).

So, this PR fixes 2 problems:

  • disable the test case for opset14 in onnxruntime-web tests ( still run the same test for opset15 and opset17 )
  • re-enable the test case by removing if from "current_failing_tests" in onnxruntime\test\testdata\onnx_backend_test_series_filters.jsonc (which should have been done when upgrading to onnx v1.10)

Motivation and Context

@fs-eire fs-eire force-pushed the disable-failed-convtranspose-test-case branch from 53025ea to f760b1a Compare April 19, 2025 19:38
@fs-eire fs-eire marked this pull request as ready for review April 20, 2025 00:59
@fs-eire fs-eire merged commit 81bb013 into microsoft:main Apr 22, 2025
71 of 84 checks passed
intbf pushed a commit to intbf/onnxruntime that referenced this pull request Apr 25, 2025
### Description
<!-- Describe your changes. -->

It looks like the test case `test_convtranspose_autopad_same` in ONNX
opset14(v1.9) is generated incorrectly:
onnx/onnx#3440

The problem is already fixed in opset15+(v1.10).

So, this PR fixes 2 problems:
- disable the test case for opset14 in onnxruntime-web tests ( still run
the same test for opset15 and opset17 )
- re-enable the test case by removing if from "current_failing_tests" in
onnxruntime\test\testdata\onnx_backend_test_series_filters.jsonc (which
should have been done when upgrading to onnx v1.10)

### Motivation and Context
<!-- - Why is this change required? What problem does it solve?
- If it fixes an open issue, please link to the issue here. -->

Signed-off-by: bfilipek <bartlomiej.filipek@intel.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants