| 文件 | 最后提交记录 | 最后更新时间 |
|---|---|---|
hcomm参数一致性校验 Co-authored-by: leironghao1<leironghao1@h-partners.com> # message auto-generated for no-merge-commit merge: !2630 merge hcomm_dev_new into master hcomm参数一致性校验 Created-by: leironghao1 Commit-by: leironghao1 Merged-by: cann-robot Description: ## 描述 A5【Hcomm】【框架】集合通信基础包支持在建链时校验通信域级别的配置信息(校验ranktanle, 子通讯域参数，环境变量HCCL_BUFFSIZE,cann包版本) **rank_consistency_checker_v2.cc** 中新增上述校验参数的crc记录和比对方法 **param_check_v2.cc**中新增ranktable crc获取代码 **coll_comm_res_c_adpt.cc**中新增A5建链入口cann包版本，环境变量获取代码 **op_base.cc**中新增子通讯域校验参数crc记录代码 在BatchExchangeAndCheckConsistency方法中使用socket进行Hcomm信息交换并校验 ## 变更类型 请选择本次引入的变更类型： <!-- [x] 表示选中 --> - [ ] 🐛 Bug修复 - [x] ✨ 新特性 - [ ] 🚀 性能优化 - [ ] 📝 文档更新 - [ ] 📋 其他，请描述： ## 关联的Issue <!-- 如果这个PR是为了解决特定的Issue，请在这里提供Issue链接。--> <!-- 如果这个PR不涉及Issue，可填写"NA"。--> ## 测试 <!--描述进行了哪些测试来验证你的改动。包括但不限于构造对应xx测试用例、二级冒烟、算子泛化等。--> 已完成的测试用例和场景： UT 新增4个UT用例 |测试用例 | 覆盖的分支 | |-|-| |Ut_CompareCheckFrameV2_RankTable_Expect_Success|ranktable校验一致| |Ut_CompareCheckFrameV2_Version_Expect_Success|CANN版本校验一致| |Ut_CompareCheckFrameV2_RankTableMismatch_Expect_INTERNAL|ranktable不一致| |Ut_CompareCheckFrameV2_VersionMismatch_Expect_INTERNAL|CANN版本不一致| 全量 RDV ## 文档更新 <!--如果这个PR包含文档的更新，请在这里指出。例如：更新了README.md文件。--> ## 合入检查 <!-- 在正式合入前，请做好必要的代码测试，用例补充，软件代码风格检查等。提高合入效率。--> <!-- [x] 表示选中 --> - [x] 🧐 已经详细阅读了贡献指南（CONTRIBUTING.md），并遵守了其中的所有规定，包括但不限于commit message的格式、无效commit的合并等 - [ ] 🔍 邀请 committer评论 /lgtm前的必要检查 - [x] 🏷️ 标题中使用了合适的类型标签（如：[feat], [fix]） - [x] 📄 代码修改内容已简要描述，相关文档已更新 - [x] 📝 代码注释已更新，代码遵循项目整体代码风格 - [x] 🧪 代码UT测试已更新，覆盖率已达标 - [x] 🔬 验证方法已更新到"测试"部分 - [x] 🛠️ 代码已通过静态分析工具检查，无错误 - [x] 👥 代码检视/code review/同行评议和必要的代码串讲，确保代码质量 - [x] ✅ 代码检视意见已处理或答复，无未处理的检视意见 - [ ] 🚀 预约 前冒烟 用例前的必要检查 - [x] ✔️ 代码已有committer的/lgtm 和 模块committer的/lgtm评论 - [x] 🔧 代码已通过compile，编译无错误，无告警 - [x] 🖥️ 代码已通过基本功能本地测试或者在线测试，确保基本功能正常 - [ ] 🎯 预约 approver评论/approve，正式合入前的必要检查 - [x] 📊 前冒烟 用例已全量通过 - [ ] 📦 新增功能已同步补充基本功能测试用例到前冒烟里 See merge request: cann/hcomm!2630 | 10 天前 | |
子包适配 Co-authored-by: jiyuanhao<jiyuanhao1@huawei.com> | 5 个月前 | |
fix: remove MAX_CCU_MC2_SERVER_NUM interception and align LINK_PROTO_TYPE_NUM with enum Co-authored-by: junhui_huang<1293728669@qq.com> # message auto-generated for no-merge-commit merge: !3363 merge fix/variable-intercept-and-func-enum into master fix: remove MAX_CCU_MC2_SERVER_NUM interception and align LINK_PROTO_TYPE_NUM with enum Created-by: alaving Commit-by: junhui_huang Merged-by: cann-robot Description: ## 描述 修改任务： 1.CCU MC2 server 数量硬编码拦截不合理,评审结论为直接删除这个拦截。采用删除常量定义 + 删除拦截块 + 清理随之失效的测试用例。 2.LINK_PROTO_TYPE_NUM 由硬编码 4 改为绑定 LinkProtoType::__COUNT__，按照评审结论跟枚举大小保持一致。 衍生的问题： fixup commit 处理的是审查报告里的 3 个低严重度清理项，全部是基于第一次 commit 删除拦截逻辑后留下的代码残骸：原来为了统计全局 CCU MC2 server 数量而定义的局部变量、原本用于断言"未超 20 上限"的两个测试用例，都在拦截逻辑删除后失去了存在意义——前者成了没人引用的死代码，因为"20 这个上限"已不存在、名字和断言逻辑都成了对已删除约束的引用，本质上是测试与被测代码语义脱钩的滞留样本。 ## 变更类型 请选择本次引入的变更类型： <!-- [x] 表示选中 --> - [ ] 🐛 Bug修复 - [ ] ✨ 新特性 - [X] 🚀 性能优化 - [ ] 📝 文档更新 - [ ] 📋 其他，请描述： ## 关联的Issue NA ## 测试 NA 补充的UT用例： ## 文档更新 NA ## 合入检查 <!-- 在正式合入前，请做好必要的代码测试，用例补充，软件代码风格检查等。提高合入效率。--> <!-- [x] 表示选中 --> - [ ] 🧐 已经详细阅读了贡献指南（CONTRIBUTING.md），并遵守了其中的所有规定，包括但不限于commit message的格式、无效commit的合并等 - [ ] 🔍 邀请 committer评论 /lgtm前的必要检查 - [ ] 🏷️ 标题中使用了合适的类型标签（如：[feat], [fix]） - [ ] 📄 代码修改内容已简要描述，相关文档已更新 - [ ] 📝 代码注释已更新，代码遵循项目整体代码风格 - [ ] 🧪 代码UT测试已更新，覆盖率已达标 - [ ] 🔬 验证方法已更新到"测试"部分 - [ ] 🛠️ 代码已通过静态分析工具检查，无错误 - [ ] 👥 代码检视/code review/同行评议和必要的代码串讲，确保代码质量 - [ ] ✅ 代码检视意见已处理或答复，无未处理的检视意见 - [ ] 🚀 预约 前冒烟 用例前的必要检查 - [ ] ✔️ 代码已有committer的/lgtm 和 模块committer的/lgtm评论 - [ ] 🔧 代码已通过compile，编译无错误，无告警 - [ ] 🖥️ 代码已通过基本功能本地测试或者在线测试，确保基本功能正常 - [ ] 🎯 预约 approver评论/approve，正式合入前的必要检查 - [ ] 📊 前冒烟 用例已全量通过 - [ ] 📦 新增功能已同步补充基本功能测试用例到前冒烟里 See merge request: cann/hcomm!3363 | 3 天前 |
| 文件 | 最后提交记录 | 最后更新时间 |
|---|---|---|
| 10 天前 | ||
| 5 个月前 | ||
| 3 天前 |