已合并
[Bugfix] support CANN version with beta string #3493
[Bugfix] support CANN version with beta string #3493
已合并
Ganon创建于 17 天前
Ganon
Ganon成员
17 天前

Bugfix: support CANN version with beta string like 9.2.0.beta1

likedislike
Pull Request已成功合入, 合并人@ascend-robot
(感谢 Ganon 的贡献)
GanonGanon成员
17 天前 创建了 pull request,commit eb735732
atomgit-bot
atomgit-bot
17 天前 评论:

变更摘要

本次变更旨在修复 torchair/npu_graph_executor/npu_utils.cppVersionToNum 函数对 CANN 测试版(beta)版本号解析的支持问题。此前该函数无法识别形如 major.minor.release-beta.betaNumber 的版本字符串,导致返回 0L。此次改动新增了对应的正则匹配分支,使 beta 版本号能够被正确解析为数值表示。

主要改动

  • 新增 beta 版本号正则匹配分支: 在 VersionToNum 函数中增加 else if 分支,使用正则表达式 ([0-9]+)\.([0-9]+)\.([0-9]+)-beta\.([0-9]+) 匹配 CANN beta 版本号格式,并提取 majorminorrelease 三个版本字段进行解析,使 beta 版本不再因格式不识别而返回 0L
likedislike
atomgit-bot
atomgit-bot
17 天前 评论:

代码审查

✅ 未发现问题

likedislike
ascend-robotascend-robot成员
17 天前 添加了label:smoke-pipeline-running
ascend-robotascend-robot成员
17 天前 添加了label:ascend-cla/yes
此处折叠了84条消息 查看更多
XDaoHong
XDaoHong成员
16 天前 评论:

/approve

likedislike
ascend-robotascend-robot成员
16 天前 添加了label:approved
fu_hao
fu_hao成员
16 天前 评论:

/approve

likedislike
ascend-robotascend-robot成员
16 天前 添加了label:lgtm
ascend-robotascend-robot成员
16 天前 合入了pull request