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

Pull Request

描述

  • 修复 DAGAdapter::RefreshStreamIdsToGENetOutput 节点类型判断错误。
  • 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成员
9 天前 创建了 pull request,commit 311f1bee
CANN-robot
CANN-robot成员
9 天前 评论:

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成员
9 天前 添加了label:cann-cla/yes
CANN-robotCANN-robot成员
9 天前 将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成员
9 天前 将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成员
8 天前 添加了label:lgtm
KKenChow成员
8 天前 修改了pull request 的描述
张德鹏成员
8 天前 评论:

/approve

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