[CoreML MLProgram] Support Float16 (1/N) #34583
Triggered via pull request
September 30, 2024 05:25
wejoncy
Status
Failure
Total duration
14m 5s
Artifacts
–
lint.yml
on: pull_request
Optional Lint
34s
Python format
3m 31s
Optional Lint C++
13m 56s
Lint JavaScript
23s
Annotations
1 error and 14 warnings
|
Python format
Process completed with exit code 1.
|
|
Python format
The following actions uses node12 which is deprecated and will be forced to run on node16: actions-rs/toolchain@v1. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
|
|
Python format
The following actions use a deprecated Node.js version and will be forced to run on node20: actions-rs/toolchain@v1. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
|
|
Python format
The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
|
|
Python format
The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
|
|
Python format
The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
|
|
Python format
The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
|
|
Optional Lint C++:
onnxruntime/core/providers/coreml/builders/impl/base_op_builder.cc#L18
[cpplint] reported by reviewdog 🐶
Add #include <string> for string [build/include_what_you_use] [4]
Raw Output:
onnxruntime/core/providers/coreml/builders/impl/base_op_builder.cc:18: Add #include <string> for string [build/include_what_you_use] [4]
|
|
Optional Lint C++:
onnxruntime/core/providers/coreml/builders/impl/builder_utils.cc#L140
[cpplint] reported by reviewdog 🐶
Using deprecated casting style. Use static_cast<float>(...) instead [readability/casting] [4]
Raw Output:
onnxruntime/core/providers/coreml/builders/impl/builder_utils.cc:140: Using deprecated casting style. Use static_cast<float>(...) instead [readability/casting] [4]
|
|
Optional Lint C++:
onnxruntime/core/providers/coreml/builders/impl/builder_utils.cc#L139
[cpplint] reported by reviewdog 🐶
Add #include <algorithm> for transform [build/include_what_you_use] [4]
Raw Output:
onnxruntime/core/providers/coreml/builders/impl/builder_utils.cc:139: Add #include <algorithm> for transform [build/include_what_you_use] [4]
|
|
Optional Lint C++:
onnxruntime/core/providers/coreml/builders/impl/gridsample_op_builder.cc#L74
[cpplint] reported by reviewdog 🐶
Lines should be <= 120 characters long [whitespace/line_length] [2]
Raw Output:
onnxruntime/core/providers/coreml/builders/impl/gridsample_op_builder.cc:74: Lines should be <= 120 characters long [whitespace/line_length] [2]
|
|
Optional Lint C++:
onnxruntime/core/providers/coreml/builders/impl/slice_op_builder.cc#L233
[cpplint] reported by reviewdog 🐶
If an else has a brace on one side, it should have it on both [readability/braces] [5]
Raw Output:
onnxruntime/core/providers/coreml/builders/impl/slice_op_builder.cc:233: If an else has a brace on one side, it should have it on both [readability/braces] [5]
|
|
Optional Lint C++:
onnxruntime/core/providers/coreml/builders/impl/slice_op_builder.cc#L233
[cpplint] reported by reviewdog 🐶
If/else bodies with multiple statements require braces [readability/braces] [4]
Raw Output:
onnxruntime/core/providers/coreml/builders/impl/slice_op_builder.cc:233: If/else bodies with multiple statements require braces [readability/braces] [4]
|
|
Optional Lint C++:
onnxruntime/core/providers/coreml/builders/impl/slice_op_builder.cc#L234
[cpplint] reported by reviewdog 🐶
At least two spaces is best between code and comments [whitespace/comments] [2]
Raw Output:
onnxruntime/core/providers/coreml/builders/impl/slice_op_builder.cc:234: At least two spaces is best between code and comments [whitespace/comments] [2]
|
|
Optional Lint C++:
onnxruntime/core/providers/coreml/builders/impl/unary_op_builder.cc#L28
[cpplint] reported by reviewdog 🐶
Do not use namespace using-directives. Use using-declarations instead. [build/namespaces] [5]
Raw Output:
onnxruntime/core/providers/coreml/builders/impl/unary_op_builder.cc:28: Do not use namespace using-directives. Use using-declarations instead. [build/namespaces] [5]
|