已合并
adapt dpu data #208
wangzixuan创建于 4月25日
adapt dpu data #208
已合并
Pull Request已成功合入, 合并人@ascend-robot
（感谢 wangzixuan 的贡献）
ascend-robot
4月25日 评论：
4月25日 评论：
ascend-robot
4月25日 评论：
4月25日 评论：
Thanks for your pull-request.
The full list of commands accepted by me can be found at here。
You can get sig-info at here
PR Approval Progress
✅ Congratulations! All modules have met the lgtm and approve requirements.
Module Approval Details
| module | lgtm status | approve status |
|---|---|---|
| repo-Ascend/msprof | ✅ 陈豪, mei-feiyao (2/2) | ✅ 陈豪 (1/1) |
💡 Tip:
- Committer can comment
/approveor/lgtm- Commenting
/approveimplies both code review (lgtm) and intent to merge (approve)
CLA Signature Pass
Seanesmhxocism, thanks for your pull request. All authors of the commits have signed the CLA. 👍
4月25日 添加了label：ascend-cla/yes
4月25日 修改了pull request 的描述
wangzixuan
4月25日 评论：
4月25日 评论：
compile
此处折叠了43条消息 查看更多
5月25日 解决了最后一个问题
analysis/msmodel/dpu/dpu_task_model.py
@@ -0,0 +48,4 @@
48+ class for dpu task viewer
49+ """
50+
51+ def __init__(self: any, result_dir: str, db_name: str, table_list: list) -> None:
[review]
问题：db_name 和 table_list 完全由外部传入，但类名是 DPUTaskViewModel，理论上应绑定 DPU 相关 DB 和表。
影响：调用方可能传入错误 DB 或表，导致 get_timeline_data 查询固定 DPU 表时与初始化表列表不一致。
建议： 如果该类只服务 DPU，建议构造函数内部固定使用 DBNameConstant.DB_DPU 和 DPU 表列表；如果必须外部传参，需要校验参数合法性。
5月25日 解决了最后一个问题
analysis/msmodel/dpu/dpu_task_model.py
@@ -0,0 +51,4 @@
51+ def __init__(self: any, result_dir: str, db_name: str, table_list: list) -> None:
52+ super().__init__(result_dir, db_name, table_list)
53+
54+ def get_timeline_data(self: any) -> tuple:
[review]
问题： DPUTrackDto 同时用于 task track 和 HCCL track 两种字段数量、字段语义明显不同的数据。
影响：DTO 字段如果不完全兼容，可能出现字段错位、属性缺失或后续解析逻辑混乱。
建议： 建议分别定义 DPUTaskTrackDto 和 DPUHcclTrackDto；如果必须复用，需要确认 DTO 明确支持两类查询结果，并补充注释说明
5月25日 解决了最后一个问题
PR 合入模板
注：经过自检不涉及的可标注“不涉及”或直接打勾，特殊情况请文字备注。不符合规范的 PR 不允许合入，请（后备）commit 注意。
1. 修改描述
适配dpu新数据内容解析。
当前新增部分不涉及业务代码的dpu数据解析基础类。
2. 功能验证
3. 分支合并要求
3. 代码检视
要求：
检视意见数：____ 条 （请填写本次检视的意见总数，用于commit合入前审视）
4. 安全自检
Python、C++
C++
5. 变更知会