已合并
【重构】win本地执行 ut失败问题处理,部分ut代码重构 #24
wangzixuan创建于 1月9日
【重构】win本地执行 ut失败问题处理,部分ut代码重构 #24
已合并
wangzixuan创建于 1月9日
wangzixuan成员
1月9日

PR 合入模板

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


1. 修改描述

  • 修改原因:
    win平台在PyCharm工具上直接执行pytest,由于平台问题存在较多失败用例
    针对win平台相关问题进行bug修复
  • 修改内容:
    修改相关ut失败问题,对ut代码进行部分重构,提高可读性

2. 功能验证


3. 分支合并要求


3. 代码检视

  • 要求:

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


4. 安全自检

Python、C++:

C++:


5. 变更知会


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

Thank your for your pull-request.

The full list of commands accepted by me can be found at here.

You can get sig-info at here

likedislike
ascend-robot
ascend-robot成员
1月9日 评论:

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

Module List Reviewers Committers
repo-Ascend/msprof aerfaliang, chenhao_1209, kali20gakki1, Seanesmhxocism, eejiechu xhahn77, aerfaliang, chenhao_1209, kali20gakki1
likedislike
ascend-robotascend-robot成员
1月9日 添加了label:ascend-cla/yes
Wwangzixuan成员
1月9日 修改了pull request 的描述
Wwangzixuan成员
1月9日 修改标题为 “【重构】win本地执行 ut失败问题处理”,原标题为“win本地执行 ut失败问题处理”
Wwangzixuan成员
1月9日 修改标题为 “【重构】win本地执行 ut失败问题处理,部分ut代码重构”,原标题为“【重构】win本地执行 ut失败问题处理”
wangzixuan成员
1月9日 评论:

compile

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

/lgtm

likedislike
Wwangzixuan成员
1月9日 强制推送  1 个提交:eb4f123a-fix ut testcase failed on win
ascend-robot
ascend-robot成员
1月9日 评论:

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

Module List Reviewers Committers
repo-Ascend/msprof aerfaliang, chenhao_1209, kali20gakki1, Seanesmhxocism, eejiechu xhahn77, aerfaliang, chenhao_1209, kali20gakki1
likedislike
ascend-robot
ascend-robot成员
1月9日 评论:

CLA Signature Pass

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

likedislike
wangzixuan成员
1月9日 评论:

compile

likedislike
ascend-robotascend-robot成员
1月9日 删除了label:ci-pipeline-failed
此处折叠了6条事件消息 查看更多
ascend-robotascend-robot成员
1月9日 添加了label:ci-pipeline-passed
ascend-robot
ascend-robot成员
1月9日 评论:
流水线 PR-pipeline_msprof#59(重试第1次)已完成
阶段 任务名 状态 详情
编译构建 Build_msprof_x86 >>>
Build_msprof_arm >>>
恶意代码检查 Antipoison_msprof >>>
编码安全与规范检查 CodeCheck_msprof >>>
开源片段检查 SCA_msprof >>>
开发者测试 UT_msprof >>>
流水线 PR-pipeline_msprof >>>
likedislike
mei-feiyao
mei-feiyao成员
1月12日 评论:

/lgtm

likedislike
chenhao_1209成员
1月12日 评论:

/approve

likedislike
ascend-robotascend-robot成员
1月12日 添加了label:approved
WangJie
WangJie成员
1月12日 评论:

/lgtm

likedislike
ascend-robotascend-robot成员
1月12日 添加了label:lgtm
ascend-robot
ascend-robot成员
1月12日 评论:

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, chenhao_1209.

likedislike
ascend-robotascend-robot成员
1月12日 合入了pull request