-
Notifications
You must be signed in to change notification settings - Fork 6.8k
[v1.x] Add more onnx export unit tests, refactor onnxruntime tests. #19689
Conversation
Hey @josephevans , Thanks for submitting the PR
CI supported jobs: [unix-gpu, centos-gpu, edge, sanity, centos-cpu, clang, windows-gpu, unix-cpu, windows-cpu, miscellaneous, website] Note: |
@pytest.mark.parametrize('dtype', ['float32', 'float64', 'int32', 'int64']) | ||
@pytest.mark.parametrize('axis', [0, 1, -1]) | ||
def test_onnx_export_stack(tmp_path, dtype, axis): | ||
M = def_model('stack', axis=axis) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
shall we also try higher dimensional inputs?
aa2f201
to
87f6765
Compare
… all nodes created in export operators.
87f6765
to
7616a68
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
Description
Add unit tests for onnx export for the following operators:
Refactor onnxruntime tests to use tmp_path from pytest and parametrize model names so pytest splits up tests.