Closed
[Documentation|文档反馈]: aclnnKthvalue 文档将 k 的合法范围下界误写为 0 #6506
KaranocaVeAdd on  Oct 12Closed at  23 days ago
KaranocaVe
KaranocaVe
Oct 12 Create

[Documentation|文档反馈]: aclnnKthvalue 文档将 k 的合法范围下界误写为 0

index/kth_value/docs/aclnnKthvalue.md 将 k 的范围写为 [0, self.size(dim)],并只把 k < 0 或超过维度长度描述为错误,因此读者会认为 k=0 有效。实现中的 CheckParamValid 会拒绝 k <= 0 和 k > self.size(dim),算子 README 也将范围写为 [1, x.shape[dim]]。

请将 API 文档的合法范围和错误说明统一为 [1, self.size(dim)]。该修正只澄清文档,不改变算子行为。

Validation

验证记录

  • index/kth_value/README.md 将 k 定义为 [1, x.shape[dim]]。
  • aclnn_kthvalue.cpp 的 CheckParamValid 在 k <= 0 或 k > self.size(dim) 时返回参数错误;后续计算以 k - 1 定位元素。
  • 文档补丁在当前 master 基线上通过 git apply --check --whitespace=error-all,源码工作树 git diff --check 通过。
  • 未运行设备侧测试;本修复只澄清文档,参数范围由源码校验逻辑确认。

证据等级:E2(源码校验与 README 交叉核对)。

likedislike
yuning_chen
yuning_chenCollaborators
Oct 12 Comments:

初步分析

感谢反馈,我们已开始核对这个问题。

现象理解:aclnnKthvalue 文档把 k 的下界写成 0,实现和 README 可能只接受从 1 开始。
当前判断:初步判断,待复现确认。k=0 是否会被参数校验拒绝,需对照当前文档和校验代码核实。
涉及范围:aclnnKthvalue 文档中 k 的取值范围。
下一步:我们会核对合法范围说明,评估改成从 1 到对应维度长度。

likedislike
yuning_chen
yuning_chenCollaborators
Oct 12 Comments:

您好,问题已收到,当前问题 @ConanHuang 正在处理中。

likedislike
黄黄晓彬Collaborators
Oct 12 Assign ConanHuang as responsible person
黄黄晓彬Collaborators
23 days ago Associated pull request
CANN-robotCANN-robotCollaborators
23 days ago Close issue
CANN-robotCANN-robotCollaborators
23 days ago Added label: resolved
KaranocaVe
KaranocaVe
21 days ago Comments:

@yolic 您好,这个 Issue 对应的修复已经合入:https://gitcode.com/cann/ops-nn/merge_requests/12372

开发者计划反馈,Issue 需要有 Accepted 标签才能计入积分,仅关闭或标记 resolved 还不够。目前这条 Issue 尚未标记 Accepted,麻烦您结合问题和已合入的修复复核一下;确认符合受理条件后,帮忙补上 Accepted 标签。

感谢您的支持!

likedislike