-
Notifications
You must be signed in to change notification settings - Fork 3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
upgrade protobuf to 3.20.2 and onnx to 1.13 #14279
Changes from 29 commits
79c6d37
a926c22
488b7f4
6e488ad
79e9f3b
b5a7f28
d73d925
3b45872
56f7d8d
aa0ea4a
a92be8e
0dd4358
0c1f723
0fb4e5d
42c2f85
c47f71f
bb09daa
a5d45bf
a05dc54
1b845e9
a7e1ad3
9c503d4
9f42643
b07d385
544524a
1845b4e
9043929
48270e4
39efc46
a14f672
96843a4
1d21267
70d671b
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -763,7 +763,7 @@ if (MSVC) | |
# The warning means the type of two integral values around a binary operator is narrow than their result. | ||
# If we promote the two input values first, it could be more tolerant to integer overflow. | ||
# However, this is test code. We are less concerned. | ||
target_compile_options(onnxruntime_test_all PRIVATE "/wd26451") | ||
target_compile_options(onnxruntime_test_all PRIVATE "/wd26451" "/wd4244") | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 4244 is a critical warning that we should not disable. It is required by BinSkim. However, this is test code. We don't ship it. So it is kind of ok. |
||
else() | ||
target_compile_options(onnxruntime_test_all PRIVATE "-Wno-parentheses") | ||
endif() | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -345,6 +345,83 @@ | |
"^test_training_dropout_mask", // Runs but there's accuracy mismatch | ||
"^test_training_dropout_default" // Runs but there's accuracy mismatch | ||
], | ||
"current_failing_tests_OPENVINO_opset18": [ | ||
// pending opset 18 support, RUNTIME_EXCEPTION : Encountered unknown exception in Initialize() | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Those tests are introduced in the Latest ONNX(1.13). They are not running in the old OpenVINO pipeline. ONNX 1.12 used the protobuf 2.18.3, which doesn't meet the compliance. |
||
"^test_center_crop_pad_crop_axes_chw", | ||
"^test_center_crop_pad_crop_axes_chw_expanded", | ||
"^test_center_crop_pad_crop_axes_hwc", | ||
"^test_center_crop_pad_crop_axes_hwc_expanded", | ||
"^test_group_normalization_epsilon", | ||
"^test_group_normalization_example", | ||
"^test_layer_normalization_2d_axis1_expanded_ver18", | ||
"^test_layer_normalization_2d_axis_negative_1_expanded_ver18", | ||
"^test_layer_normalization_3d_axis1_epsilon_expanded_ver18", | ||
"^test_layer_normalization_3d_axis2_epsilon_expanded_ver18", | ||
"^test_layer_normalization_3d_axis_negative_1_epsilon_expanded_ver18", | ||
"^test_layer_normalization_3d_axis_negative_2_epsilon_expanded_ver18", | ||
"^test_layer_normalization_4d_axis1_expanded_ver18", | ||
"^test_layer_normalization_4d_axis2_expanded_ver18", | ||
"^test_layer_normalization_4d_axis3_expanded_ver18", | ||
"^test_layer_normalization_4d_axis_negative_1_expanded_ver18", | ||
"^test_layer_normalization_4d_axis_negative_2_expanded_ver18", | ||
"^test_layer_normalization_4d_axis_negative_3_expanded_ver18", | ||
"^test_layer_normalization_default_axis_expanded_ver18", | ||
"^test_reduce_l1_do_not_keepdims_example_expanded", | ||
"^test_reduce_l1_do_not_keepdims_random_expanded", | ||
"^test_reduce_log_sum_asc_axes_expanded", | ||
"^test_reduce_log_sum_desc_axes_expanded", | ||
"^test_reduce_sum_square_do_not_keepdims_example_expanded", | ||
"^test_reduce_sum_square_do_not_keepdims_random_expanded", | ||
"^test_scatter_elements_with_reduction_max", | ||
"^test_scatter_elements_with_reduction_min", | ||
"^test_scatternd_max", | ||
"^test_scatternd_min", | ||
// pending opset 18 support, test failures | ||
"^test_group_normalization_epsilon_expanded", | ||
"^test_group_normalization_example_expanded", | ||
"^test_logsoftmax_large_number_expanded_ver18", | ||
"^test_mvn_expanded_ver18_cpu", | ||
"^test_reduce_l1_do_not_keepdims_example", | ||
"^test_reduce_l1_do_not_keepdims_random", | ||
"^test_reduce_l1_keep_dims_example", | ||
"^test_reduce_l1_keep_dims_random", | ||
"^test_reduce_l1_negative_axes_keep_dims_example", | ||
"^test_reduce_l1_negative_axes_keep_dims_random", | ||
"^test_reduce_l2_do_not_keepdims_example", | ||
"^test_reduce_l2_do_not_keepdims_random", | ||
"^test_reduce_l2_keep_dims_example", | ||
"^test_reduce_l2_keep_dims_random", | ||
"^test_reduce_l2_negative_axes_keep_dims_example", | ||
"^test_reduce_l2_negative_axes_keep_dims_random", | ||
"^test_reduce_log_sum_asc_axes", | ||
"^test_reduce_log_sum_desc_axes", | ||
"^test_reduce_log_sum_negative_axes", | ||
"^test_reduce_max_do_not_keepdims_example", | ||
"^test_reduce_max_do_not_keepdims_random", | ||
"^test_reduce_max_keepdims_example", | ||
"^test_reduce_max_keepdims_random", | ||
"^test_reduce_max_negative_axes_keepdims_example", | ||
"^test_reduce_max_negative_axes_keepdims_random", | ||
"^test_reduce_mean_do_not_keepdims_example", | ||
"^test_reduce_mean_do_not_keepdims_random", | ||
"^test_reduce_mean_keepdims_example", | ||
"^test_reduce_mean_keepdims_random", | ||
"^test_reduce_mean_negative_axes_keepdims_example", | ||
"^test_reduce_mean_negative_axes_keepdims_random", | ||
"^test_reduce_prod_do_not_keepdims_example", | ||
"^test_reduce_prod_do_not_keepdims_random", | ||
"^test_reduce_prod_keepdims_example", | ||
"^test_reduce_prod_keepdims_random", | ||
"^test_reduce_prod_negative_axes_keepdims_example", | ||
"^test_reduce_prod_negative_axes_keepdims_random", | ||
"^test_reduce_sum_square_do_not_keepdims_example", | ||
"^test_reduce_sum_square_do_not_keepdims_random", | ||
"^test_reduce_sum_square_keepdims_example", | ||
"^test_reduce_sum_square_keepdims_random", | ||
"^test_reduce_sum_square_negative_axes_keepdims_example", | ||
"^test_reduce_sum_square_negative_axes_keepdims_random", | ||
"^test_softmax_large_number_expanded_ver18" | ||
], | ||
"current_failing_tests_MIGRAPHX": [ | ||
"^test_constant_pad_cpu", | ||
"^test_round_cpu", | ||
|
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.
I think it is better to only disable the warning for protobuf related code. If you add the compile flag here, our code will start to have such warnings(though they are suppressed).
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.
There're 2 types of errors caused by this
_unused-parameter
. More exception info is in the description of this PR.onnx/onnx_pb.h
.So far, there're 44 c++ and 4 Objective-c files include the header.
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.
So, I've patched ONNX directly. @snnn