已合并
fix_tensorpipe_test #95
SCh_zx创建于 6 天前
fix_tensorpipe_test #95
已合并
Pull Request已成功合入, 合并人@ascend-robot
（感谢 SCh_zx 的贡献）
atomgit-bot
6 天前 评论：
6 天前 评论：
变更摘要
本 PR 旨在修复 tensorpipe 相关测试的编译问题：将 cmake/Options.cmake 中 CMake 配置项 TP_BUILD_TESTING 的默认值由 ON 改为 OFF。此举使默认构建不再编译 tensorpipe_test，与原生社区行为保持一致，对 torch_npu 的使用无影响，同时提升编译稳定性。
主要改动
- 调整
TP_BUILD_TESTING默认值: 在cmake/Options.cmake中将option(TP_BUILD_TESTING "Build tests" ON)改为OFF，使测试构建默认关闭。 - 默认跳过
tensorpipe_test编译: 通过默认禁用TP_BUILD_TESTING，避免在常规构建流程中编译tensorpipe_test，与原生社区保持一致，增强编译稳定性，且不影响torch_npu功能。
atomgit-bot
6 天前 评论：
6 天前 评论：
6 天前 添加了label：ascend-cla/yes
ascend-robot
6 天前 评论：
6 天前 评论：
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.
PR Approval Progress
✅ Congratulations! All modules have met the lgtm and approve requirements.
Module Approval Details
| module | lgtm status | approve status |
|---|---|---|
| repo-Ascend/Tensorpipe | ✅ li_jing_hw, liangsongwei (2/2) | ✅ li_jing_hw, liangsongwei (2/1) |
💡 Tip:
- Committer can comment
/approveor/lgtm- Commenting
/approveimplies both code review (lgtm) and intent to merge (approve)
CLA Signature Pass
SCh_zx, thanks for your pull request. All authors of the commits have signed the CLA. 👍
ascend-robot
6 天前 评论：
6 天前 评论：
ascend docs pipeline is running...
6 天前 添加了label：docs-ci-pipeline-running
6 天前 添加了label：ci-pipeline-running
ascend-robot
6 天前 评论：
6 天前 评论：
✅ 跳过 docs ci 检查，没有需要检查的文档文件
6 天前 删除了label：docs-ci-pipeline-running
6 天前 添加了label：docs-ci-pipeline-success
ascend-robot
6 天前 评论：
6 天前 评论：
ascend docs pipeline is running...
6 天前 删除了label：docs-ci-pipeline-success
6 天前 添加了label：docs-ci-pipeline-running
ascend-robot
6 天前 评论：
6 天前 评论：
✅ 跳过 docs ci 检查，没有需要检查的文档文件
6 天前 删除了label：docs-ci-pipeline-running
6 天前 添加了label：docs-ci-pipeline-success
6 天前 删除了label：ci-pipeline-running
6 天前 添加了label：ci-pipeline-passed
AtlasAccount
6 天前 评论：
6 天前 评论：
流水线 PR-pipeline_Tensorpipe#25 [ commitID：49da39bd ] 已完成
| 阶段 | 任务名 | 状态 | 详情 |
|---|---|---|---|
| 编译构建 | Build | ✅ | >>> |
| 恶意代码检查 | Antipoison | ✅ | >>> |
| 编码安全与规范检查 | CodeCheck | ✅ | >>> |
| 开源片段检查 | SCA | ✅ | >>> |
| 流水线 | PR-pipeline_Tensorpipe | ✅ | >>> |
- compile : 运行流水线
- retry : 重试流水线所有失败子任务
- retry <任务名> : 仅重试指定失败子任务
- stop : 停止流水线
li_jing_hw
6 天前 评论：
6 天前 评论：
/approve
6 天前 添加了label：approved
梁松伟
6 天前 评论：
6 天前 评论：
/approve
6 天前 添加了label：lgtm
6 天前 合入了pull request
What type of PR is this?
What does this PR do / why do we need it:
与原生社区保持一致，默认不编译tensorpipe_test。
对torch_npu使用无影响，增加编译稳定性。
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewers: