feat(AWML): introduce new deployment and onnx/tensorRT evaluation pipeline#115
feat(AWML): introduce new deployment and onnx/tensorRT evaluation pipeline#115vividf wants to merge 43 commits into
Conversation
68d1700 to
7ac5b8b
Compare
Signed-off-by: vividf <yihsiang.fang@tier4.jp>
Signed-off-by: vividf <yihsiang.fang@tier4.jp>
Signed-off-by: vividf <yihsiang.fang@tier4.jp> Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Signed-off-by: vividf <yihsiang.fang@tier4.jp>
Signed-off-by: vividf <yihsiang.fang@tier4.jp>
Signed-off-by: vividf <yihsiang.fang@tier4.jp>
Signed-off-by: vividf <yihsiang.fang@tier4.jp>
Signed-off-by: vividf <yihsiang.fang@tier4.jp>
Signed-off-by: vividf <yihsiang.fang@tier4.jp>
Signed-off-by: vividf <yihsiang.fang@tier4.jp>
5256306 to
2b28f60
Compare
Signed-off-by: vividf <yihsiang.fang@tier4.jp>
Signed-off-by: vividf <yihsiang.fang@tier4.jp>
Signed-off-by: vividf <yihsiang.fang@tier4.jp>
Signed-off-by: vividf <yihsiang.fang@tier4.jp>
Signed-off-by: vividf <yihsiang.fang@tier4.jp>
Signed-off-by: vividf <yihsiang.fang@tier4.jp>
Signed-off-by: vividf <yihsiang.fang@tier4.jp>
Signed-off-by: vividf <yihsiang.fang@tier4.jp>
Signed-off-by: vividf <yihsiang.fang@tier4.jp>
Signed-off-by: vividf <yihsiang.fang@tier4.jp>
Signed-off-by: vividf <yihsiang.fang@tier4.jp>
Signed-off-by: vividf <yihsiang.fang@tier4.jp>
Signed-off-by: vividf <yihsiang.fang@tier4.jp>
Signed-off-by: vividf <yihsiang.fang@tier4.jp> Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Signed-off-by: vividf <yihsiang.fang@tier4.jp>
Signed-off-by: vividf <yihsiang.fang@tier4.jp>
Signed-off-by: vividf <yihsiang.fang@tier4.jp>
Signed-off-by: vividf <yihsiang.fang@tier4.jp>
Signed-off-by: vividf <yihsiang.fang@tier4.jp>
Signed-off-by: vividf <yihsiang.fang@tier4.jp>
Signed-off-by: vividf <yihsiang.fang@tier4.jp>
8756669 to
1d4c27f
Compare
KSeangTan
left a comment
There was a problem hiding this comment.
Only review the first part, please address the comments accordingly first.
| Args: | ||
| log_file_path: Absolute or resolved path to the log file. | ||
| """ | ||
| path = os.path.abspath(os.path.expanduser(log_file_path)) |
| """Require a non-empty checkpoint path that exists as a regular file.""" | ||
| if not isinstance(checkpoint_path, str): | ||
| raise TypeError(f"checkpoint_path must be a string, got {type(checkpoint_path).__name__}.") | ||
| path = os.path.expanduser(checkpoint_path.strip()) |
|
|
||
| def _needs_cuda_device(self) -> bool: | ||
| """Determine if current deployment config requires a CUDA device.""" | ||
| if self.export_config.should_export_tensorrt: |
There was a problem hiding this comment.
I believe this function is unnecessary complicated, for example, we only need to check if tensorrt is enabled since we will use only tensorrt with cuda right?
There was a problem hiding this comment.
Thanks, renamed _needs_cuda_device → _uses_tensorrt to make the intent clear. Kept the three branches: TensorRT (hence CUDA) can be pulled in without an export step — evaluation can run a pre-built .engine, and a verification scenario can set ref/test_backend = tensorrt — so should_export_tensorrt alone would miss those and fail mid-run instead of at config validation.
| if not output_names: | ||
| output_names = ("output",) | ||
| settings_dict = { | ||
| "opset_version": onnx_config.opset_version, |
There was a problem hiding this comment.
self._onnx_export_config.opset_config
There was a problem hiding this comment.
self._onnx_config is the shared OnnxConfig section (global ONNX knobs), not the per-component ONNXExportConfig — the latter is assembled per component in get_onnx_settings by combining OnnxConfig with that component's I/O. The field is opset_version, so I kept these names as-is for that distinction
Signed-off-by: vividf <yihsiang.fang@tier4.jp>
Signed-off-by: vividf <yihsiang.fang@tier4.jp>
Signed-off-by: vividf <yihsiang.fang@tier4.jp>
…se.py Signed-off-by: vividf <yihsiang.fang@tier4.jp>
Signed-off-by: vividf <yihsiang.fang@tier4.jp>
Signed-off-by: vividf <yihsiang.fang@tier4.jp>
Signed-off-by: vividf <yihsiang.fang@tier4.jp>
Signed-off-by: vividf <yihsiang.fang@tier4.jp>
Signed-off-by: vividf <yihsiang.fang@tier4.jp>
Signed-off-by: vividf <yihsiang.fang@tier4.jp>
Signed-off-by: vividf <yihsiang.fang@tier4.jp>
Signed-off-by: vividf <yihsiang.fang@tier4.jp>
AWML Unified Deployment Framework
Introduced a unified deployment and evaluation framework for AWML, significantly improving consistency, scalability, and maintainability across projects
Key Contributions
1. Unified Deployment Pipeline
2. Cross-Backend Verification
3. Integrated Evaluation Framework
How to run