| fix: 修复 add_example 头文件 include guard 宏名错误 Co-authored-by: Ding_Jing<dingjing19@huawei.com> # message auto-generated for no-merge-commit merge: !1202 merge fix/include-guard-add-example into master fix: 修复 add_example 头文件 include guard 宏名错误 Created-by: Ding_Jing Commit-by: Ding_Jing Merged-by: cann-robot Description: ## 描述 修复 examples/ 下 2 个头文件的 include guard 宏名错误 **1) 拷贝残留** —— examples/add_example/op_kernel/add_example_tiling_data.h guard 名取自 RotaryPositionEmbeddingGrad 算子,系拷贝后未修改: diff -#ifndef _ROTARY_POSITION_EMBEDDING_GRAD_TILING_DATA_H_ -#define _ROTARY_POSITION_EMBEDDING_GRAD_TILING_DATA_H_ +#ifndef _ADD_EXAMPLE_TILING_DATA_H_ +#define _ADD_EXAMPLE_TILING_DATA_H_ ... -#endif +#endif // _ADD_EXAMPLE_TILING_DATA_H_ **2) 两个不同算子共用同一 guard** —— examples/add_example_aicpu/op_graph/add_example_proto.h 该文件与 examples/add_example/op_graph/add_example_proto.h 同用 OPS_OP_PROTO_INC_AddEXAMPLE_H_。二者若被同一编译单元先后 include,后者整体被 guard 跳过,**导致 REG_OP(AddExampleAicpu) 静默丢失**: diff -#ifndef OPS_OP_PROTO_INC_AddEXAMPLE_H_ -#define OPS_OP_PROTO_INC_AddEXAMPLE_H_ +#ifndef OPS_OP_PROTO_INC_AddEXAMPLEAICPU_H_ +#define OPS_OP_PROTO_INC_AddEXAMPLEAICPU_H_ ... -#endif // OPS_OP_PROTO_INC_AddEXAMPLE_H_ +#endif // OPS_OP_PROTO_INC_AddEXAMPLEAICPU_H_ 两处均保持 #ifndef / #define / #endif 三行一致。非 aicpu 版 proto 的 guard 保持不变,与 ops-math 的处理一致。 ## 关联的Issue 关联Issue #671 ## 测试 改动仅涉及预处理宏,不改变任何运行期逻辑,故以编译级验证为主。 **1. 缺陷 2 的修复前后对比(决定性验证)** 构造同时 include 两个 proto 的编译单元: cpp #include "examples/add_example/op_graph/add_example_proto.h" #include "examples/add_example_aicpu/op_graph/add_example_proto.h" int main() { ge::op::AddExample a("a"); ge::op::AddExampleAicpu b("b"); return 0; } - 修复前:error: 'AddExampleAicpu' is not a member of 'ge::op'; did you mean 'AddExample'? —— 算子注册确实丢失 - 修复后:g++ -fsyntax-only -std=c++17 编译通过,两个算子均可见 **2. guard 生效验证** 对 add_example_tiling_data.h 做重复 include 的完整编译并运行,无重定义错误,AddExampleTilingData 结构体可正常构造使用。 **3. 宏名唯一性** 全仓 514 个头文件扫描,新引入的 _ADD_EXAMPLE_TILING_DATA_H_ 与 OPS_OP_PROTO_INC_AddEXAMPLEAICPU_H_ 各仅出现于 1 个文件,未与既有宏冲突。 **4. 无残留引用** _ROTARY_POSITION_EMBEDDING_GRAD_TILING_DATA_H_ 在本次改动的文件中已清零;OPS_OP_PROTO_INC_AddEXAMPLE_H_ 仅余 examples/add_example/op_graph/add_example_proto.h 自身三行(有意保留)。 ## 文档更新 无。 ## 类型标签 <!-- [x] 表示选中 --> - [x] 🐛 Bug修复 - [ ] ✨ 新特性 - [ ] ⚡ 性能优化 - [ ] ♻️ 重构 - [ ] 🧪 测试 - [ ] 📦 构建/CI - [ ] 🔧 配置变更 - [ ] 📝 文档更新 - [ ] ⬆️ 依赖升级 - [ ] 🔒 安全修复 - [ ] 🧹 代码清理 - [ ] ❓ 其他,请描述: See merge request: cann/ops-cv!1202 | 1 个月前 |
| fix: 修复 add_example 头文件 include guard 宏名错误 Co-authored-by: Ding_Jing<dingjing19@huawei.com> # message auto-generated for no-merge-commit merge: !1202 merge fix/include-guard-add-example into master fix: 修复 add_example 头文件 include guard 宏名错误 Created-by: Ding_Jing Commit-by: Ding_Jing Merged-by: cann-robot Description: ## 描述 修复 examples/ 下 2 个头文件的 include guard 宏名错误 **1) 拷贝残留** —— examples/add_example/op_kernel/add_example_tiling_data.h guard 名取自 RotaryPositionEmbeddingGrad 算子,系拷贝后未修改: diff -#ifndef _ROTARY_POSITION_EMBEDDING_GRAD_TILING_DATA_H_ -#define _ROTARY_POSITION_EMBEDDING_GRAD_TILING_DATA_H_ +#ifndef _ADD_EXAMPLE_TILING_DATA_H_ +#define _ADD_EXAMPLE_TILING_DATA_H_ ... -#endif +#endif // _ADD_EXAMPLE_TILING_DATA_H_ **2) 两个不同算子共用同一 guard** —— examples/add_example_aicpu/op_graph/add_example_proto.h 该文件与 examples/add_example/op_graph/add_example_proto.h 同用 OPS_OP_PROTO_INC_AddEXAMPLE_H_。二者若被同一编译单元先后 include,后者整体被 guard 跳过,**导致 REG_OP(AddExampleAicpu) 静默丢失**: diff -#ifndef OPS_OP_PROTO_INC_AddEXAMPLE_H_ -#define OPS_OP_PROTO_INC_AddEXAMPLE_H_ +#ifndef OPS_OP_PROTO_INC_AddEXAMPLEAICPU_H_ +#define OPS_OP_PROTO_INC_AddEXAMPLEAICPU_H_ ... -#endif // OPS_OP_PROTO_INC_AddEXAMPLE_H_ +#endif // OPS_OP_PROTO_INC_AddEXAMPLEAICPU_H_ 两处均保持 #ifndef / #define / #endif 三行一致。非 aicpu 版 proto 的 guard 保持不变,与 ops-math 的处理一致。 ## 关联的Issue 关联Issue #671 ## 测试 改动仅涉及预处理宏,不改变任何运行期逻辑,故以编译级验证为主。 **1. 缺陷 2 的修复前后对比(决定性验证)** 构造同时 include 两个 proto 的编译单元: cpp #include "examples/add_example/op_graph/add_example_proto.h" #include "examples/add_example_aicpu/op_graph/add_example_proto.h" int main() { ge::op::AddExample a("a"); ge::op::AddExampleAicpu b("b"); return 0; } - 修复前:error: 'AddExampleAicpu' is not a member of 'ge::op'; did you mean 'AddExample'? —— 算子注册确实丢失 - 修复后:g++ -fsyntax-only -std=c++17 编译通过,两个算子均可见 **2. guard 生效验证** 对 add_example_tiling_data.h 做重复 include 的完整编译并运行,无重定义错误,AddExampleTilingData 结构体可正常构造使用。 **3. 宏名唯一性** 全仓 514 个头文件扫描,新引入的 _ADD_EXAMPLE_TILING_DATA_H_ 与 OPS_OP_PROTO_INC_AddEXAMPLEAICPU_H_ 各仅出现于 1 个文件,未与既有宏冲突。 **4. 无残留引用** _ROTARY_POSITION_EMBEDDING_GRAD_TILING_DATA_H_ 在本次改动的文件中已清零;OPS_OP_PROTO_INC_AddEXAMPLE_H_ 仅余 examples/add_example/op_graph/add_example_proto.h 自身三行(有意保留)。 ## 文档更新 无。 ## 类型标签 <!-- [x] 表示选中 --> - [x] 🐛 Bug修复 - [ ] ✨ 新特性 - [ ] ⚡ 性能优化 - [ ] ♻️ 重构 - [ ] 🧪 测试 - [ ] 📦 构建/CI - [ ] 🔧 配置变更 - [ ] 📝 文档更新 - [ ] ⬆️ 依赖升级 - [ ] 🔒 安全修复 - [ ] 🧹 代码清理 - [ ] ❓ 其他,请描述: See merge request: cann/ops-cv!1202 | 1 个月前 |
| modified md files(for readability improvement) Co-authored-by: duhuiping<duhuiping@h-partners.com> # message auto-generated for no-merge-commit merge: !1097 merge master into master modified md files(for readability improvement) Created-by: gitee-duhuiping Commit-by: duhuiping Merged-by: cann-robot Description: ## 描述 Changes: 1. The name of torch_npu is changed to TorchNPU. 2. The ACL API manual is renamed. 3. The word "commerical" in the link is changed to "community". ## 关联的Issue [#607](https://gitcode.com/cann/ops-cv/issues/607) ## 测试 Only involves updating the MD document description, does not involve testing. ## 文档更新 md files ## 类型标签 <!-- [x] 表示选中 --> - [ ] 🐛 Bug修复 - [ ] ✨ 新特性 - [ ] ⚡ 性能优化 - [ ] ♻️ 重构 - [ ] 🧪 测试 - [ ] 📦 构建/CI - [ ] 🔧 配置变更 - [x] 📝 文档更新 - [ ] ⬆️ 依赖升级 - [ ] 🔒 安全修复 - [ ] 🧹 代码清理 - [ ] ❓ 其他,请描述: See merge request: cann/ops-cv!1097 | 2 个月前 |
| fix fast kernel launch example. Co-authored-by: liu-wei<lovline.liuwei@huawei.com> # message auto-generated for no-merge-commit merge: !171 merge fix_fast_kernel into master fix fast kernel launch example. Created-by: Ding_Jing Commit-by: liu-wei Merged-by: cann-robot Description: ## 描述 <!--在这里详细描述你的改动,包括改动的原因和所采取的方法。--> ## 关联的Issue <!-- 如果这个PR是为了解决特定的Issue,请在这里提供Issue链接。例如:关联Issue #123--> <!-- 如果这个PR是为了解决特定的问题单,请在这里描述问题单单号。--> ## 测试 <!--描述进行了哪些测试来验证你的改动。包括但不限于二级冒烟、算子泛化等。--> ## 文档更新 <!--如果这个PR包含文档的更新,请在这里指出。例如:更新了README.md文件。--> ## 类型标签 <!-- [x] 表示选中 --> - [ ] Bug修复 - [x] 新特性 - [ ] 性能优化 - [ ] 文档更新 - [ ] 其他,请描述: See merge request: cann/ops-cv!171 | 7 个月前 |
| modified md files(for readability improvement) Co-authored-by: duhuiping<duhuiping@h-partners.com> # message auto-generated for no-merge-commit merge: !655 merge master_md into master modified md files(for readability improvement) Created-by: gitee-duhuiping Commit-by: duhuiping Merged-by: cann-robot Description: ## 描述 modified md files(for readability improvement) ## 关联的Issue Data issue ## 测试 Optimizing the data format ## 文档更新 md files ## 类型标签 <!-- [x] 表示选中 --> - [ ] 🐛 Bug 修复 - [ ] ✨ 新特性 - [ ] ⚡ 性能优化 - [ ] ♻️ 重构 - [ ] 🧪 测试 - [ ] 📦 构建/CI - [ ] 🔧 配置变更 - [x] 📝 文档更新 - [ ] ⬆️ 依赖升级 - [ ] 🔒 安全修复 - [ ] 🧹 代码清理 - [ ] ❓ 其他,请描述: See merge request: cann/ops-cv!655 | 5 个月前 |