已合并
add Core reviewer #668
梁松伟创建于 13 天前
add Core reviewer #668
已合并
梁松伟创建于 13 天前
梁松伟
梁松伟
13 天前
add Core reviewer
likedislike
Pull Request已成功合入, 合并人@ascend-robot
(感谢 梁松伟 的贡献)
atomgit-bot
atomgit-bot成员
13 天前 评论:

变更摘要

此 PR 主要对 AscendForPyTorch/sigs/core/sig-info.yaml 中的 Core SIG 评审人配置进行更新,新增三名核心评审人,并清理/调整了部分仓库模块的评审人列表。

主要改动

  • 新增三名 Core 评审人:在多个仓库(Ascend/pytorchAscend/op-plugin)的 committers 列表中统一添加了 xuyun15wanglijun55xiu_21 三个 gitcode_id,使其具备 Core 评审权限。
  • 移除重复或不再需要的评审人:从 Ascend/pytorch 仓库的 profiler 路径条目中移除了 ffmh,并从 Ascend/op-plugin 仓库中移除了 sunyu-xuan
  • 调整 wanglijun55 的位置:将 wanglijun55Ascend/op-plugin 仓库顶层 committers 列表中原有位置移除,改为与 xuyun15xiu_21 一道统一添加到该仓库及 Ascend/pytorch 的多个子路径条目中。
likedislike
atomgit-bot
atomgit-bot成员
13 天前 评论:

代码审查

✅ 未发现问题

likedislike
ascend-robotascend-robot成员
13 天前 添加了label:ascend-cla/yes
ascend-robot
ascend-robot成员
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
sig-core 楚浩田, chujinjin (2/2) 楚浩田 (1/1)

💡 Tip:

  • Committer can comment /approve or /lgtm
  • Commenting /approve implies both code review (lgtm) and intent to merge (approve)

CLA Signature Pass

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

likedislike
ascend-robot
ascend-robot成员
13 天前 评论:

ascend docs pipeline is running...

likedislike
ascend-robotascend-robot成员
13 天前 添加了label:docs-ci-pipeline-running
ascend-robotascend-robot成员
13 天前 添加了label:ci-pipeline-running
ascend-robot
ascend-robot成员
13 天前 评论:

✅ 文档门禁通过!

检查项 检查结果 详情
markdownlint ✅ 已通过 查看详情
link-validity-check ✅ 已通过 查看详情
resource-existence-check ✅ 已通过 查看详情
tag-closed-check ✅ 已通过 查看详情
likedislike
ascend-robotascend-robot成员
13 天前 删除了label:docs-ci-pipeline-running
ascend-robotascend-robot成员
13 天前 添加了label:docs-ci-pipeline-success
ascend-robot
ascend-robot成员
13 天前 评论:

✅ 规则检查门禁通过!

检查项 检查结果 错误详情
YAML格式检查 ✅ 已通过 -
必须字段检查 ✅ 已通过 -
gitcode_id存在性检查 ✅ 已通过 -
仓库存在性检查 ✅ 已通过 -
文件位置检查 ✅ 已通过 -
字段一致性检查 ✅ 已通过 -
maintainer数量检查 ✅ 已通过 -
仓库类型检查 ✅ 已通过 -
sig名唯一性检查 ✅ 已通过 -
sig目录注册检查 ✅ 已通过 -
likedislike
ascend-robotascend-robot成员
13 天前 删除了label:ci-pipeline-running
ascend-robotascend-robot成员
13 天前 添加了label:ci-pipeline-passed
chenrayray
chenrayray成员13 天前进行代码检视2
AscendForPyTorch/sigs/core/README.md
已过期
@@ -1,0 +1,1 @@
1-# Torch\-NPU Core SIG
1+# TorchNPU Core SIG
chenrayray
chenrayray13 天前评论:
改动建议
1
- # TorchNPU Core SIG
1
+ # Ascend for PyTorch Core SIG
应用建议
likedislike
System
系统消息系统
13 天前 评论:
htchu成员13 天前进行代码检视2
AscendForPyTorch/sigs/core/sig-info.yaml
已过期
@@ -32,7 +32,7 @@
3232 - gitcode_id: dilililiwhy
3333 - gitcode_id: guoqi1024
3434 - gitcode_id: htchu
3535 - gitcode_id: wangmin0104
htchu13 天前评论:

删除wangmin

likedislike
System
系统消息系统
13 天前 评论:
htchu成员13 天前进行代码检视2
AscendForPyTorch/sigs/core/sig-info.yaml
@@ -52,30 +55,30 @@
5255 entry_configs:
5356 - path:
5457 - torch_npu/npu
@@ -88,10 +91,12 @@ repositories:
8891 - gitcode_id: liujunzhu
8992 - gitcode_id: hbhu_bin
9093 - gitcode_id: luochao60
91- - gitcode_id: ffmh
9294 - gitcode_id: huangyunlong2022
9395 - gitcode_id: chengpeng25
9496 - gitcode_id: renyujin
97+ - gitcode_id: xuyun15
98+ - gitcode_id: wanglijun55
99+ - gitcode_id: xiu_21
95100 - path:
96101 - torch_npu/profiler
97102 - torch_npu/csrc/profiler
@@ -136,6 +141,9 @@ repositories:
136141 - gitcode_id: chengpeng25
137142 - gitcode_id: daifu1234567
138143 - gitcode_id: renyujin
144+ - gitcode_id: xuyun15
145+ - gitcode_id: wanglijun55
146+ - gitcode_id: xiu_21
139147- repo:
140148 - Ascend/op-plugin
141149 committers:
@@ -166,7 +174,6 @@ repositories:
166174 - gitcode_id: culechan
167175 - gitcode_id: liangchenghui
168176 - gitcode_id: ffmh
htchu13 天前评论:

lijun的权限保留

likedislike
梁松伟
梁松伟
13 天前 评论:
htchu成员13 天前进行代码检视2
AscendForPyTorch/sigs/core/sig-info.yaml
已过期
@@ -52,30 +54,30 @@
54+ - gitcode_id: xiu_21
5255 entry_configs:
5356 - path:
5457 - torch_npu/npu
@@ -88,10 +91,12 @@ repositories:
8891 - gitcode_id: liujunzhu
8992 - gitcode_id: hbhu_bin
9093 - gitcode_id: luochao60
91- - gitcode_id: ffmh
9294 - gitcode_id: huangyunlong2022
9395 - gitcode_id: chengpeng25
9496 - gitcode_id: renyujin
97+ - gitcode_id: xuyun15
98+ - gitcode_id: wanglijun55
99+ - gitcode_id: xiu_21
95100 - path:
96101 - torch_npu/profiler
97102 - torch_npu/csrc/profiler
@@ -136,6 +141,9 @@ repositories:
136141 - gitcode_id: chengpeng25
137142 - gitcode_id: daifu1234567
138143 - gitcode_id: renyujin
144+ - gitcode_id: xuyun15
145+ - gitcode_id: wanglijun55
146+ - gitcode_id: xiu_21
139147- repo:
140148 - Ascend/op-plugin
141149 committers:
@@ -166,7 +174,6 @@ repositories:
166174 - gitcode_id: culechan
167175 - gitcode_id: liangchenghui
htchu13 天前评论:

chenghui的权限清掉

likedislike
System
系统消息系统
13 天前 评论:
htchu成员13 天前进行代码检视2
AscendForPyTorch/sigs/core/sig-info.yaml
已过期
@@ -53,30 +56,30 @@
5356 - path:
5457 - torch_npu/npu
@@ -88,10 +91,12 @@ repositories:
8891 - gitcode_id: liujunzhu
8992 - gitcode_id: hbhu_bin
9093 - gitcode_id: luochao60
91- - gitcode_id: ffmh
9294 - gitcode_id: huangyunlong2022
9395 - gitcode_id: chengpeng25
9496 - gitcode_id: renyujin
97+ - gitcode_id: xuyun15
98+ - gitcode_id: wanglijun55
99+ - gitcode_id: xiu_21
95100 - path:
96101 - torch_npu/profiler
97102 - torch_npu/csrc/profiler
@@ -136,6 +141,9 @@ repositories:
136141 - gitcode_id: chengpeng25
137142 - gitcode_id: daifu1234567
138143 - gitcode_id: renyujin
144+ - gitcode_id: xuyun15
145+ - gitcode_id: wanglijun55
146+ - gitcode_id: xiu_21
139147- repo:
140148 - Ascend/op-plugin
141149 committers:
@@ -154,7 +162,7 @@
154162 reviewers:
155163 - gitcode_id: li_jing_hw
156164 - gitcode_id: liangsongwei
157165 - gitcode_id: dilililiwhy
htchu13 天前评论:

dilililiwhy也清理掉

likedislike
System
系统消息系统
13 天前 评论:
htchu成员13 天前进行代码检视2
AscendForPyTorch/sigs/core/sig-info.yaml
@@ -95,2 +98,4 @@
98+ - gitcode_id: wanglijun55
99+ - gitcode_id: xiu_21
95100 - path:
96101 - torch_npu/profiler
htchu13 天前评论:

工具模块给mindstudio加个reviewer

likedislike
梁松伟
梁松伟
13 天前 评论:
梁松伟梁松伟
13 天前 推送  1 个提交:94525273-remove unuse list
ascend-robotascend-robot成员
13 天前 添加了label:stat/needs-squash
ascend-robotascend-robot成员
13 天前 删除了label:ascend-cla/yes
ascend-robotascend-robot成员
13 天前 删除了label:ci-pipeline-passed
ascend-robot
ascend-robot成员
13 天前 评论:

Notification

This pull request has been changed(code update) or closed, so removes the following label(s): ci-pipeline-passed.

likedislike
ascend-robotascend-robot成员
13 天前 添加了label:ascend-cla/yes
ascend-robot
ascend-robot成员
13 天前 评论:

ascend docs pipeline is running...

likedislike
ascend-robotascend-robot成员
13 天前 删除了label:docs-ci-pipeline-success
ascend-robotascend-robot成员
13 天前 添加了label:docs-ci-pipeline-running
ascend-robotascend-robot成员
13 天前 添加了label:ci-pipeline-running
ascend-robot
ascend-robot成员
13 天前 评论:

✅ 文档门禁通过!

检查项 检查结果 详情
markdownlint ✅ 已通过 查看详情
link-validity-check ✅ 已通过 查看详情
resource-existence-check ✅ 已通过 查看详情
tag-closed-check ✅ 已通过 查看详情
likedislike
ascend-robot
ascend-robot成员
13 天前 评论:

✅ 规则检查门禁通过!

检查项 检查结果 错误详情
YAML格式检查 ✅ 已通过 -
必须字段检查 ✅ 已通过 -
gitcode_id存在性检查 ✅ 已通过 -
仓库存在性检查 ✅ 已通过 -
文件位置检查 ✅ 已通过 -
字段一致性检查 ✅ 已通过 -
maintainer数量检查 ✅ 已通过 -
仓库类型检查 ✅ 已通过 -
sig名唯一性检查 ✅ 已通过 -
sig目录注册检查 ✅ 已通过 -
likedislike
ascend-robotascend-robot成员
13 天前 删除了label:ci-pipeline-running
ascend-robotascend-robot成员
13 天前 删除了label:docs-ci-pipeline-running
ascend-robotascend-robot成员
13 天前 添加了label:ci-pipeline-passed
ascend-robotascend-robot成员
13 天前 添加了label:docs-ci-pipeline-success
htchu成员
13 天前 评论:

/approve

likedislike
ascend-robotascend-robot成员
13 天前 添加了label:approved
Hhtchu成员
13 天前 解决了最后一个问题
chujinjin
13 天前 评论:

/lgtm

likedislike
ascend-robotascend-robot成员
13 天前 添加了label:lgtm
ascend-robotascend-robot成员
13 天前 合入了pull request
ascend-robot
ascend-robot成员
13 天前 评论:

Pull Request 已合并或已关闭。

If you want to solve this problem, you can click here to do it in the FAQs.

likedislike
ascend-robot
ascend-robot成员
13 天前 评论:

Pull Request 已合并或已关闭。

If you want to solve this problem, you can click here to do it in the FAQs.

likedislike