已合并
修改验证 reference_practice/pytorch_online_inference_operator_optimize #462
meme创建于 17 天前
修改验证 reference_practice/pytorch_online_inference_operator_optimize #462
已合并
meme创建于 17 天前
已删除 :test合入到cann/cann-learning-hubtest
meme成员
17 天前

描述

关联的Issue

测试

文档更新

类型标签

  • Bug修复
  • 新特性
  • 内容优化
  • 其他,请描述:

其他信息

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

变更摘要

此 PR 主要对 reference_practice/pytorch_online_inference_operator_optimize/ 目录下的多个 Jupyter Notebook 实训文件进行了环境适配性修改:将 modelscope 的安装方式从初始的 pip3 install 命令中分离出来,改为条件检测后按需安装,同时将 Notebook 内核从 MindSpore 切换为标准的 Python 3python3),并更新了对应的 Python 版本号。

主要改动

  • modelscope 安装方式调整:在 02_pytorch_profiling_tool_usage.ipynb04_pytorch_op_extension_develop_and_apply.ipynb05_chapter_practice.ipynb 中,将 modelscope 从初始的 pip3 install 一行命令中移除,改为通过 importlib.util.find_spec("modelscope") 检测是否已安装,若未安装则通过 subprocess.run 单独执行安装,避免因 modelscope 安装失败影响 transformers 等其余依赖的安装。
  • Notebook 内核元数据更新:在 02_pytorch_profiling_tool_usage.ipynb04_pytorch_op_extension_develop_and_apply.ipynb05_chapter_practice.ipynb 中,将 kernelspecdisplay_nameMindSpore 改为 Python 3namemindspore 改为 python3,统一使用标准 Python 内核。
  • Python 版本号更新:在上述三个 Notebook 的 language_info 元数据中,将 version3.9.10 更新为 3.11.4
  • 03_operator_develop.ipynb 大范围内容修订:该文件有约 973 行新增和 971 行删除,属于较大幅度的内容调整,但本次未提供具体补丁内容。
likedislike
atomgit-bot
atomgit-bot
17 天前 评论:

代码审查

审查总结

优先级 数量
P0 0
P1 0
P2 0
P3 2

各文件审查结果:

  • 02_pytorch_profiling_tool_usage.ipynb:1 个 P3 问题(execution_count 泄露)
  • 03_operator_develop.ipynb:无问题(diff 内容因体积过大被截断,但当前工作区文件状态干净:所有 execution_count 为 null、无 MindSpore 残留、kernel/Python 版本与其余文件一致)
  • 04_pytorch_op_extension_develop_and_apply.ipynb:1 个 P3 问题(两处 execution_count 泄露)
  • 05_chapter_practice.ipynb:无问题

整体风险判断:低。 本次改动主要是四份 notebook 的元数据统一更新(kernel 从 MindSpore 切换到 Python 3、Python 版本从 3.9.10 升级到 3.11.4),以及将 modelscope 的安装从 !pip3 install shell 命令中拆出,改为 Python 代码中的条件安装。这些改动逻辑正确、方向合理。仅有的两个 P3 问题是文件 02 和 04 中部分单元格的 execution_countnull 变为具体数字(非功能性缺陷,仅影响 notebook 的干净程度),建议在合入前恢复为 null

⚠️ 已识别出整体风险,但无法提取行内评论,请参考整体评估。

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

CLA Signature Pass

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

likedislike
CANN-robot
CANN-robot成员
17 天前 评论:

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 fuyangchenghu, zutao (2/2) zutao (1/1)

💡 Tip:

  • Committer can comment /approve or /lgtm
  • Commenting /approve implies both code review (lgtm) and intent to merge (approve)
likedislike
wanzutao
wanzutao成员
16 天前 评论:

/approve
/lgtm

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

/lgtm

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