Closed
[Documentation|文档反馈]: CeluV2 Proto 注释遗漏 BF16 输入类型 #6840
KaranocaVeAdd on  24 days agoClosed at  23 days ago
KaranocaVe
KaranocaVe
24 days ago Create

[Documentation|文档反馈]: CeluV2 Proto 注释遗漏 BF16 输入类型

现象

activation/celu_v2/op_graph/celu_v2_proto.h 将 CeluV2 的 x 输入类型说明为 float16、float32,但同一 REG_OP(CeluV2) 的输入和输出类型集合均包含 DT_BF16。

预期与实际

预期:Proto 接口说明列出已注册支持的输入类型。
实际:注释遗漏 bfloat16,与注册的输入类型集合不一致。

影响

依据 Proto 注释核对接口的调用方会得到比实际注册能力更窄的 dtype 合同,可能误以为 BF16 不受支持。

修复

在 x 的类型注释中补充 bfloat16;不修改类型注册或执行实现。

验证

  • 静态核对确认 Proto 注释、注册定义和 OpDef 均包含 BF16;修复后输入类型说明与注册集合一致。

关联 PR: https://gitcode.com/cann/ops-nn/merge_requests/12208

likedislike
KaranocaVeKaranocaVe
24 days ago Associated pull request
KaranocaVeKaranocaVe
24 days ago Modify description of issue
yuning_chenyuning_chenCollaborators
24 days ago Assign KaranocaVe as responsible person
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/12208

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

感谢您的支持!

likedislike