已合并
[fix_torch_profiler_2.10.0]修复profiler L0级别kernel_details.csv无shape信息问题 #32102
yuliangbin创建于 3月20日
[fix_torch_profiler_2.10.0]修复profiler L0级别kernel_details.csv无shape信息问题 #32102
已合并
Pull Request已成功合入, 合并人@ascend-robot
(感谢 yuliangbin 的贡献)3月20日 创建了 pull request,commit 283a17e1
3月20日 关联了issue:[Bug]: profiler l1级别下kernel_details.csv文件缺少shape等信息
ascend-robot
3月20日 评论:
3月20日 评论:
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 |
|---|---|---|
| test | ✅ chenhao_1209, htchu (2/2) | ✅ chenhao_1209, htchu (2/1) |
| torch_npu/profiler | ✅ chenhao_1209, htchu (2/2) | ✅ chenhao_1209, htchu (2/1) |
💡 Tip:
- Committer can comment
/approveor/lgtm- Commenting
/approveimplies both code review (lgtm) and intent to merge (approve)
CLA Signature Pass
yu_liangbin, thanks for your pull request. All authors of the commits have signed the CLA. 👍


AtlasAccount
3月20日 评论:
3月20日 评论:
3月20日 添加了label:ascend-cla/yes
yuliangbin
3月20日 评论:
3月20日 评论:
compile


3月20日 添加了label:ci-pipeline-running
3月20日 删除了label:ci-pipeline-running
3月20日 添加了label:ci-pipeline-passed
ascend-robot
3月20日 评论:
3月20日 评论:
流水线 PR-pipeline_pytorch#11791 已完成
| 阶段 | 任务名 | 状态 | 详情 |
|---|---|---|---|
| 编译构建 | Build_X86 | ✅ | >>> |
| Build_ARM | ✅ | >>> | |
| Build_LibTorch_x86 | ✅ | >>> | |
| Build_X86_torchair | 🛑 | >>> | |
| Build_ARM_torchair | 🛑 | >>> | |
| Build_ARM_inductor | 🛑 | >>> | |
| 恶意代码检查 | Antipoison | ✅ | >>> |
| 编码安全与规范检查 | CodeCheck | ✅ | >>> |
| check_error | ✅ | >>> | |
| 开源片段检查 | SCA | ✅ | >>> |
| 开发者测试 | UT_X86_Part_01 | 🛑 | >>> |
| UT_X86_Part_02 | 🛑 | >>> | |
| UT_ARM_A3_Part_01 | 🛑 | >>> | |
| UT_ARM_A3_Part_02 | 🛑 | >>> | |
| UT_DIST_X86_Part_01 | 🛑 | >>> | |
| UT_DIST_X86_Part_02 | 🛑 | >>> | |
| UT_DIST_X86_Part_03 | 🛑 | >>> | |
| UT_DIST_X86_Part_04 | 🛑 | >>> | |
| UT_inductor_Part_01 | 🛑 | >>> | |
| UT_inductor_Part_02 | 🛑 | >>> | |
| UT_inductor_Part_03 | 🛑 | >>> | |
| UT_inductor_Part_04 | 🛑 | >>> | |
| UT_ARM_A2_Part_01 | ✅ | >>> | |
| UT_ARM_A2_Part_02 | ✅ | >>> | |
| 流水线 | PR-pipeline_pytorch | ✅ | >>> |


chenhao_1209
3月23日 评论:
3月23日 评论:
/approve


3月23日 添加了label:approved
3月23日 添加了label:lgtm
ascend-robot
3月23日 评论:
3月23日 评论:
Review Guide
This pull-request passes review.
Committers who wrote a comment of /approve are: htchu, chenhao_1209.
Reviewers who wrote a comment of /lgtm are: htchu, chenhao_1209.


3月23日 合入了pull request
【合入来源】
【修改方案】
1.功能:解析得时候会根据是否为aicore_none进行kernel_details.csv的表头过滤,但是在l1、aicore_metrics非aicore_none场景下会导致kernel_details.csv表头缺失shape字段,实际上只有l0场景下kernel_details.csv的表头比较少,l1场景下不应该缺失表头,因此过滤逻辑应当是是否为l0,若是l0场景,则过滤,反之不过滤;
2.UT:UT适配
【资料变更】
不涉及
【接口变更】
不涉及
【功能验证】
验证结果:离线解析一份l1+aicore_none场景的数据,kernel_details.csv的表头与op_summary.csv的表头一致,未被过滤
【CheckList】