已合并
hccl Aicpu展开算子在python解析时被归为hccl问题 #70
wangzixuan创建于 2月9日
hccl Aicpu展开算子在python解析时被归为hccl问题 #70
已合并
wangzixuan创建于 2月9日
wangzixuan成员
2月9日

PR 合入模板

注:经过自检不涉及的可标注“不涉及”或直接打勾,特殊情况请文字备注。不符合规范的 PR 不允许合入,请(后备)commit 注意。


1. 修改描述

  • 修改原因:
    在A3 hccl aicpu展开场景中,python解析由于taskType判定异常,导致数据被归为hccl,没有和c化解析结果对齐。
    hccl上报aicpu的nodeBasicInfo为HCCL类型,
    python侧:但是python内部判定是用COMMUNICATION进行taskType判定,导致判定失效。aicpu类型保留了HCCL类型,并导致后续aicpu.csv导出筛选aicpu类型失败。
    c侧:通过枚举值进行taskType判定,该枚举值和HCCL的枚举值保持一致,因此c判定正常
  • 修改内容:
    python侧临时增加对hccl类型的判定,避免判定异常,保持和c化解析结果对齐。

后续统一修正hccl aicpu展开类算子的taskType处理流程
涉及处理点,helper的异常分支处理和aicpu导出taskType筛选


2. 功能验证


3. 分支合并要求


3. 代码检视

  • 要求:

    • 合入代码超过 200 行,需三人以上会议检视。
    • 检视密度≥1个/100行。
    • 检视缺陷密度未达要求需提供说明。
    • 大于 1000 行代码原则上不允许合入,需进行备案。
  • 检视意见数:____ 条 (请填写本次检视的意见总数,用于commit合入前审视)


4. 安全自检

Python、C++:

C++:


5. 变更知会


likedislike
Pull Request已成功合入, 合并人@ascend-robot
(感谢 wangzixuan 的贡献)
ascend-robot
ascend-robot成员
2月9日 评论:

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

以下是根据您提交的修改文件推荐的Reviewer和Committer序列,需各模块评审通过后方可合入.

Module List Reviewers Committers
repo-Ascend/msprof kali20gakki1, fanhong, Seanesmhxocism, eejiechu, fanglanyue0916 xhahn77, aerfaliang, chenhao_1209, kali20gakki1, fanhong
likedislike
ascend-robotascend-robot成员
2月9日 添加了label:ascend-cla/yes
ascend-robot
ascend-robot成员
2月9日 评论:

CLA Signature Pass

Seanesmhxocism, thanks for your pull request. All authors of the commits have signed the CLA. 👍

likedislike
wangzixuan成员
2月9日 评论:

compile

likedislike
ascend-robotascend-robot成员
2月9日 添加了label:ci-pipeline-running
Wwangzixuan成员
2月9日 修改了pull request 的描述
ascend-robotascend-robot成员
2月9日 删除了label:ci-pipeline-running
ascend-robotascend-robot成员
2月9日 添加了label:ci-pipeline-passed
ascend-robot
ascend-robot成员
2月9日 评论:
流水线 PR-pipeline_msprof#222 已完成
阶段 任务名 状态 详情
编译构建 Build_msprof_x86 >>>
Build_msprof_arm >>>
恶意代码检查 Antipoison_msprof >>>
编码安全与规范检查 CodeCheck_msprof >>>
开源片段检查 SCA_msprof >>>
开发者测试 UT_msprof >>>
流水线 PR-pipeline_msprof >>>
likedislike
WangJie
WangJie成员
2月9日 评论:

/lgtm

likedislike
mei-feiyao
mei-feiyao成员
2月9日 评论:

/lgtm

likedislike
ascend-robotascend-robot成员
2月9日 添加了label:lgtm
Wwangzixuan成员
2月10日 修改了pull request 的描述
chenhao_1209成员
2月10日 评论:

/approve

likedislike
ascend-robotascend-robot成员
2月10日 添加了label:approved
ascend-robot
ascend-robot成员
2月10日 评论:

Review Guide

This Pull-Request Passes Review.
Committers who wrote a comment of /approve are: chenhao_1209.
Reviewers who wrote a comment of /lgtm are: eejiechu, mei-feiyao, chenhao_1209.

likedislike
ascend-robotascend-robot成员
2月10日 合入了pull request