已合并
security check #4644
daifu1234567创建于 4月3日
security check #4644
已合并
D
daifu1234567成员
4月3日
4月3日
# 【合入来源】
> **如有社区issue,请关联issue链接**\
> **请勿携带内部流程信息(需求链接、问题单、内部issue等)**
- [ ] 需求
- [x] 问题单
- [ ] issue/工单
- [ ] 重构优化
- [ ] 资料更新
# 【修改方案】
> 请描述修改内容的具体实现,涉及哪些组件之间进行交互,可以用1、2、3、...进行罗列\
> 如果是需求或者重构类的PR,需要补充详细设计文档(说明上下游组件关系、时序图、类图、DFX能力等内容)
1. gencode.sh中新增PYTORCH_VERSION校验,只能包含数字字母-.
2. torchnpugen/gen.py、torchnpugen/gen_derivatives.py、torchnpugen/gen_op_plugin_functions.py中在打开文件之前先校验islink
3. torchnpugen/gen_op_plugin_functions.py中del前先检查是否存在kry
# 【资料变更】
> 请确认是否涉及资料变更。如涉及,需要在PR中体现,并简要说明修改内容。如不涉及,需填写“不涉及”
不涉及
# 【接口变更】
> 请确认是否涉及跨代码仓或者客户面可见的接口变更。如涉及,需要详细说明接口以及对应的变更内容,同时需要在资料中体现。如不涉及,需填写“不涉及”
不涉及
# 【功能验证】
> 说明测试场景,测试方法。如果本次测试方式与常规单元测试不同,请详细说明您的测试步骤\
> 新增/变更内容是否已新增/适配UT测试用例看护,并补充测试自验证截图
安全问题,不涉及
# 【CheckList】
> PR提交人对以下CheckList自检项进行全量自检,自检通过或不涉及,均修改 [ ] 为 [x]
- [x] 代码注释完备,正确记录错误日志
- [x] 代码实现进行了返回值、空指针等校验
- [x] PR标题正确使用类型标签,如:feat、fix、refactor、docs、test等
- [x] PR持续集成流水线(CI)执行通过,代码检查无异常


Pull Request已成功合入, 合并人@ascend-robot
(感谢 daifu1234567 的贡献)4月3日 创建了 pull request,commit 1e2362a9
ascend-robot
4月3日 评论:
4月3日 评论:
ascend-robot
4月3日 评论:
4月3日 评论:
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-Ascend/op-plugin | ✅ 楚浩田, 梁松伟 (2/2) | ✅ 楚浩田 (1/1) |
💡 Tip:
- Committer can comment
/approveor/lgtm- Commenting
/approveimplies both code review (lgtm) and intent to merge (approve)
CLA Signature Pass
daifu1234567, thanks for your pull request. All authors of the commits have signed the CLA. 👍


4月3日 添加了label:ascend-cla/yes
daifu1234567
4月3日 评论:
4月3日 评论:
compile


此处折叠了79条消息 查看更多
4月3日 添加了label:approved
梁松伟
4月3日 评论:
4月3日 评论:
/lgtm


4月3日 添加了label:lgtm
ascend-robot
4月3日 评论:
4月3日 评论:
4月3日 合入了pull request