已合并
支持下一代cos_stateless_bernoulli_stateless_random_normal_v2算子实现 #548
hawdonz创建于 2025年12月29日
支持下一代cos_stateless_bernoulli_stateless_random_normal_v2算子实现 #548
已合并
Pull Request已成功合入, 合并人@CANN-robot
（感谢 hawdonz 的贡献）
CANN-robot
2025年12月29日 评论：
2025年12月29日 评论：
CANN-robot
2025年12月29日 评论：
2025年12月29日 评论：
以下是根据您提交的修改文件推荐的Reviewer和Committer序列，需各模块评审通过后方可合入
| Module List | Reviewers | Committers |
|---|---|---|
| math/cos | N/A | zhou-qilong, loov1, wangrui_, songkai111, rxtfeng |
| repo-cann/ops-math | N/A | zhou-qilong, gubaocheng, loov1, wangrui_, songkai111 |
2025年12月29日 添加了label：cann-cla/yes
CANN-robot
2025年12月29日 评论：
2025年12月29日 评论：
🔵 source code change are detected, tasks labels is removed in this pull request!
math/cos/op_api/aclnn_cos.h
已过期
@@ -0,0 +62,4 @@
62+/**
63+ * @brief aclnnInplaceCos的第一段接口，根据具体的计算流程，计算workspace大小。
64+ * @domain aclnn_math
65+ * 功能描述：从输入矩阵的每一个元素进行反余弦操作后输出。
文档错误: 函数aclnnInplaceCosGetWorkspaceSize的注释中，功能描述和计算公式存在明显错误。功能描述写的是'从输入矩阵的每一个元素进行反余弦操作后输出'，计算公式写的是'out_{i}=tan^{-1}(input_{i})'。这与其函数名'aclnnInplaceCos'（原地余弦）以及同一文件中aclnnCos函数的描述（求余弦）相矛盾。这会导致文档与代码功能不一致，误导接口使用者。
问题类型: 文档错误
文件路径: math/cos/op_api/aclnn_cos.h
行号: 65
问题代码:
* 功能描述：从输入矩阵的每一个元素进行反余弦操作后输出。
* 计算公式：
* out_{i}=tan^{-1}(input_{i})
修改建议:
将功能描述和计算公式修改为与函数名'aclnnInplaceCos'以及'aclnnCos'函数一致的余弦操作。建议修改为：
* 功能描述：对输入Tensor的每一个元素进行原地余弦操作。
* 计算公式：
* input_{i}=cos(input_{i})
此评论由代码审查工具自动生成
系统消息
2025年12月29日 评论：
2025年12月29日 评论：
changed this line on feb7a7ed in version 5 of the diff
此处折叠了125条消息 查看更多
CANN-robot
2025年12月30日 评论：
2025年12月30日 评论：
The MR can not be merged, because of CodeReview discussion not resolved
If you want to solve this problem, you can click here to do it in the FAQs.
2025年12月30日 解决了最后一个问题
CANN-robot
2025年12月30日 评论：
2025年12月30日 评论：
Review Guide
This Pull-Request Passes Review.
Committers who writed a comment of /approve are: wangrui_.
Reviewers who writed a comment of /lgtm are: songkai111, wangrui_.
2025年12月30日 关闭了关联的issue
2025年12月30日 合入了pull request
描述
关联的Issue
测试
文档更新
类型标签