已合并
fix:数据类型枚举值的错误映射 #681
fix:数据类型枚举值的错误映射 #681
已合并
Saturday22266创建于 16 天前
Saturday22266
Saturday22266
16 天前

描述

修复 skills/ascendc-ops-project/SKILL.mdPart 10:数据类型枚举值 的错误映射。

原文中部分 DT_* 枚举值与 CANN 数据类型定义不一致:

  • DT_INT6449
  • DT_UINT854
  • DT_BOOL612

修正后与 CANN 中对应的数据类型枚举定义保持一致,避免根据该表生成 dtype 判断或 Kernel 分发逻辑时出现类型匹配错误。

关联的Issue

Closes #421

测试

  • 对照 CANN 头文件 graph/c_types.h 中的数据类型枚举定义进行核对。
  • 确认修改后的对应关系为:
    • DT_FLOAT = 0
    • DT_FLOAT16 = 1
    • DT_INT8 = 2
    • DT_INT32 = 3
    • DT_UINT8 = 4
    • DT_INT64 = 9
    • DT_BOOL = 12
  • 本次仅修改 Skill 文档中的枚举值说明,不涉及算子实现逻辑变更。

文档更新

修改文件:

  • skills/ascendc-ops-project/SKILL.md

类型标签

其他信息

该表可能被用于生成 dtype 判断或 Kernel 类型分发代码。错误的枚举值可能导致数据类型匹配到错误的 Kernel 实现,因此对相关枚举值进行修正。

likedislike
Pull Request已成功合入, 合并人@CANN-robot
(感谢 Saturday22266 的贡献)
Saturday22266Saturday22266
16 天前 创建了 pull request,commit 768f8888
atomgit-bot
atomgit-bot
16 天前 评论:

变更摘要

本次变更修复了 skills/ascendc-ops-project/SKILL.md 中 Part 10 数据类型枚举值的错误映射,使 DT_INT64DT_UINT8DT_BOOL 三个枚举值与 CANN 头文件 graph/c_types.h 中的定义保持一致,避免基于该表生成 dtype 判断或 Kernel 分发逻辑时出现类型匹配错误。

主要改动

  • 修正 DT_INT64 枚举值:将文档中 DT_INT64 的映射值从 4 更正为 9,与 CANN 实际定义对齐
  • 修正 DT_UINT8 枚举值:将文档中 DT_UINT8 的映射值从 5 更正为 4,与 CANN 实际定义对齐
  • 修正 DT_BOOL 枚举值:将文档中 DT_BOOL 的映射值从 6 更正为 12,与 CANN 实际定义对齐
likedislike
atomgit-bot
atomgit-bot
16 天前 评论:

🤖 本次改动均为数据 / 生成 / 二进制文件(如测试用例、锁文件、媒体资源),没有可审查的代码逻辑,已跳过 AI 代码审查。

likedislike
CANN-robotCANN-robot成员
16 天前 添加了label:cann-cla/yes
CANN-robot
CANN-robot成员
16 天前 评论:

Thanks for your pull-request.
The full list of commands accepted by me can be found at here
You can get sig-info at here


PR Approval Progress

Congratulations! All modules have met the lgtm and approve requirements.

Module Approval Details

module lgtm status approve status
repo-cann/cann-learning-hub waimaidaole, fuyangchenghu (2/2) waimaidaole (1/1)

💡 Tip:

  • Committer can comment /approve or /lgtm
  • Commenting /approve implies both code review (lgtm) and intent to merge (approve)

CLA Signature Pass

Saturday22266, thanks for your pull request. All authors of the commits have signed the CLA. 👍

likedislike
Saturday22266Saturday22266
16 天前 修改了pull request 的描述
meme成员
15 天前 评论:

/lgtm
/approve

likedislike
CANN-robotCANN-robot成员
15 天前 添加了label:approved
fuyangchenghu成员
15 天前 评论:

/lgtm

likedislike
CANN-robotCANN-robot成员
15 天前 添加了label:lgtm
CANN-robotCANN-robot成员
15 天前 合入了pull request