[Cpp API Compatibility] Normalize adapted-from-PyTorch declarations#78590
Conversation
|
你的PR提交成功,感谢你对开源项目的贡献! |
|
Self-review summary:
@SigureMo 麻烦看下这个单独拆出来的 follow-up PR。 |
|
@ShigureNyako 你扫一下是否有哪些文件你觉得漏加声明的(主要实现参考 PyTorch 的都应该加,如果实现与 PyTorch 不同,只是接口与 PyTorch 相同则无需添加),在这里回复就好,我确认下哪些需要额外加一下 |
我先按“实现里已经直接写了 follow / match / mirror / parity with PyTorch”这个标准粗扫了一遍,强候选我觉得有这些:
这几份文件里已经明确写了“match/follow/mirror/parity with PyTorch”之类的实现语义注释,我倾向于把它们视为“实现层也在参考 upstream”,不只是接口名兼容。 再宽一点看,下面这些也有比较明显的 PyTorch 语义对齐痕迹,但我觉得更像 “Paddle kernel + PyTorch 语义包装”,要不要补声明我暂时放在第二档:
像 如果你觉得上面第一档要继续补,我可以再单独补一轮,把范围尽量收敛在这些“实现参考 upstream 痕迹比较明确”的文件上。 |
第一档都补一下吧,在这个 PR 直接加 |
Co-authored-by: Nyakku Shigure <sigure.qaq@gmail.com>
已按这条意见直接补在当前 PR 里,新增 commit 本次补了这 8 个文件的 adapted-from-PyTorch 声明:
自查结论:
@SigureMo 麻烦再看下这一轮补充是否符合你的预期。 |
|
冲突了,需要解一下 |
…e-pytorch-adapted-from
已处理。我这边把最新 这次同步后,我确认前面补的 adapted-from-PyTorch 声明仍然都在:
本次新增提交的作用主要是把分支 re-sync 到最新 develop;没有再额外扩这条 PR 的目标范围。 @SigureMo 麻烦再看下这一版冲突处理后的结果。 |
|
/re-run all-failed |
PR Category
Execute Infrastructure
PR Types
Improvements
Description
This PR is a follow-up extracted from #78581.
It normalizes malformed adapted-from-PyTorch reference declarations across the existing compat files and Python shim modules so they match the style already used in the latest compatibility updates:
#markersLicensed under BSD-style license -line spacingThis is a comment-only cleanup and does not change ABI, behavior, dispatch logic, or public APIs.
是否引起精度变化
否