已关闭
[Code Quality] adaptive_avg_pool2d 生产代码中遗留 TODO 注释 #2086
shermanrowena创建于  4月15日关闭于  5月7日
shermanrowena
4月15日 创建

问题描述

在 pooling/adaptive_avg_pool2d/op_kernel/arch35/adaptive_pool2d_big_kernel.h 第 99 行,GetDtypeMinValue 函数定义前有一个孤立的 //TODO 注释,没有任何描述说明该函数还有哪些未完成的工作或待优化点。

问题位置

文件:pooling/adaptive_avg_pool2d/op_kernel/arch35/adaptive_pool2d_big_kernel.h,第 99 行

代码片段

//TODO
template <typename T>
template <typename U>
__aicore__ inline U AdaptivePool2dBigKernel<T>::GetDtypeMinValue() //取min操作
{
    U minValue = 0;
    if constexpr (IsSameType<U, half>::value) { ... }
    else if constexpr (IsSameType<U, bfloat16_t>::value) { ... }
    else { ... }
    return minValue;
}

影响

  • 遗留的 TODO 注释使读者无法判断该函数是否已完整实现,增加维护负担;
  • 如果该 TODO 对应真实的遗留工作(如缺少 INT32 最小值支持),则存在功能缺失风险。

修复建议

  1. 如函数已完整实现,删除 //TODO 注释;
  2. 如确有未完成工作,补充具体描述,例如:// TODO(user): 补充 int32_t 等整型的最小值处理,并创建追踪任务。

严重程度

低(代码质量 / 可维护性问题)

likedislike
Cchenqi317成员
4月24日 将 chaotang233 设为负责人
oscillatedoscillated成员
4月30日 移除了负责人 chaotang233
oscillated
oscillated成员
4月30日 评论:

您好,感谢反馈,问题已收到,已分配对应负责同事。
@u010470851

likedislike
sgqt12sgqt12成员
4月30日 关联了pull request:清除AdaptiveAvgPool2d代码中遗留注释
CANN-robot
CANN-robot成员
4月30日 评论:

Notice

This issue can not be assigned to u010470851. Please try to assign to the repository members.

likedislike
zhang-wu成员
5月7日 评论:

/lgtm

likedislike
CANN-robotCANN-robot成员
5月7日 关闭了 issue
CANN-robotCANN-robot成员
5月7日 添加了label:resolved