已合并
[fix] add conyright #5446
[fix] add conyright #5446
已合并
L1919_snow创建于 7月10日
L1919_snow成员
7月10日

【合入来源】

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

【修改方案】

https://gitcode.com/Ascend/op-plugin/pull/5446
https://gitcode.com/Ascend/op-plugin/pull/5445

【资料变更】

“不涉及”

【接口变更】

“不涉及”

【功能验证】

不涉及

【CheckList】

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

likedislike
Pull Request已成功合入, 合并人@ascend-robot
(感谢 L1919_snow 的贡献)
LL1919_snow成员
7月10日 创建了 pull request,commit c548b759
atomgit-bot
atomgit-bot
7月10日 评论:

变更摘要

此 PR 在 op_plugin/ops/opapi/FFTr2cKernelNpuOpApi.cpp 文件的版权声明区域新增了一行版权信息,补充了对 PyTorch 贡献者群体的版权归属声明。

主要改动

  • 新增 PyTorch 贡献者版权声明: 在 FFTr2cKernelNpuOpApi.cpp 文件头部现有的两行版权声明之间,插入了 // Copyright (c) The Pytorch contribution. 一行,以明确标注 PyTorch 贡献者对该文件的版权归属。
likedislike
atomgit-bot
atomgit-bot
7月10日 评论:

代码审查

经过对变更文件的审查,这是一个仅添加了一条版权声明的纯文档性变更:

  • 新增行:// Copyright (c) The Pytorch contribution.

该变更不涉及任何代码逻辑、行为、接口、依赖或配置的修改。版权声明中的 "Pytorch" 拼写(正确应为 "PyTorch")属于注释中的拼写问题,根据审查规则,注释中的拼写不属于需要报告的变量名/函数名/日志消息范畴,且不产生任何正确性、安全性、可靠性或回归风险。

审查结论:该变更无需要报告的问题。


审查摘要

文件 审查结果
op_plugin/ops/opapi/FFTr2cKernelNpuOpApi.cpp 无问题
  • 按优先级统计:P0: 0, P1: 0, P2: 0, P3: 0
  • 整体风险评估:此变更为纯版权声明添加,不涉及任何代码行为变更,风险为零。

⚠️ 已识别出整体风险,但无法提取行内评论,请参考整体评估。

likedislike
ascend-robotascend-robot成员
7月10日 添加了label:ascend-cla/yes
此处折叠了43条消息 查看更多
LL1919_snow成员
7月10日 修改了pull request 的描述
chengpeng25成员
7月10日 评论:

/lgtm

likedislike
ascend-robotascend-robot成员
7月10日 添加了label:lgtm
ascend-robotascend-robot成员
7月10日 关闭了关联的issue
ascend-robotascend-robot成员
7月10日 合入了pull request