已关闭
[Documentation|文档反馈]: math/atan2 README 称“仅包含 aclnn 接口”,但同目录 op_kernel 已有 3 个实现文件 #3712
liuhanwen8创建于 9 天前关闭于 4 天前
9 天前 添加了label:documentation
陈思
9 天前 评论:
9 天前 评论:
您好,感谢反馈!
核对 cann/ops-math 提交 b19eaee4f58e5e1e3cb3b2ecbfed2d02c11c820a,README仍写“仅包含 aclnn 接口”,但同目录已存在真实的 AscendC 入口 atan2_apt.cpp:它包含 arch35/atan2_dag.h 与 atan2_struct.h,入口通过 BroadcastSch 的 Process(x1, x2, y) 执行双输入计算。因此,这处占位说明确实与该版本的交付内容不一致,不能据此判断 Atan2 尚无实现。
一个可行的调整是,README 分别给出 aclnn 接口、当前 arch35 实现和样例的入口;参数、dtype 及产品支持范围直接链接到已有的接口文档,避免在两处复制后发生差异。arch35 实现存在也不意味着所有产品采用同一实现路径,产品与接口约束仍应按详细文档说明。
关于 CI 建议,可以考虑只检测“明确宣称仅接口”与“实际有效 kernel 入口”之间的矛盾,并区分共享实现、空壳文件与无独立 kernel 的目录,降低误报和维护成本。该校验规则仍需评估。#3691 的清单状态可在本项修订有合入依据后同步;当前尚无据此关闭本项的依据。我们会继续确认这项文档调整。


9 天前 将 yue-ma 设为负责人
9 天前 添加了label:pending-confirmation
9 天前 删除了label:pending-confirmation
7 天前 关联了pull request:增加atan2算子资料
4 天前 关闭了 issue
4 天前 添加了label:resolved
问题描述
math/atan2/README.md在当前 HEAD3e45432d仍为占位空壳:去掉 H1 后正文只有 147 字节一句话——本目录仅包含Atan2算子对应的aclnn接口;如您想要贡献该算子的AscendC实现,请参考[贡献流程](../../CONTRIBUTING.md)。实际交付件(HEAD
3e45432d)op_api/aclnn_atan2.cpp、op_api/aclnn_atan2.h、op_api/atan2.cpp、op_api/atan2.h)op_host/op_host/arch35/atan2_tiling_arch35.cpp、op_host/arch35/atan2_tiling_arch35.h、op_host/atan2_def.cpp…共 6 个)op_kernel/op_kernel/arch35/atan2_dag.h、op_kernel/arch35/atan2_struct.h、op_kernel/atan2_apt.cpp)op_graph/op_graph/atan2_proto.hdocs/docs/aclnnAtan2&aclnnInplaceAtan2.mdexamples/docs/zh/op_list.md与占位语不符之处
op_kernel/已有 3 个 AscendC 实现文件(入口op_kernel/atan2_apt.cpp),实现真实存在。docs/aclnnAtan2&aclnnInplaceAtan2.md,README 未给出任何docs/入口。修复建议
op_api/aclnn_atan2.cpp;文档 docs/aclnnAtan2&aclnnInplaceAtan2.md)、AscendC 实现(op_kernel/,入口 atan2_apt.cpp)、样例(examples/)。docs/aclnnAtan2&aclnnInplaceAtan2.md,不复制正文,避免两份资料不同步。.gitkeep/空壳文件),“本目录无独立 kernel”不判为缺陷。关联