已关闭
[Bug-Report|缺陷反馈]: activation/clipped_swiglu/op_host/clipped_swiglu_def.cpp 变量命名不规范 #1788
huanglei92创建于 3月28日关闭于 4月1日
jiangjiawei
3月31日 评论:
3月31日 评论:
/assign @fengqiuyue


CANN-robot
3月31日 评论:
3月31日 评论:
3月31日 关联了pull request:clippedswiglu删除重复set
4月1日 关闭了 issue
4月1日 添加了label:resolved
Thanks for sending an issue! Please fill in the following template to help quickly solve your problem.
Describe the current behavior / 问题描述 (Mandatory / 必填)
代码检视过程中,发现cann/ops-nn 库master分支,存在变量命名拼写错误;
https://gitcode.com/cann/ops-nn/blob/master/activation/clipped_swiglu/op_host/clipped_swiglu_def.cpp
18 19行
constexpr float DEFALUT_ALPHA = 1.702;
constexpr float DEFALUT_LIMIT = 7.0;
变量命名存在拼写错误,DEFALUT_ALPHA 和 DEFALUT_LIMIT 应为 DEFAULT_ALPHA 和 DEFAULT_LIMIT。这是一个明显的错误,虽然不影响功能,但影响代码可读性和专业性,影响昇腾的口碑
Environment / 环境信息 (Mandatory / 必填)
https://gitcode.com/cann/ops-nn/blob/master/activation/clipped_swiglu/op_host/clipped_swiglu_def.cpp
Steps to reproduce the issue / 重现步骤 (Mandatory / 必填)
code review
Describe the expected behavior / 预期结果 (Mandatory / 必填)
DEFALUT_ALPHA 和 DEFALUT_LIMIT 应为 DEFAULT_ALPHA 和 DEFAULT_LIMIT
Related log / screenshot / 日志 / 截图 (Mandatory / 必填)
无
Special notes for this issue/备注 (Optional / 选填)