Skip to content

Commit

Permalink
lint
Browse files Browse the repository at this point in the history
Signed-off-by: Liqun Fu <[email protected]>
  • Loading branch information
liqunfu committed Nov 26, 2023
1 parent 1230dd3 commit 4860aa7
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 7 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -37,4 +37,4 @@ class ReduceKernelBase {
bool select_last_index_;
};
} // namespace onnxruntime
#endif // !CORE_PROVIDERS_CPU_REDUCTION_KERNEL_BASE_H
#endif // !CORE_PROVIDERS_CPU_REDUCTION_KERNEL_BASE_H

Check warning on line 40 in onnxruntime/core/providers/cpu/reduction/reduction_kernel_base.h

View workflow job for this annotation

GitHub Actions / cpplint

[cpplint] onnxruntime/core/providers/cpu/reduction/reduction_kernel_base.h#L40

#endif line should be "#endif // ONNXRUNTIME_CORE_PROVIDERS_CPU_REDUCTION_REDUCTION_KERNEL_BASE_H_" [build/header_guard] [5]
Raw output
onnxruntime/core/providers/cpu/reduction/reduction_kernel_base.h:40:  #endif line should be "#endif  // ONNXRUNTIME_CORE_PROVIDERS_CPU_REDUCTION_REDUCTION_KERNEL_BASE_H_"  [build/header_guard] [5]
12 changes: 6 additions & 6 deletions onnxruntime/test/providers/cpu/reduction/reduction_ops_test.cc
Original file line number Diff line number Diff line change
Expand Up @@ -924,7 +924,7 @@ TEST(ReductionOpTest, ReduceMax_default_axes_do_not_keep_dims) {
55.0f, 1.0f,
60.0f, 2.0f});
test.AddOutput<float>("reduced", {}, {60.0f});
test.Run(OpTester::ExpectResult::kExpectSuccess, "", {kTensorrtExecutionProvider});// TensorRT: full reduce without keepDimensions is not supported with explicit batch //TensorRT: axis must be 0
test.Run(OpTester::ExpectResult::kExpectSuccess, "", {kTensorrtExecutionProvider}); // TensorRT: full reduce without keepDimensions is not supported with explicit batch //TensorRT: axis must be 0

Check warning on line 927 in onnxruntime/test/providers/cpu/reduction/reduction_ops_test.cc

View workflow job for this annotation

GitHub Actions / cpplint

[cpplint] onnxruntime/test/providers/cpu/reduction/reduction_ops_test.cc#L927

Lines should be <= 120 characters long [whitespace/line_length] [2]
Raw output
onnxruntime/test/providers/cpu/reduction/reduction_ops_test.cc:927:  Lines should be <= 120 characters long  [whitespace/line_length] [2]
}

#ifndef USE_MIGRAPHX
Expand Down Expand Up @@ -1200,7 +1200,7 @@ TEST(ReductionOpTest, test_bool_ReduceMin_19) {
kOpenVINOExecutionProvider,
});
}
#endif // USE_MIGRAPHX
#endif // USE_MIGRAPHX

TEST(ReductionOpTest, ReduceMax_do_not_keepdims) {
OpTester test("ReduceMax");
Expand Down Expand Up @@ -1361,7 +1361,7 @@ TEST(ReductionOpTest, ReduceMax_int32) {
#if defined(OPENVINO_CONFIG_GPU_FP32) || defined(OPENVINO_CONFIG_GPU_FP16)
test.Run(OpTester::ExpectResult::kExpectSuccess, "", {kTensorrtExecutionProvider, kOpenVINOExecutionProvider}); // OpenVINO: Disabled temporarily
#else
test.Run(OpTester::ExpectResult::kExpectSuccess, "", {kTensorrtExecutionProvider}); // TensorRT: axis must be 0
test.Run(OpTester::ExpectResult::kExpectSuccess, "", {kTensorrtExecutionProvider}); // TensorRT: axis must be 0

Check warning on line 1364 in onnxruntime/test/providers/cpu/reduction/reduction_ops_test.cc

View workflow job for this annotation

GitHub Actions / cpplint

[cpplint] onnxruntime/test/providers/cpu/reduction/reduction_ops_test.cc#L1364

Lines should be <= 120 characters long [whitespace/line_length] [2]
Raw output
onnxruntime/test/providers/cpu/reduction/reduction_ops_test.cc:1364:  Lines should be <= 120 characters long  [whitespace/line_length] [2]
#endif
}

Expand All @@ -1382,7 +1382,7 @@ TEST(ReductionOpTest, ReduceMax_int64) {
#if defined(OPENVINO_CONFIG_GPU_FP32) || defined(OPENVINO_CONFIG_GPU_FP16)
test.Run(OpTester::ExpectResult::kExpectSuccess, "", {kTensorrtExecutionProvider, kOpenVINOExecutionProvider}); // OpenVINO: Disabled temporarily
#else
test.Run(OpTester::ExpectResult::kExpectSuccess, "", {kTensorrtExecutionProvider}); // TensorRT: axis must be 0
test.Run(OpTester::ExpectResult::kExpectSuccess, "", {kTensorrtExecutionProvider}); // TensorRT: axis must be 0

Check warning on line 1385 in onnxruntime/test/providers/cpu/reduction/reduction_ops_test.cc

View workflow job for this annotation

GitHub Actions / cpplint

[cpplint] onnxruntime/test/providers/cpu/reduction/reduction_ops_test.cc#L1385

Lines should be <= 120 characters long [whitespace/line_length] [2]
Raw output
onnxruntime/test/providers/cpu/reduction/reduction_ops_test.cc:1385:  Lines should be <= 120 characters long  [whitespace/line_length] [2]
#endif
}

Expand Down Expand Up @@ -3557,7 +3557,7 @@ TEST(ReductionOpTest, ReduceDimWithZero1) {
test.AddOutput<float>("reduced", {1, 1, 1}, {0.0f});
run(test);
}
#endif // USE_MIGRAPHX
#endif // USE_MIGRAPHX

TEST(ReductionOpTest, OptimizeShapeForFastReduce_ReduceDimWithZero2) {
FastReduceKind fast_kind;
Expand Down Expand Up @@ -5918,6 +5918,6 @@ TEST(ReductionOpTest, empty_set_ReduceSumSquare_13) {

test_empty_set("ReduceSumSquare", 13, false, 0.0f);
}
#endif // USE_MIGRAPHX
#endif // USE_MIGRAPHX
} // namespace test
} // namespace onnxruntime

0 comments on commit 4860aa7

Please sign in to comment.