| 【PR】: 对外重复头文件修改对内部分新增文件,待legacy仓修改后再删除
Co-authored-by: SeasonChu<zhuxincheng2@huawei.com>
# message auto-generated for no-merge-commit merge:
!3198 merge fixIncPart2 into develop
【PR】: 对外重复头文件修改对内部分新增文件,待legacy仓修改后再删除
Created-by: SeasonChu
Commit-by: SeasonChu
Merged-by: cann-robot
Description:
# Pull Request
## 描述
文件名相同,内容不同,一个对外,一个内部,内部的头文件整改文件名
问题原因:
历史遗留问题
问题影响:
外部开发者易用性体验,不易理解
解决方案:
内部头文件整改文件名,保留原文件,待leagcy仓修改后再删除
## 变更类型
请选择本次引入的变更类型:
<!-- [x] 表示选中 -->
- [ ] 🐛 Bug 修复
- [ ] ✨ 新功能
- [ ] 💄 代码风格更新(格式化,局部变量)
- [x] ♻️ 重构(既不修复错误也不增加功能的代码变动)
- [ ] 📦 构建过程或辅助工具的变动
- [ ] 📝 文档内容更新
## 关联的Issue
<!-- 如果这个PR是为了解决特定的Issue,请在当前页面的右侧'关联Issue'部分添加相应Issue链接,并勾选'合并后关闭已关联的 Issue'选项。 -->
## 如何测试
描述测试此变更的步骤和前提条件:
1.
2.
## 核对清单
<!-- [x] 表示选中 -->
- [x] 我的代码遵循了项目的代码风格
- [x] 我已对代码进行了自测
- [x] 我已更新了相关的文档
- [x] 我在标题中使用了合适的类型标签(如:feat:, fix:)
- [x] 我已经详细阅读了贡献指南(CONTRIBUTING.md),并遵守了其中的所有规定,包括但不限于commit message的格式、无效commit的合并等
## 其他信息
在此添加任何其他关于本次 PR 的说明。
See merge request: cann/ge!3198 | 5 天前 |
| 【feat】: 将func2graph.py与本地protobuf解耦
Co-authored-by: kantao1<kantao1@huawei.com>
# message auto-generated for no-merge-commit merge:
!2551 merge decouple_protobuf into develop
【feat】: 将func2graph.py与本地protobuf解耦
Created-by: kantao1
Commit-by: kantao1
Merged-by: cann-robot
Description:
# Pull Request
## 描述
### 背景
run包里的脚本 func2graph.py 依赖 protoc 生成的 py 文件,执行时会使用本地环境上的 protobuf。由于 protoc 与 protobuf 之间版本有约束(protoc版本>=protobuf版本),所以在使用 func2graph.py 的时候可能会因为版本问题而失败。
### 整体介绍
**当前实现:**
1. 通过 protoc 转换生成 graph 相关的 **py** 文件,然后打到 run 包里给 func2graph.py 调用。
2. func2graph.py 直调提供的 py 接口
```mermaid
flowchart LR
A[proto文件]
B[py文件]
C[run包]
D[func2graph.py]
A -->|protoc转换| B -->|打包到| C -->|被直调| D
```
<br/>
**解耦方案:**
1. 通过 protoc 转换生成 graph 相关的 **cc** 文件,再通过代理提供一套 C接口备用,最后编译得到动态库so,且链接阶段需要链接 protobuf 静态库(执行时就不需要使用本地的protobuf了)
2. 修改 func2graph.py,将原本直调 py 接口的逻辑替换为通过 ctypes 调用 C接口
```mermaid
flowchart LR
A[proto文件]
B[cc文件]
C[C接口]
D[生成动态库]
E[run包]
F[func2graph.py通过ctypes调用C接口]
A -->|protoc转换| B -->|代理| C -->|链接protobuf静态库| D -->|打包到| E -->|被调用| F
```
## 变更类型
请选择本次引入的变更类型:
<!-- [x] 表示选中 -->
- [ ] 🐛 Bug 修复
- [ ] ✨ 新功能
- [ ] 💄 代码风格更新(格式化,局部变量)
- [ ] ♻️ 重构(既不修复错误也不增加功能的代码变动)
- [x] 📦 构建过程或辅助工具的变动
- [ ] 📝 文档内容更新
## 关联的Issue
<!-- 如果这个PR是为了解决特定的Issue,请在当前页面的右侧'关联Issue'部分添加相应Issue链接,并勾选'合并后关闭已关联的 Issue'选项。 -->
## 如何测试
描述测试此变更的步骤和前提条件:
1. func2graph.py 脚本使用方式不变
2. func2graph.py 不再依赖本地 protobuf,可删除本地 protobuf 进行验证
## 核对清单
<!-- [x] 表示选中 -->
- [x] 我的代码遵循了项目的代码风格
- [x] 我已对代码进行了自测
- [x] 我已更新了相关的文档
- [x] 我在标题中使用了合适的类型标签(如:feat:, fix:)
- [x] 我已经详细阅读了贡献指南(CONTRIBUTING.md),并遵守了其中的所有规定,包括但不限于commit message的格式、无效commit的合并等
## 其他信息
在此添加任何其他关于本次 PR 的说明。
See merge request: cann/ge!2551 | 27 天前 |
| Initial commit
| 5 个月前 |
| Initial commit
| 5 个月前 |