已合并
[msprof_master]msprof编包支持根据tag名刷新run+whl包名 #182
[msprof_master]msprof编包支持根据tag名刷新run+whl包名 #182
已合并
yuliangbin创建于 4月13日
yuliangbin成员
4月13日

PR 合入模板

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


1. 修改描述

  • 修改原因:
    msprof编的解析whl包需要根据version.info文件刷新版本号
  • 修改内容:
  1. set_up.py读取whl_version环境变量,刷新到whl包的version;
  2. whl包安装适配;
  3. 将version刷新到version.info中;

2. 功能验证

  1. 执行build.sh编包
    image.png
  2. 安装run包
    image.png
    image.png
  3. version.info刷新
    image.png

3. 分支合并要求


3. 代码检视

  • 要求:

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


4. 安全自检

Python、C++

C++


5. 变更知会


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

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 Seanesmhxocism, fanhong (2/2) fanhong (1/1)

💡 Tip:

  • Committer can comment /approve or /lgtm
  • Commenting /approve implies 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. 👍

likedislike
ascend-robotascend-robot成员
4月13日 添加了label:ascend-cla/yes
yuliangbin成员
4月13日 评论:

/build-all

likedislike
ascend-robotascend-robot成员
4月13日 添加了label:build-all-pipeline-running
此处折叠了190条消息 查看更多
wangzixuan成员
4月16日 评论:

/lgtm

likedislike
fanhong
fanhong成员
4月16日 评论:

/approve

likedislike
ascend-robotascend-robot成员
4月16日 添加了label:approvedlgtm
ascend-robot
ascend-robot成员
4月16日 评论:

Review Guide

This pull-request passes review.
Committers who wrote a comment of /approve are: fanhong.
Reviewers who wrote a comment of /lgtm are: fanhong, Seanesmhxocism.

likedislike
ascend-robotascend-robot成员
4月16日 合入了pull request