Closed
[Documentation|文档反馈]: math/log_add_exp README 称“仅包含 aclnn 接口”,但同目录 op_kernel 已有 3 个实现文件 #3721
liuhanwen8Add on 20 days agoClosed at 15 days ago
20 days ago Added label: documentation
陈思
20 days ago Comments:
20 days ago Comments:
您好,感谢反馈!
核对 cann/ops-math 提交 b19eaee4f58e5e1e3cb3b2ecbfed2d02c11c820a,README仍写“仅包含 aclnn 接口”,但已存在计算实现:log_add_exp_apt.cpp 包含 arch35 DAG 与结构定义,log_add_exp 入口按 formulaType 选择计算分支并调用广播调度器的 Process(x1, x2, y)。因此,这处占位说明与该版本的实际交付内容不一致,不能据此判断 LogAddExp 尚无实现。
一个可行的调整是,在 README 分别说明 aclnn 接口、当前实现的实际位置和样例入口,并将参数、dtype、产品支持约束直接链接到已有接口文档,避免复制两份说明后发生差异。这里需分别保留 aclnnLogAddExp 和 aclnnLogAddExp2 的入口:两份文档分别描述以 e 和以 2 为底的运算,不能只链接其中一份。当前核查到的 arch35 实现不能扩大为所有产品都走同一路径,产品限制仍需保留详细文档的适用范围。
关于 CI 建议,可以考虑检测“明确宣称仅接口”与“存在有效实现入口”的矛盾,同时区分共享实现、空壳文件及无独立 kernel 的目录,控制误报和维护成本;具体规则仍需评估。#3691 的清单可在本项有修订合入依据后同步,当前尚无据此关闭本项的依据。我们会继续确认这项文档调整。


20 days ago Assign yue-ma as responsible person
20 days ago Added label: pending-confirmation
20 days ago Removed label: pending-confirmation
19 days ago Associated pull request
15 days ago Close issue
15 days ago Added label: resolved
问题描述
math/log_add_exp/README.md在当前 HEAD3e45432d仍为占位空壳:去掉 H1 后正文只有 151 字节一句话——本目录仅包含LogAddExp算子对应的aclnn接口;如您想要贡献该算子的AscendC实现,请参考[贡献流程](../../CONTRIBUTING.md)。实际交付件(HEAD
3e45432d)op_api/aclnn_logaddexp.cpp、op_api/aclnn_logaddexp.h、op_api/aclnn_logaddexp2.cpp、op_api/aclnn_logaddexp2.h…共 6 个)op_host/op_host/arch35/log_add_exp_tiling_arch35.cpp、op_host/arch35/log_add_exp_tiling_arch35.h、op_host/config/ascend950/log_add_exp_binary.json…共 5 个)op_kernel/op_kernel/arch35/log_add_exp_dag.h、op_kernel/arch35/log_add_exp_struct.h、op_kernel/log_add_exp_apt.cpp)op_graph/docs/docs/aclnnLogAddExp.md、docs/aclnnLogAddExp2.mdexamples/docs/zh/op_list.md与占位语不符之处
op_kernel/已有 3 个 AscendC 实现文件(入口op_kernel/log_add_exp_apt.cpp),实现真实存在。docs/aclnnLogAddExp.md、docs/aclnnLogAddExp2.md,README 未给出任何docs/入口。修复建议
op_api/aclnn_logaddexp.cpp;文档 docs/aclnnLogAddExp.md)、AscendC 实现(op_kernel/,入口 log_add_exp_apt.cpp)、样例(examples/)。docs/aclnnLogAddExp.md、docs/aclnnLogAddExp2.md,不复制正文,避免两份资料不同步。.gitkeep/空壳文件),“本目录无独立 kernel”不判为缺陷。关联