已合并
ttfhw-解决关联issue #3483
关龙锋创建于 7月31日
ttfhw-解决关联issue #3483
已合并
关龙锋创建于 7月31日
关龙锋成员
7月31日

Description: 网站:https://gitcode.com/TTFHVV/ttfhw-skills
ttfhw-verify-baseos torchair 让大模型来搭建 环境,并执行ut、st验证

likedislike
Pull Request已成功合入, 合并人@ascend-robot
(感谢 关龙锋 的贡献)
关龙锋成员
7月31日 创建了 pull request,commit 1b598f3f
atomgit-bot
atomgit-bot
7月31日 评论:

变更摘要

此 PR 主要将项目中默认的 PyTorch 版本从 2.13.0 回退至 2.12.0,涉及开发容器配置、Docker 构建脚本及帮助文档中相关默认值的统一调整。PR 标题 ttfhw 未提供更多业务背景,但从变更内容看属于版本降级操作。

主要改动

  • 开发容器默认 Torch 版本降级.devcontainer/devcontainer.json 中构建参数 TORCH_VERSION2.13.0 改为 2.12.0,影响开发容器内的 PyTorch 环境。
  • Dockerfile 构建默认版本降级docker/DockerfileARG TORCH_VERSION 默认值由 2.13.0 改为 2.12.0,影响容器镜像构建时的 PyTorch 版本。
  • 构建脚本默认参数与帮助信息同步docker/builder.shTORCH_VERSION 变量默认值从 2.13.0 改为 2.12.0,同时移除了帮助信息中关于 dev 版本(如 2.13.0.dev20260610)的示例说明。
likedislike
atomgit-bot
atomgit-bot
7月31日 评论:

代码审查

复查完成。两个已报告位置没有隐藏的第二个独立缺陷(如交换索引/反转比较/字段取错等)。现在给出总结。


审查总结

文件 审查结果
.devcontainer/devcontainer.json 无问题 — 仅 TORCH_VERSION2.13.02.12.0,JSON 合法,与其余文件一致
README.md 无问题 — 新增 NPU 调试章节、移除开发者自测 UT 小节,均为文档调整,无正确性或安全性缺陷
docker/Dockerfile 无问题 — 仅 ARG TORCH_VERSION 默认值从 2.13.02.12.0,其余逻辑不变
docker/README.md 1 个 P2 问题:脚本参数表中 --torch-version 默认值未同步更新,仍显示旧默认 2.13.0.dev20260610
docker/builder.sh 1 个 P3 问题--torch-version 帮助文本丢失默认值与 dev 版本说明

按优先级统计:

  • P0: 0
  • P1: 0
  • P2: 1
  • P3: 1

整体风险评估:低。 两个发现均为文档一致性问题——同一版本变更在部分位置漏更新,不会导致构建失败、运行时错误或安全漏洞。建议在合并前补齐这两处文档遗漏以保持内部一致性。

类型 数量
🔴 阻塞 0
🟡 建议 3

💬 仅评论

likedislike
ascend-robotascend-robot成员
7月31日 添加了label:smoke-pipeline-running
ascend-robotascend-robot成员
7月31日 添加了label:stat/needs-squash
此处折叠了109条消息 查看更多
dingdairong成员
8月3日 评论:

/lgtm

likedislike
ascend-robotascend-robot成员
8月3日 添加了label:lgtm
ascend-robotascend-robot成员
8月3日 合入了pull request
ascend-robot
ascend-robot成员
8月3日 评论:

Pull Request 已合并或已关闭。

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

likedislike
ascend-robot
ascend-robot成员
8月3日 评论:

Pull Request 已合并或已关闭。

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

likedislike