已合并
fix: correct NetOutput type comparison in DAG adapter #4625
fix: correct NetOutput type comparison in DAG adapter #4625
已合并
KenChow创建于 27 天前
KenChow成员
27 天前

Pull Request

描述

  • 修复 DAGAdapter::RefreshStreamIdsToGE 中 NetOutput 节点类型判断错误。
  • GetTypePtr() 返回 const char *,原实现使用指针与字符串字面量比较,无法可靠识别 NetOutput;改为按字符串内容比较,确保 NetOutput 节点被正确过滤。
  • 新增 RefreshStreamIdsToGE_SkipNetOutput UT,验证普通节点 stream id 正常刷新,同时 NetOutput 保持原始 stream id。

变更类型

请选择本次引入的变更类型:

关联的Issue

无

如何测试

描述测试此变更的步骤和前提条件:

  1. 新增 UT RefreshStreamIdsToGE_SkipNetOutput,设置普通节点和 NetOutput 的原始 stream id 为有效值,验证刷新结果符合预期。
  2. 按需求未进行本地编译。
  3. 通过 PR 评论 compile 触发 GitCode CI;现有 UT/ST 随流水线执行通过,流水线 #1098123 已成功完成。

核对清单

其他信息

本次未新增 ST;未进行本地编译,最终以 GitCode CI 验证结果为准。

likedislike
Pull Request已成功合入, 合并人@CANN-robot
(感谢 KenChow 的贡献)
KKenChow成员
27 天前 创建了 pull request,commit 311f1bee
CANN-robot
CANN-robot成员
27 天前 评论:

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.
You can self-configure the PR merge rules for this repository. For more details, please refer to Here.
For more, you also can visit HICANN.


PR Approval Progress

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

Module Approval Details

module lgtm status approve status
repo-cann/ge ✅ 张德鹏, ClarkXie, Mexyy (3/2) ✅ 张德鹏 (1/1)

💡 Tip:

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

CLA Signature Pass

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

likedislike
CANN-robotCANN-robot成员
27 天前 添加了label:cann-cla/yes
CANN-robotCANN-robot成员
27 天前 将yangyongqiang0606,kobemini,ClarkXie,tangqunzhang,xchu42,hugo111,lining23666,fu-jun2,depeng1994,wqtshg_wt,peiyang,shengnan666,zhangfan_hanq,zhanj,zhujingjing,Mexyy,wangxiaotian995,stevenaw0,yuht9设为评审人
CANN-robotCANN-robot成员
27 天前 将yangyongqiang0606,kobemini,ClarkXie,tangqunzhang,xchu42,hugo111,lining23666,fu-jun2,depeng1994,wqtshg_wt,peiyang,shengnan666,zhangfan_hanq,zhanj,zhujingjing,Mexyy,wangxiaotian995,stevenaw0,yuht9设为审查人
此处折叠了57条消息 查看更多
CANN-robotCANN-robot成员
26 天前 添加了label:lgtm
KKenChow成员
26 天前 修改了pull request 的描述
张德鹏成员
26 天前 评论:

/approve

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