已合并
[Refactor] HIXL头文件整改 #438
ZhaoBaiwei创建于 5月7日
[Refactor] HIXL头文件整改 #438
已合并
ZhaoBaiwei创建于 5月7日
ZhaoBaiwei
ZhaoBaiwei成员
5月7日
**提交 PR 前,请先阅读 [HIXL贡献指南](https://gitcode.com/cann/hixl/blob/master/CONTRIBUTING.md) ,查看 PR 提交方式并遵守相关提交规范。** ## 类型标签 - [ ] Bug修复 - [ ] 新特性 - [x] 代码重构 - [ ] 文档更新 - [ ] 其他,请描述: ## 描述 | 头文件类型 | 问题原因 | 问题影响 | 整改方案 | | -------------------- | ------------------------------------------------------------ | --------------------------------------------- | ------------------------------------------------------------ | | 仓内重名头文件 | 非对外同名头文件,内容不同,面向不同业务场景 | 可改名,影响开发者体验 | 头文件不对外,可直接改名,保证仓内不重名 | | 不再使用的头文件 | HIXL仓内部不再使用的文件 | 业务侧不再调用,无影响 | 直接删除 | | 子包间重复安装头文件 | LLM_Datadist V1当前归属于ge-compiler,待废弃;LLM_Datadist V2设计时继承了V1,导致头文件名称和安装路径相同 | 重复安装头文件到同一目录导致无法发布deb/rpm包 | 1.LLM_Datadist V1的C++接口已确认无用户使用。如果想使用V1这三个头文件的功能,提醒用户安装toolkit + ops(hixl),并在资料中补充说明。
2. ge-compiler仓中头文件仅编译不打包 | ## 测试项 * 整改前后,无重复头文件问题 * 功能验证 ## 测试结果 - [x] 整改后,无重复头文件问题 - [x] LLT无问题 ## Checklist - [x] 该Pull Request的代码风格和项目的代码风格一致 - [x] 提交的代码已经过充分的验证 - [x] 涉及的相关文档修改已更新 - [x] 标题中正确使用了类型标签(例如:feat/bugfix/refactor/docs/test等) ## 其它(可选)
likedislike
Pull Request已成功合入, 合并人@CANN-robot
(感谢 ZhaoBaiwei 的贡献)
ZhaoBaiweiZhaoBaiwei成员
5月7日 创建了 pull request,commit 32879c53
CANN-robotCANN-robot成员
5月7日 添加了label:stat/needs-squash
CANN-robot
CANN-robot成员
5月7日 评论:

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


PR Approval Progress

Congratulations! All modules have met the lgtm and approve requirements.

Module Approval Details

module lgtm status approve status
repo-cann/hixl zhujingjing, youyx, zhaozhihui (3/2) zhujingjing (1/1)

💡 Tip:

  • Committer can comment /approve or /lgtm
  • Commenting /approve implies both code review (lgtm) and intent to merge (approve)

CLA Signature Pass

ZhaoBaiwei, thanks for your pull request. All authors of the commits have signed the CLA. 👍

likedislike
CANN-robotCANN-robot成员
5月7日 将leabclove,lining23666,youyx,yanyefeng,peiyang,zhujingjing,songmingyang,zhaozhihui,wenxuemin设为评审人
CANN-robotCANN-robot成员
5月7日 添加了label:cann-cla/yes
此处折叠了77条消息 查看更多
ZhaoBaiweiZhaoBaiwei成员
5月19日 修改了pull request 的描述
zhujingjing成员
5月19日 评论:

/approve

likedislike
CANN-robotCANN-robot成员
5月19日 添加了label:approved
CANN-robot
CANN-robot成员
5月19日 评论:

Review Guide

This pull-request passes review.
Committers who wrote a comment of /approve are: zhujingjing.
Reviewers who wrote a comment of /lgtm are: zhaozhihui, zhujingjing, youyx.

likedislike
CANN-robotCANN-robot成员
5月19日 合入了pull request