已合并
fix: lintrunner --all-files --take NEWLINE -a #35875
Jingwei Huang创建于 5月17日
fix: lintrunner --all-files --take NEWLINE -a #35875
已合并
Jingwei Huang创建于 5月17日
Jingwei Huang
Jingwei Huang成员
5月17日

【合入来源】

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

【修改方案】

  1. 执行:lintrunner --all-files --take NEWLINE -a 修复换行符,不影响代码逻辑。
  2. 屏蔽两个分布式用例,已提issue,给对应接口人跟踪。

【资料变更】

不涉及

【接口变更】

不涉及

【功能验证】

不涉及

【CheckList】

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

likedislike
Pull Request已成功合入, 合并人@ascend-robot
(感谢 Jingwei Huang 的贡献)
Jingwei HuangJingwei Huang成员
5月17日 创建了 pull request,commit 05cd1b60
openLiBingCI成员5月17日进行代码检视1
tools/flight_recorder/components/builder.py
@@ -325,0 +45,7 @@
45+ logger.warning("tabulate is not installed. Proceeding without it.")
46+ 
47+ # Define a no-op tabulate function
48+ def tabulate(data: Any, headers: Any = None) -> Any: # type: ignore[misc]
49+ return data
50+ 
51+ 
52+"""
openLiBingCI5月17日评论:

此条代码评论区间+45+52

【openlibing.ci】识别到代码检查告警抑制注释,匹配工具:mypy,请Committer检视其合理性。

likedislike
openLiBingCI成员5月17日进行代码检视1
tools/flight_recorder/components/types.py
@@ -584,0 +30,5 @@
30+class TypeInfo(NamedTuple):
31+ name: str
32+ fields: list[tuple[str, type]] # type: ignore[type-arg]
33+ 
34+ @classmethod
35+ def from_type(cls, c: T) -> "TypeInfo":
openLiBingCI5月17日评论:

此条代码评论区间+30+35

【openlibing.ci】识别到代码检查告警抑制注释,匹配工具:mypy,请Committer检视其合理性。

likedislike
ascend-robot
ascend-robot成员
5月17日 评论:

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 李伟, renyujin (2/2) 李伟 (1/1)
test renyujin, 李伟 (2/2) 李伟 (1/1)
torch_npu/_inductor 李伟, renyujin (2/2) 李伟 (1/1)
torch_npu/csrc/distributed 李伟, renyujin (2/2) 李伟 (1/1)
torch_npu/distributed 李伟, renyujin (2/2) 李伟 (1/1)
torch_npu/dynamo 李伟, renyujin (2/2) 李伟 (1/1)
torch_npu/npu 李伟, renyujin (2/2) 李伟 (1/1)
torch_npu/utils renyujin, 李伟 (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

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

likedislike
此处折叠了82条消息 查看更多
ascend-robot
ascend-robot成员
5月18日 评论:

Review Guide

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

likedislike
ascend-robotascend-robot成员
5月18日 合入了pull request
Jingwei HuangJingwei Huang成员
5月18日 修改了pull request 的描述
Jingwei HuangJingwei Huang成员
5月18日 修改了pull request 的描述
Jingwei HuangJingwei Huang成员
5月18日 修改了pull request 的描述