Skip to content

update

update #23105

Triggered via push November 4, 2023 04:07
Status Success
Total duration 23s
Artifacts
Validation
15s
Validation
Fit to window
Zoom out
Zoom in

Annotations

143 warnings
[cpplint] include/onnxruntime/core/session/onnxruntime_c_api.h#L4515: include/onnxruntime/core/session/onnxruntime_c_api.h#L4515
Line ends in whitespace. Consider deleting these extra spaces. [whitespace/end_of_line] [4]
[cpplint] onnxruntime/core/providers/tensorrt/tensorrt_execution_provider.cc#L711: onnxruntime/core/providers/tensorrt/tensorrt_execution_provider.cc#L711
Redundant blank line at the start of a code block should be deleted. [whitespace/blank_line] [2]
[cpplint] onnxruntime/core/providers/tensorrt/tensorrt_execution_provider.cc#L896: onnxruntime/core/providers/tensorrt/tensorrt_execution_provider.cc#L896
Line ends in whitespace. Consider deleting these extra spaces. [whitespace/end_of_line] [4]
[cpplint] onnxruntime/core/providers/tensorrt/tensorrt_execution_provider.cc#L896: onnxruntime/core/providers/tensorrt/tensorrt_execution_provider.cc#L896
Redundant blank line at the start of a code block should be deleted. [whitespace/blank_line] [2]
[cpplint] onnxruntime/core/providers/tensorrt/tensorrt_execution_provider.cc#L2199: onnxruntime/core/providers/tensorrt/tensorrt_execution_provider.cc#L2199
Line ends in whitespace. Consider deleting these extra spaces. [whitespace/end_of_line] [4]
[cpplint] onnxruntime/core/providers/tensorrt/tensorrt_execution_provider.cc#L2399: onnxruntime/core/providers/tensorrt/tensorrt_execution_provider.cc#L2399
Line ends in whitespace. Consider deleting these extra spaces. [whitespace/end_of_line] [4]
[cpplint] onnxruntime/core/providers/tensorrt/tensorrt_execution_provider.cc#L2411: onnxruntime/core/providers/tensorrt/tensorrt_execution_provider.cc#L2411
Using C-style cast. Use reinterpret_cast<char*>(...) instead [readability/casting] [4]
[cpplint] onnxruntime/core/providers/tensorrt/tensorrt_execution_provider.cc#L2421: onnxruntime/core/providers/tensorrt/tensorrt_execution_provider.cc#L2421
Line ends in whitespace. Consider deleting these extra spaces. [whitespace/end_of_line] [4]
[cpplint] onnxruntime/core/providers/tensorrt/tensorrt_execution_provider.cc#L2438: onnxruntime/core/providers/tensorrt/tensorrt_execution_provider.cc#L2438
Line ends in whitespace. Consider deleting these extra spaces. [whitespace/end_of_line] [4]
[cpplint] onnxruntime/core/providers/tensorrt/tensorrt_execution_provider.cc#L2471: onnxruntime/core/providers/tensorrt/tensorrt_execution_provider.cc#L2471
Missing username in TODO; it should look like "// TODO(my_username): Stuff." [readability/todo] [2]
[cpplint] onnxruntime/core/providers/tensorrt/tensorrt_execution_provider.cc#L2678: onnxruntime/core/providers/tensorrt/tensorrt_execution_provider.cc#L2678
Line ends in whitespace. Consider deleting these extra spaces. [whitespace/end_of_line] [4]
[cpplint] onnxruntime/core/providers/tensorrt/tensorrt_execution_provider.cc#L2856: onnxruntime/core/providers/tensorrt/tensorrt_execution_provider.cc#L2856
If an else has a brace on one side, it should have it on both [readability/braces] [5]
[cpplint] onnxruntime/core/providers/tensorrt/tensorrt_execution_provider.cc#L3005: onnxruntime/core/providers/tensorrt/tensorrt_execution_provider.cc#L3005
Using C-style cast. Use reinterpret_cast<char*>(...) instead [readability/casting] [4]
[cpplint] onnxruntime/core/providers/tensorrt/tensorrt_execution_provider.cc#L3168: onnxruntime/core/providers/tensorrt/tensorrt_execution_provider.cc#L3168
Missing username in TODO; it should look like "// TODO(my_username): Stuff." [readability/todo] [2]
[cpplint] onnxruntime/core/providers/tensorrt/tensorrt_execution_provider.cc#L3264: onnxruntime/core/providers/tensorrt/tensorrt_execution_provider.cc#L3264
Using C-style cast. Use reinterpret_cast<char*>(...) instead [readability/casting] [4]
[cpplint] onnxruntime/core/providers/tensorrt/tensorrt_execution_provider.cc#L3654: onnxruntime/core/providers/tensorrt/tensorrt_execution_provider.cc#L3654
Small and focused functions are preferred: TensorrtExecutionProvider::CreateNodeComputeFromOrtGraph() has 742 non-comment lines (error triggered by exceeding 500 lines). [readability/fn_size] [1]
[cpplint] onnxruntime/core/providers/tensorrt/tensorrt_execution_provider.h#L535: onnxruntime/core/providers/tensorrt/tensorrt_execution_provider.h#L535
Line ends in whitespace. Consider deleting these extra spaces. [whitespace/end_of_line] [4]
[cpplint] onnxruntime/core/providers/tensorrt/tensorrt_execution_provider_utils.h#L17: onnxruntime/core/providers/tensorrt/tensorrt_execution_provider_utils.h#L17
For a static/global string constant, use a C style string instead: "static const char EP_CONTEXT_OP_TYPE[]". [runtime/string] [4]
[cpplint] onnxruntime/core/providers/tensorrt/tensorrt_execution_provider_utils.h#L18: onnxruntime/core/providers/tensorrt/tensorrt_execution_provider_utils.h#L18
For a static/global string constant, use a C style string instead: "static const char EP_CONTEXT_ATTR_EMBED_MODE[]". [runtime/string] [4]
[cpplint] onnxruntime/core/providers/tensorrt/tensorrt_execution_provider_utils.h#L19: onnxruntime/core/providers/tensorrt/tensorrt_execution_provider_utils.h#L19
For a static/global string constant, use a C style string instead: "static const char EP_CONTEXT_ATTR_CACHE_CTX[]". [runtime/string] [4]
[cpplint] onnxruntime/core/providers/tensorrt/tensorrt_execution_provider_utils.h#L753: onnxruntime/core/providers/tensorrt/tensorrt_execution_provider_utils.h#L753
Line ends in whitespace. Consider deleting these extra spaces. [whitespace/end_of_line] [4]
[cpplint] onnxruntime/core/session/custom_ops.cc#L314: onnxruntime/core/session/custom_ops.cc#L314
Lines should be <= 120 characters long [whitespace/line_length] [2]
[cpplint] onnxruntime/core/session/custom_ops.cc#L317: onnxruntime/core/session/custom_ops.cc#L317
Lines should be <= 120 characters long [whitespace/line_length] [2]
[cpplint] onnxruntime/core/session/ort_apis.h#L187: onnxruntime/core/session/ort_apis.h#L187
Lines should be <= 120 characters long [whitespace/line_length] [2]
[cpplint] onnxruntime/test/providers/cpu/nn/dropout_op_test.cc#L33: onnxruntime/test/providers/cpu/nn/dropout_op_test.cc#L33
Lines should be <= 120 characters long [whitespace/line_length] [2]