已合并
[v2.10.0][Fix] Fix static check errors detected by TABS #37928
thickhair创建于 6月9日
[v2.10.0][Fix] Fix static check errors detected by TABS #37928
已合并
thickhair创建于 6月9日
thickhair
thickhair成员
6月9日

【合入来源】

如有社区issue,请关联issue链接
请勿携带内部流程信息(需求链接、问题单、内部issue等)

【修改方案】

请描述修改内容的具体实现,涉及哪些组件之间进行交互,可以用1、2、3、...进行罗列
如果是需求或者重构类的PR,需要补充详细设计文档(说明上下游组件关系、时序图、类图、DFX能力等内容)

【资料变更】

请确认是否涉及资料变更。如涉及,需要在PR中体现,并简要说明修改内容。如不涉及,需填写“不涉及”

【接口变更】

请确认是否涉及跨代码仓或者客户面可见的接口变更。如涉及,需要详细说明接口以及对应的变更内容,同时需要在资料中体现。如不涉及,需填写“不涉及”

【功能验证】

image.png

【CheckList】

PR提交人对以下CheckList自检项进行全量自检,自检通过或不涉及,均修改 [ ] 为 [x]

  • 代码注释完备,正确记录错误日志
  • 代码实现进行了返回值、空指针等校验
  • PR标题正确使用类型标签,如:feat、fix、refactor、docs、test等
  • PR持续集成流水线(CI)执行通过,代码检查无异常
likedislike
Pull Request已成功合入, 合并人@ascend-robot
(感谢 thickhair 的贡献)
thickhairthickhair成员
6月9日 创建了 pull request,commit 900d58ea
ascend-robotascend-robot成员
6月9日 添加了label:ascend-cla/yes
ascend-robot
ascend-robot成员
6月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


PR Approval Progress

Congratulations! All modules have met the lgtm and approve requirements.

Module Approval Details

module lgtm status approve status
repo-Ascend/pytorch 李伟, huangjingwei (2/2) 李伟 (1/1)
test 李伟, huangjingwei (2/2) 李伟 (1/1)
torch_npu/_inductor 李伟, huangjingwei (2/2) 李伟 (1/1)
torch_npu/csrc/distributed 李伟, huangjingwei (2/2) 李伟 (1/1)
torch_npu/utils 李伟, huangjingwei (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

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

likedislike
thickhair
thickhair成员
6月9日 评论:

compile

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

当前仓库存在以下 保护分支

Protected Branch Version Release
master
v2.11.0
v2.7.1
v2.9.0
v2.10.0
v2.12.0
v2.7.1-26.1.0
v2.10.0-26.1.0
v2.9.0-26.1.0
v2.11.0-26.1.0
v2.12.0-26.1.0
ci-test
sync-pr28113--to-v2.9.0

评论 /sync <branch1> <branch2> ... 可将当前 PR 修改同步到其它分支(创建同步 PR):
a) 如果当前 PR 是 Open 状态,同步操作将延迟到 PR 被合并时执行
b) 如果当前 PR 已经 Merged,将立即执行同步操作

注意:

  1. /sync 命令可以指定同步到多个分支,仅最后一个 /sync 命令生效
  2. 如果创建的同步 PR 不正确,可通过向同步 PR 的源分支提交轻量级 PR 完善,或使用 /close 命令关闭
likedislike
ascend-robotascend-robot成员
6月9日 添加了label:ci-pipeline-running
ascend-robot
ascend-robot成员
6月9日 评论:

Ascend docs pipeline is running...

likedislike
ascend-robotascend-robot成员
6月9日 添加了label:docs-ci-pipeline-running
ascend-robot
ascend-robot成员
6月9日 评论:

Ascend docs pipeline is running...

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

✅ 跳过 docs ci 检查,没有需要检查的文档文件

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

✅ 跳过 docs ci 检查,没有需要检查的文档文件

likedislike
ascend-robotascend-robot成员
6月9日 删除了label:docs-ci-pipeline-running
ascend-robotascend-robot成员
6月9日 添加了label:docs-ci-pipeline-success
ascend-robotascend-robot成员
6月9日 删除了label:ci-pipeline-running
ascend-robotascend-robot成员
6月9日 添加了label:ci-pipeline-passed
ascend-robot
ascend-robot成员
6月9日 评论:
流水线 PR-pipeline_pytorch#30551 已完成
阶段 任务名 状态 详情
编译构建 Build_X86 >>>
Build_ARM >>>
Build_LibTorch_x86 >>>
Build_LibTorch_ARM >>>
Build_X86_torchair 🛑 >>>
Build_ARM_torchair 🛑 >>>
patch_test 🛑 >>>
恶意代码检查 Antipoison >>>
编码安全与规范检查 CodeCheck >>>
check_error >>>
CodeCheck_lintrunner >>>
开源片段检查 SCA >>>
开发者测试 UT_X86_Part_01 🛑 >>>
UT_X86_Part_02 🛑 >>>
UT_ARM_A3_Part_01 🛑 >>>
UT_ARM_A3_Part_02 🛑 >>>
UT_ARM_A2_Part_01 >>>
UT_ARM_A2_Part_02 >>>
UT_ARM_A2_Part_03 >>>
UT_inductor_Part_01 >>>
UT_inductor_Part_02 >>>
UT_inductor_Part_03 >>>
UT_inductor_Part_04 >>>
UT_DIST_ARM_Part_01 >>>
UT_DIST_ARM_Part_02 >>>
UT_DIST_ARM_Part_03 >>>
UT_DIST_ARM_Part_04 >>>
UT_ARM_A2_Select_Part_01 >>>
UT_ARM_A2_Select_Part_02 >>>
流水线 PR-pipeline_pytorch >>>
likedislike
thickhairthickhair成员
6月10日 修改了pull request 的描述
thickhairthickhair成员
6月10日 修改了pull request 的描述
thickhairthickhair成员
6月10日 修改了pull request 的描述
thickhairthickhair成员
6月10日 修改了pull request 的描述
thickhairthickhair成员
6月10日 修改了pull request 的描述
thickhairthickhair成员
6月10日 修改了pull request 的描述
Jingwei Huang
Jingwei Huang成员
6月10日 评论:

/lgtm

likedislike
Jingwei Huang
Jingwei Huang成员6月10日进行代码检视2
torch_npu/_inductor/lowering_op_list.py
已过期
@@ -82,3 +82,3 @@
8282 torch.ops.higher_order.cond,
8383 torch.ops._inductor_test.realize,
84- torch.ops._inductor_test.realize.default,
84+ torch.ops._inductor_test.realize.default,
Jingwei Huang
Jingwei Huang6月10日评论:

看起来多了一个SPACES

likedislike
System
系统消息系统
6月11日 评论:
likedislike
thickhairthickhair成员
6月11日 update merge request[project id: 7404318, iid: 37928, commit_id: 9968115fb1be444ad6c3219d08c23cc6a5d32ea7] virtual merging success
thickhairthickhair成员
6月11日 强制推送  1 个提交:5fdfc119-[v2.10.0][Fix] Fix static check errors detected by TABS
thickhairthickhair成员
6月11日 update merge request[project id: 7404318, iid: 37928, commit_id: c87ecd13c85cb96a32c58ec3a1ecbcf23148cee3] virtual merging success
thickhair
thickhair成员
6月11日 评论:

compile

likedislike
ascend-robotascend-robot成员
6月11日 删除了label:ci-pipeline-passed
ascend-robot
ascend-robot成员
6月11日 评论:

Notification

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

likedislike
ascend-robotascend-robot成员
6月11日 添加了label:ci-pipeline-running
ascend-robot
ascend-robot成员
6月11日 评论:

Ascend docs pipeline is running...

likedislike
ascend-robotascend-robot成员
6月11日 删除了label:docs-ci-pipeline-success
ascend-robotascend-robot成员
6月11日 添加了label:docs-ci-pipeline-running
ascend-robot
ascend-robot成员
6月11日 评论:

Ascend docs pipeline is running...

likedislike
ascend-robot
ascend-robot成员
6月11日 评论:

✅ 跳过 docs ci 检查,没有需要检查的文档文件

likedislike
ascend-robotascend-robot成员
6月11日 删除了label:docs-ci-pipeline-running
ascend-robotascend-robot成员
6月11日 添加了label:docs-ci-pipeline-success
ascend-robot
ascend-robot成员
6月11日 评论:

✅ 跳过 docs ci 检查,没有需要检查的文档文件

likedislike
ascend-robotascend-robot成员
6月11日 删除了label:ci-pipeline-running
ascend-robotascend-robot成员
6月11日 添加了label:ci-pipeline-passed
ascend-robot
ascend-robot成员
6月11日 评论:
流水线 PR-pipeline_pytorch#31354 已完成
阶段 任务名 状态 详情
编译构建 Build_X86 >>>
Build_ARM >>>
Build_LibTorch_x86 >>>
Build_LibTorch_ARM >>>
Build_X86_torchair 🛑 >>>
Build_ARM_torchair 🛑 >>>
patch_test 🛑 >>>
恶意代码检查 Antipoison >>>
编码安全与规范检查 CodeCheck >>>
check_error >>>
CodeCheck_lintrunner >>>
开源片段检查 SCA >>>
开发者测试 UT_X86_Part_01 🛑 >>>
UT_X86_Part_02 🛑 >>>
UT_ARM_A3_Part_01 🛑 >>>
UT_ARM_A3_Part_02 🛑 >>>
UT_ARM_A2_Part_01 >>>
UT_ARM_A2_Part_02 >>>
UT_ARM_A2_Part_03 >>>
UT_inductor_Part_01 >>>
UT_inductor_Part_02 >>>
UT_inductor_Part_03 >>>
UT_inductor_Part_04 >>>
UT_DIST_ARM_Part_01 >>>
UT_DIST_ARM_Part_02 >>>
UT_DIST_ARM_Part_03 >>>
UT_DIST_ARM_Part_04 >>>
UT_ARM_A2_Select_Part_01 >>>
UT_ARM_A2_Select_Part_02 >>>
流水线 PR-pipeline_pytorch >>>
likedislike
Jingwei Huang
Jingwei Huang成员
6月11日 评论:

/lgtm

likedislike
thickhairthickhair成员
6月11日 解决了最后一个问题
liwei386成员
6月11日 评论:

/approve

likedislike
ascend-robotascend-robot成员
6月11日 添加了label:approvedlgtm
ascend-robotascend-robot成员
6月11日 合入了pull request
ascend-robot
ascend-robot成员
6月11日 评论:

Pull Request 已合并或已关闭。

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

likedislike
ascend-robot
ascend-robot成员
6月11日 评论:
流水线 pytorch_gitcode_PR_multiVersion#10534 [ commitID:5fdfc119 ] 已完成
likedislike