已合并
修正UT依赖安装与AddExample调测、安装指引 #11986
yangyang016创建于 19 天前
修正UT依赖安装与AddExample调测、安装指引 #11986
已合并
yangyang016创建于 19 天前
yangyang016成员
19 天前

描述

修复安装、UT依赖和AddExample调测指引中的不一致,确保文档与当前代码及依赖配置对应。

  • tests/requirements.txt 新增 -r ../requirements.txt,使UT依赖安装统一包含项目根目录的依赖及版本约束。
  • 安装文档区分仅编译与执行UT两种场景,明确UT依赖已包含根目录依赖,无需重复安装。
  • 更新AddExample的printf示例,移除与当前实现不一致的旧Tiling代码,明确在 Init 中的插入位置,使用 AscendC::printf 和与 int64_t blockLength_ 对应的 %lld。
  • 补充自定义算子包安装时报 shared library validation failed 的处理说明:确认CANN版本与构建环境兼容后,可按需使用 --force 跳过共享库依赖检查;同时说明运行时加载失败风险,不建议默认使用。

关联的Issue

关联Issue #6699:https://gitcode.com/cann/ops-nn/issues/6699

测试

  • 已执行 git diff --check HEAD^ HEAD,通过。
  • 已静态核对UT依赖引用路径,以及printf示例中的变量类型与插入位置。
  • 未执行实际依赖安装、CANN环境下的编译、UT及算子包安装验证,相关验证待补充。

文档更新

  • docs/QUICKSTART.md:更新printf调测示例,补充安装共享库校验失败时的处理说明。
  • docs/zh/install/quick_install.md:区分编译依赖与UT依赖的安装场景。
  • docs/zh/install/compile.md:明确UT依赖包含根目录依赖约束。

类型标签

AI/Agent生成声明

likedislike
Pull Request已成功合入, 合并人@CANN-robot
(感谢 yangyang016 的贡献)
Yyangyang016成员
19 天前 创建了 pull request,commit 49f496db
atomgit-bot
atomgit-bot
19 天前 评论:

变更摘要

该 PR 标题为 fix issue,属于针对已有问题的修复类改动,但所提供的 PR 描述为空(仅保留提交模板内容),且变更文件列表中未附加任何文件路径与补丁内容,因此无法从材料中提炼其涉及的具体模块、接口或行为变化。依据统计信息,本次改动共涉及 4 个文件、新增 31 行、删除 16 行,整体规模较小,符合局部修复的特征。在缺少补丁与描述的情况下,无法进一步确认修复对象、实现方式及影响范围,不建议对其实现细节做任何推断。

主要改动

  • 缺少可分析的补丁内容: 任务材料中的变更文件列表为空白,未内嵌任何文件的补丁,无法定位被修改的函数、代码行或配置项。
  • 缺少业务背景说明: PR 描述未填写改动原因与所采取的方法,无法判断该修复对应的具体问题与影响范围。
  • 可确认的改动规模: 仅能从统计得知本次改动涉及 4 个文件、新增 31 行、删除 16 行,属于小范围调整。

🧩 本次 Review 已使用自定义 Skill:

likedislike
不准确?
atomgit-bot
atomgit-bot
19 天前 评论:

代码审查

✅ 未发现问题


🧩 本次 Review 已使用自定义 Skill:

likedislike
不准确?
CANN-robotCANN-robot成员
19 天前 添加了label:cann-cla/yes
CANN-robot
CANN-robot成员
19 天前 评论:

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
docs ✅ 陈娇, 唐玮玮 (2/2) ✅ 陈娇 (1/1)
tests ✅ 刘波, 唐玮玮 (2/2) ✅ 刘波, 唐玮玮 (2/1)

💡 Tip:

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

CLA Signature Pass

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

likedislike
此处折叠了46条消息 查看更多
CANN-robotCANN-robot成员
18 天前 关闭了关联的issue
CANN-robotCANN-robot成员
18 天前 合入了pull request
CANN-robot
CANN-robot成员
18 天前 评论:

Pull Request 已合并或已关闭。

If you want to solve this problem, you can click here to do it in the FAQs.

likedislike
chenqi317成员
18 天前 评论:

/lgtm
/approve

likedislike
yuning_chen
yuning_chen成员
18 天前 评论:

您好,pr合入已准备就绪,请尽快联系committer进行检视,谢谢!

likedislike