已合并
[feat]新增和修改sk_options #34452
mihudan创建于 4月26日
[feat]新增和修改sk_options #34452
已合并
mihudan创建于 4月26日
mihudan成员
4月26日

【合入来源】

如有社区issue,请关联issue链接
请勿携带内部流程信息(需求链接、问题单、内部issue等)

【修改方案】

新增加sk_options的py对象转换成c++结构体的逻辑,传递给ask接口

【资料变更】

不涉及,后续才会上npugraph_ex资料

【接口变更】

不涉及

【功能验证】

说明测试场景,测试方法。如果本次测试方式与常规单元测试不同,请详细说明您的测试步骤
新增/变更内容是否已新增/适配UT测试用例看护,并补充测试自验证截图

image.png

【CheckList】

PR提交人对以下CheckList自检项进行全量自检,自检通过或不涉及,均修改 [ ] 为 [x]

likedislike
Pull Request已成功合入, 合并人@ascend-robot
(感谢 mihudan 的贡献)
Mmihudan成员
4月26日 创建了 pull request,commit 2aa7b285
ascend-robotascend-robot成员
4月26日 添加了label:stat/needs-squash
ascend-robot
ascend-robot成员
4月26日 评论:

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/pytorch wangqiang160, XDaoHong (2/2) wangqiang160 (1/1)
test wangqiang160, XDaoHong (2/2) wangqiang160 (1/1)
torch_npu/npu wangqiang160, XDaoHong (2/2) wangqiang160 (1/1)

💡 Tip:

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

CLA Signature Pass

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

likedislike
ascend-robotascend-robot成员
4月26日 添加了label:ascend-cla/yes
此处折叠了44条消息 查看更多
ascend-robotascend-robot成员
4月27日 添加了label:approved
XDaoHong
XDaoHong成员
4月27日 评论:

/lgtm

likedislike
ascend-robotascend-robot成员
4月27日 添加了label:lgtm
ascend-robot
ascend-robot成员
4月27日 评论:

Review Guide

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

likedislike
ascend-robotascend-robot成员
4月27日 合入了pull request