| 文件 | 最后提交记录 | 最后更新时间 |
|---|---|---|
[build] License Header Co-authored-by: ouyangcaike<ouyangcaike@h-partners.com> # message auto-generated for no-merge-commit merge: !4556 merge build/oat into master [build] License Header Created-by: ouyangcaike Commit-by: ouyangcaike Merged-by: cann-robot Description: ## 描述 <!--在这里详细描述你的改动，包括改动的原因和所采取的方法。--> 1. 全仓C/C++、CMake、Python、Shell文件规范头部License的使用 ## 变更类型 请选择本次引入的变更类型： <!-- [x] 表示选中 --> - [ ] 🐛 Bug修复 - [ ] ✨ 新特性 - [ ] 🚀 性能优化 - [ ] 📝 文档更新 - [x] 📋 其他，请描述：工程 ## 关联的Issue <!-- 如果这个PR是为了解决特定的Issue，请在这里提供Issue链接。--> <!-- 如果这个PR不涉及Issue，可填写"NA"。--> NA ## 测试 <!--描述进行了哪些测试来验证你的改动。包括但不限于构造对应xx测试用例、二级冒烟、算子泛化等。--> 已完成的测试用例和场景： 1. 编译成功 2. 全量拦截项通过 补充的UT用例： NA ## 文档更新 <!--如果这个PR包含文档的更新，请在这里指出。例如：更新了README.md文件。--> NA ## 合入检查 <!-- 在正式合入前，请做好必要的代码测试，用例补充，软件代码风格检查等。提高合入效率。--> <!-- [x] 表示选中 --> - [x] 🧐 已经详细阅读了贡献指南（CONTRIBUTING.md），并遵守了其中的所有规定，包括但不限于commit message的格式、无效commit的合并等 - [x] 🔍 邀请 committer评论 /lgtm前的必要检查 - [x] 🏷️ 标题中使用了合适的类型标签（如：[feat], [fix]） - [x] 📄 代码修改内容已简要描述，相关文档已更新 - [x] 📝 代码注释已更新，代码遵循项目整体代码风格 - [x] 🧪 代码UT测试已更新，覆盖率已达标 - [x] 🔬 验证方法已更新到"测试"部分 - [x] 🛠️ 代码已通过静态分析工具检查，无错误 - [x] 👥 代码检视/code review/同行评议和必要的代码串讲，确保代码质量 - [x] ✅ 代码检视意见已处理或答复，无未处理的检视意见 - [x] 🚀 预约 前冒烟 用例前的必要检查 - [x] ✔️ 代码已有committer的/lgtm 和 模块committer的/lgtm评论 - [x] 🔧 代码已通过compile，编译无错误，无告警 - [x] 🖥️ 代码已通过基本功能本地测试或者在线测试，确保基本功能正常 - [x] 🎯 预约 approver评论/approve，正式合入前的必要检查 - [x] 📊 前冒烟 用例已全量通过 - [x] 📦 新增功能已同步补充基本功能测试用例到前冒烟里 See merge request: cann/hcomm!4556 | 25 天前 | |
[fix]修改日志级别错配 Co-authored-by: ren-mengguang<renmengguang@h-partners.com> # message auto-generated for no-merge-commit merge: !4739 merge rmg_code into master [fix]修改日志级别错配 Created-by: ren-mengguang Commit-by: ren-mengguang Merged-by: cann-robot Description: ## 描述 本次 PR 对 UT 测试代码进行日志级别修正与重复代码消除，主要解决 codecheck 报出的"重复代码"与"超大函数"问题，同时修复日志级别错配。全部改动仅涉及 test/ 目录，未修改 src/ 功能代码。 ### 1. 日志级别修正 - **失败场景误用 INFO → 改为 ERROR**：多个测试 SetUpTestCase 中文件打开失败的日志从 HCCL_INFO 改为 HCCL_ERROR，符合"成功 INFO、失败 ERROR"的语义（ut_hcom.cc 4 处、ut_hccl_kernel_info.cc/ut_hccl_kernel_builder.cc/ut_hvd_graph_optimizer.cc/ut_hvd_kernel_builder.cc 各 1 处）。 - **调试打印误用 ERROR → 改为 INFO**：ut_hccl_host_shm.cc 中排查用的 HCCL_ERROR("jsh ...") 降为 HCCL_INFO，并删除重复的日志块与调试前缀。 - **裸 printf/cout/中文 → HCCL 日志宏/英文**：llt_hccl_stub_error_manager.cc 的 printf 改为 HCCL_INFO（补 #include "log.h"）；orion_adapter_hccp_stub.cc 的中文 std::cout 改为英文。 ### 2. 重复代码消除（codecheck 报出的"重复代码"与"超大函数"问题） - **WaitRanksReady 辅助函数**（ut_hccl_host_shm.cc）：impl_host_shm_broadcast_task 与 impl_host_shm_all_reduce_task 中逐字复制的 11 行同步屏障代码（CAS 自旋 + sched_yield + 内存屏障）抽取为 static void WaitRanksReady(...)，两处改为一行调用。同时消除 all_reduce_task 的 52 行超大函数问题（NBNC 降至 <50）。 - **DoReduceScatterVTest 辅助函数**（ut_hcom.cc）：ut_hcom_reducescatterv 与 ut_hcom_reducescatterv_check_int64 两个 TEST_F 几乎全等（42 行重复），仅数据类型（INT8/INT64）与期望返回（SUCCESS/E_NOT_SUPPORT）不同，抽取为参数化辅助函数，两个 TEST_F 各缩为一行调用。 - **WriteRankTableFile 共享辅助**（ut_rank_table.h）：多个测试 SetUpTestCase 中"写 rank_table JSON 到文件"的样板代码（ofstream + 日志 + setw + close）跨文件重复，抽取为 inline void WriteRankTableFile(...) 放入已有的 ut_rank_table.h，ut_hccl_kernel_info.cc、ut_hccl_kernel_builder.cc、ut_hcom.cc 各替换为一行调用。 ### 3. clang-format 格式修正 - 按仓库 .clang-format（v18.1.8, style=file）对改动文件执行格式化，修正 HcomReduceScatterV 调用换行位置、短 TEST_F 体折叠、JSON 缩进等格式偏差。 ## 变更类型 请选择本次引入的变更类型： - [ ] 🐛 Bug修复 - [ ] ✨ 新特性 - [ ] 🚀 性能优化 - [ ] 📝 文档更新 - [x] 📋 其他，请描述：UT 测试代码日志级别修正 + 重复代码消除 + clang-format 格式修正 ## 关联的Issue NA ## 测试 已完成的测试用例和场景： 1. ut_hccl_host_shm.cc：WaitRanksReady 抽取后，impl_host_shm_broadcast_task / impl_host_shm_all_reduce_task 逻辑不变，同步屏障行为保持一致。 2. ut_hcom.cc：DoReduceScatterVTest 参数化后，ut_hcom_reducescatterv（INT8→SUCCESS）与 ut_hcom_reducescatterv_check_int64（INT64→E_NOT_SUPPORT）的断言预期不变。 3. ut_rank_table.h：WriteRankTableFile 抽取后，ut_hccl_kernel_info.cc、ut_hccl_kernel_builder.cc、ut_hcom.cc 的 SetUpTestCase 写文件行为不变（同样创建 JSON 文件 + 同样日志）。 4. llt_hccl_stub_error_manager.cc：补 #include "log.h" 后 HCCL_INFO 宏可解析，-fsyntax-only 编译验证通过。 补充的UT用例：无新增用例，本次为既有用例的代码质量优化。 ## 文档更新 无 ## 合入检查 - [x] 🧐 已经详细阅读了贡献指南（CONTRIBUTING.md），并遵守了其中的所有规定 - [x] 🔍 邀请 committer评论/lgtm前的必要检查 - [x] 🏷️ 标题中使用了合适的类型标签 - [x] 📄 代码修改内容已简要描述 - [x] 📝 代码遵循项目整体代码风格（clang-format v18.1.8 已通过） - [x] 🧪 代码UT测试已更新 - [x] 🔬 验证方法已更新到"测试"部分 - [x] 🛠️ 代码已通过静态分析工具检查 - [ ] 👥 代码检视/code review/同行评议 - [ ] ✅ 代码检视意见已处理或答复 - [ ] 🚀 预约前冒烟用例前的必要检查 - [ ] 🎯 预约 approver评论/approve，正式合入前的必要检查 See merge request: cann/hcomm!4739 | 17 天前 | |
[fix]修改日志级别错配 Co-authored-by: ren-mengguang<renmengguang@h-partners.com> # message auto-generated for no-merge-commit merge: !4739 merge rmg_code into master [fix]修改日志级别错配 Created-by: ren-mengguang Commit-by: ren-mengguang Merged-by: cann-robot Description: ## 描述 本次 PR 对 UT 测试代码进行日志级别修正与重复代码消除，主要解决 codecheck 报出的"重复代码"与"超大函数"问题，同时修复日志级别错配。全部改动仅涉及 test/ 目录，未修改 src/ 功能代码。 ### 1. 日志级别修正 - **失败场景误用 INFO → 改为 ERROR**：多个测试 SetUpTestCase 中文件打开失败的日志从 HCCL_INFO 改为 HCCL_ERROR，符合"成功 INFO、失败 ERROR"的语义（ut_hcom.cc 4 处、ut_hccl_kernel_info.cc/ut_hccl_kernel_builder.cc/ut_hvd_graph_optimizer.cc/ut_hvd_kernel_builder.cc 各 1 处）。 - **调试打印误用 ERROR → 改为 INFO**：ut_hccl_host_shm.cc 中排查用的 HCCL_ERROR("jsh ...") 降为 HCCL_INFO，并删除重复的日志块与调试前缀。 - **裸 printf/cout/中文 → HCCL 日志宏/英文**：llt_hccl_stub_error_manager.cc 的 printf 改为 HCCL_INFO（补 #include "log.h"）；orion_adapter_hccp_stub.cc 的中文 std::cout 改为英文。 ### 2. 重复代码消除（codecheck 报出的"重复代码"与"超大函数"问题） - **WaitRanksReady 辅助函数**（ut_hccl_host_shm.cc）：impl_host_shm_broadcast_task 与 impl_host_shm_all_reduce_task 中逐字复制的 11 行同步屏障代码（CAS 自旋 + sched_yield + 内存屏障）抽取为 static void WaitRanksReady(...)，两处改为一行调用。同时消除 all_reduce_task 的 52 行超大函数问题（NBNC 降至 <50）。 - **DoReduceScatterVTest 辅助函数**（ut_hcom.cc）：ut_hcom_reducescatterv 与 ut_hcom_reducescatterv_check_int64 两个 TEST_F 几乎全等（42 行重复），仅数据类型（INT8/INT64）与期望返回（SUCCESS/E_NOT_SUPPORT）不同，抽取为参数化辅助函数，两个 TEST_F 各缩为一行调用。 - **WriteRankTableFile 共享辅助**（ut_rank_table.h）：多个测试 SetUpTestCase 中"写 rank_table JSON 到文件"的样板代码（ofstream + 日志 + setw + close）跨文件重复，抽取为 inline void WriteRankTableFile(...) 放入已有的 ut_rank_table.h，ut_hccl_kernel_info.cc、ut_hccl_kernel_builder.cc、ut_hcom.cc 各替换为一行调用。 ### 3. clang-format 格式修正 - 按仓库 .clang-format（v18.1.8, style=file）对改动文件执行格式化，修正 HcomReduceScatterV 调用换行位置、短 TEST_F 体折叠、JSON 缩进等格式偏差。 ## 变更类型 请选择本次引入的变更类型： - [ ] 🐛 Bug修复 - [ ] ✨ 新特性 - [ ] 🚀 性能优化 - [ ] 📝 文档更新 - [x] 📋 其他，请描述：UT 测试代码日志级别修正 + 重复代码消除 + clang-format 格式修正 ## 关联的Issue NA ## 测试 已完成的测试用例和场景： 1. ut_hccl_host_shm.cc：WaitRanksReady 抽取后，impl_host_shm_broadcast_task / impl_host_shm_all_reduce_task 逻辑不变，同步屏障行为保持一致。 2. ut_hcom.cc：DoReduceScatterVTest 参数化后，ut_hcom_reducescatterv（INT8→SUCCESS）与 ut_hcom_reducescatterv_check_int64（INT64→E_NOT_SUPPORT）的断言预期不变。 3. ut_rank_table.h：WriteRankTableFile 抽取后，ut_hccl_kernel_info.cc、ut_hccl_kernel_builder.cc、ut_hcom.cc 的 SetUpTestCase 写文件行为不变（同样创建 JSON 文件 + 同样日志）。 4. llt_hccl_stub_error_manager.cc：补 #include "log.h" 后 HCCL_INFO 宏可解析，-fsyntax-only 编译验证通过。 补充的UT用例：无新增用例，本次为既有用例的代码质量优化。 ## 文档更新 无 ## 合入检查 - [x] 🧐 已经详细阅读了贡献指南（CONTRIBUTING.md），并遵守了其中的所有规定 - [x] 🔍 邀请 committer评论/lgtm前的必要检查 - [x] 🏷️ 标题中使用了合适的类型标签 - [x] 📄 代码修改内容已简要描述 - [x] 📝 代码遵循项目整体代码风格（clang-format v18.1.8 已通过） - [x] 🧪 代码UT测试已更新 - [x] 🔬 验证方法已更新到"测试"部分 - [x] 🛠️ 代码已通过静态分析工具检查 - [ ] 👥 代码检视/code review/同行评议 - [ ] ✅ 代码检视意见已处理或答复 - [ ] 🚀 预约前冒烟用例前的必要检查 - [ ] 🎯 预约 approver评论/approve，正式合入前的必要检查 See merge request: cann/hcomm!4739 | 17 天前 |