ops-math/math/cdist/op_api · SeanDictionary/ops-math

Files	Last commit	Last update
aclnn_cdist.cpp	修复issue的问题 Co-authored-by: sakuraqqz<furao15@h-partners.com> # message auto-generated for no-merge-commit merge: !6812 merge master into master 修复issue的问题 Created-by: sakuraqqz Commit-by: sakuraqqz Merged-by: cann-robot Description: ## 描述本 PR 修复 6 个算子问题，均为 Bug 修复（含 1 项文档修正）： 1. is_inf 尾块越界读写（#3925）：数据长度不足 32B 对齐时，尾块原先按 32B 整倍搬运导致 GM 越界读/写。修复为整块用 `DataCopy`、尾块用 `DataCopyPad` 按真实字节搬运，`ComputePerCore` 尾块输出按真实计数写出。`math/is_inf/op_kernel/is_inf.h`。 2. addcmul value=NaN 走错快路径（#3894）：`IsEqualToOne` 对 NaN 判等为 true（所有比较均 false 取反），导致 `value=NaN` 时误走 AxpyV2 快路径、缺失 value 项。修复为显式排除 NaN。`math/addcmul/op_api/aclnn_addcmul.cpp`。 3. cdist 非法 shape 返回码（#3929）：out 形状与推导不匹配时返回内部空指针错误 561103，修复为在 GetWorkspaceSize 增加 out 期望形状校验，返回标准参数错误 161002。`math/cdist/op_api/aclnn_cdist.cpp`。 4. div 非法参数返回码（#3962）：日志错误码与返回码不一致，统一返回 161002。`math/div/op_api/aclnn_div.cpp`。 5. VarMean 文档矛盾（#4182）：中英文档 `self` 维度列由 `1-8` 修正为 `0-8`（与「shape 支持 0 到 8 维」及 `meanOut`/`varOut` 的 `0-8` 保持一致）；参数表 `varOut`/`meanOut` 行序与原型（varOut 为第 5 参、meanOut 为第 6 参）对齐。`math/reduce_var/docs/`。 6. LinSpace legacy tiling 丢失 INT64 num 精度（#3347）： `LinSpaceGetConstValue` 的 `DT_INT64` 分支先将 `int64_t` 经 `float` 转换，`num=16777219` 被舍入为 `16777220`，与 InferShape 输出维度不一致；修复为直接读取 int64 数值、经 double 无损保留。`math/lin_space/op_host/lin_space_tiling.cpp`。 ## 关联的Issue - https://gitcode.com/cann/ops-math/issues/3925 - https://gitcode.com/cann/ops-math/issues/3894 - https://gitcode.com/cann/ops-math/issues/3929 - https://gitcode.com/cann/ops-math/issues/3962 - https://gitcode.com/cann/ops-math/issues/4182 - https://gitcode.com/cann/ops-math/issues/3347 ## 测试 910B3 / CANN 9.3.0 上板验证（2026-10-29）： - is_inf：尾块用例长度 17/25/31/33/40（非 32B 对齐），尾部放置 inf/-inf/nan，输入输出 buffer 后 64B 金丝雀检测越界，5 组全部 PASS 且无越界读写。 - addcmul：value=NaN → out=NaN（修复前误得 7.0）；value=1.0/2.0 回归正常。 - cdist：out 形状不匹配返回 161002，合法形状返回 0。 - div：shape 不匹配/非法 mode 均返回 161002。 - VarMean：issue 复现脚本实测命中 0 处（纯文档改动，无需上板）。 - LinSpace：tiling data 中 num=16777219 保留、InferShape 输出 [16777219]（num=17 对照），Host UT 6/6 PASS，基线回归证实旧实现舍入为 16777220。 ## 文档更新 - `math/reduce_var/docs/aclnnVarMean.md`、`math/reduce_var/docs/aclnnVarMean_en.md`（维度范围与参数顺序修正）。 ## 类型标签 - [x] Bug修复 - [ ] 新特性 - [ ] 性能优化 - [x] 文档更新 - [ ] 其他，请描述： See merge request: cann/ops-math!6812	10 days ago
aclnn_cdist.h	reduce all Co-authored-by: lu-yukun<luyukun4@h-partners.com> # message auto-generated for no-merge-commit merge: !888 merge reduce_final into master reduce all Created-by: lu-yukun Commit-by: lu-yukun Merged-by: cann-robot Description: ## 描述 <!--在这里详细描述你的改动，包括改动的原因和所采取的方法。--> ## 关联的Issue <!-- 如果这个PR是为了解决特定的Issue，请在这里提供Issue链接。--> <!-- 如果这个PR是为了解决特定的问题单，请在这里描述问题单单号。--> ## 测试 <!--描述进行了哪些测试来验证你的改动。包括但不限于二级冒烟、算子泛化等。--> ## 文档更新 <!--如果这个PR包含文档的更新，请在这里指出。例如：更新了README.md文件。--> ## 类型标签 <!-- [x] 表示选中 --> - [ ] Bug修复 - [ ] 新特性 - [ ] 性能优化 - [ ] 文档更新 - [ ] 其他，请描述： See merge request: cann/ops-math!888	9 months ago
cdist.cpp	Fix cdist error code and infer_shape check (#2967) Co-authored-by: gcw_LxlU8H9L<opengoodai@163.com> # message auto-generated for no-merge-commit merge: !5214 merge fix/cdist-2967-error-code-infer-shape into master Fix cdist error code and infer_shape check (#2967) Created-by: gcw_LxlU8H9L Commit-by: OpenGood_WeChat;gcw_LxlU8H9L Merged-by: cann-robot Description: ## Summary Fix two issues in the cdist operator found during the CANN ops scan (see #2967): 1. Invalid dtype error code: `math/cdist/op_api/cdist.cpp` — when the operator encounters an unsupported dtype, it silently returns `nullptr` instead of reporting an error. Changed to return `ACLNN_ERR_PARAM_INVALID` with a proper error log. 2. `INFER_SHAPE` return value not checked: the return value of `INFER_SHAPE` in the `RegBase` path was ignored, which can mask shape-inference failures. Added a check and `OP_LOGE` logging on failure. Also applied to `math/cdist_grad/op_api/cdist_grad.cpp` (same patterns). ## Changes \| File \| Change \| \|---\|---\| \| `math/cdist/op_api/cdist.cpp` \| +13 / -3 \| \| `math/cdist_grad/op_api/cdist_grad.cpp` \| +1 / -1 \| ## Testing - Local verification on CANN 9.1.0 (aarch64): all three modified files compile clean against the real toolkit headers; the full op_api UT target links successfully. - Ran the full `l2_cdist_test` suite (37 cases) with the fix and against the baseline commit `76c8d878`: all 37 tests PASS in both* — zero regression, no segfault, no new failure introduced by this change. - UT stub semantics unchanged; assertion lines untouched. Closes #2967 See merge request: cann/ops-math!5214	1 month ago
cdist.h	reduce all Co-authored-by: lu-yukun<luyukun4@h-partners.com> # message auto-generated for no-merge-commit merge: !888 merge reduce_final into master reduce all Created-by: lu-yukun Commit-by: lu-yukun Merged-by: cann-robot Description: ## 描述 <!--在这里详细描述你的改动，包括改动的原因和所采取的方法。--> ## 关联的Issue <!-- 如果这个PR是为了解决特定的Issue，请在这里提供Issue链接。--> <!-- 如果这个PR是为了解决特定的问题单，请在这里描述问题单单号。--> ## 测试 <!--描述进行了哪些测试来验证你的改动。包括但不限于二级冒烟、算子泛化等。--> ## 文档更新 <!--如果这个PR包含文档的更新，请在这里指出。例如：更新了README.md文件。--> ## 类型标签 <!-- [x] 表示选中 --> - [ ] Bug修复 - [ ] 新特性 - [ ] 性能优化 - [ ] 文档更新 - [ ] 其他，请描述： See merge request: cann/ops-math!888	9 months ago