Skip to content

Remove exclusions for ONNX model tests that now pass.#14337

Merged
skottmckay merged 3 commits into
mainfrom
skottmckay/RemoveExclusionsForCSharpOnnxModelTestsThatPass
Jan 23, 2023
Merged

Remove exclusions for ONNX model tests that now pass.#14337
skottmckay merged 3 commits into
mainfrom
skottmckay/RemoveExclusionsForCSharpOnnxModelTestsThatPass

Conversation

@skottmckay
Copy link
Copy Markdown
Contributor

Description

Remove exclusions for ONNX model tests that now pass due to kernels being implemented.
Update ONNX update doc to point to correct location for tests.

Motivation and Context

Run as many tests as possible.

Update ONNX update doc to point to correct location for tests.
yuslepukhin
yuslepukhin previously approved these changes Jan 18, 2023
@@ -260,39 +260,28 @@ private static Dictionary<string, string> GetSkippedModels(DirectoryInfo modelsD
{ "test_max_float64", "node test error"},
{ "test_min_uint8", "node test error"},
{ "test_mod_mixed_sign_float64", "node test error"},
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good

Comment thread csharp/test/Microsoft.ML.OnnxRuntime.Tests.NetCoreApp/InferenceTest.netcore.cs Outdated
skottmckay and others added 2 commits January 19, 2023 15:56
…eTest.netcore.cs

Co-authored-by: Edward Chen <18449977+edgchen1@users.noreply.github.com>
@skottmckay skottmckay merged commit c252a7f into main Jan 23, 2023
@skottmckay skottmckay deleted the skottmckay/RemoveExclusionsForCSharpOnnxModelTestsThatPass branch January 23, 2023 22:04
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.

3 participants