Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[OpTest]Fix OpTest #50788

Merged
merged 2 commits into from
Feb 23, 2023
Merged

[OpTest]Fix OpTest #50788

merged 2 commits into from
Feb 23, 2023

Conversation

Charles-hit
Copy link
Contributor

@Charles-hit Charles-hit commented Feb 22, 2023 •
edited
Loading

PR types

Bug fixes

PR changes

Others

Describe

本次Pr主要做了以下两个工作：

  1. 对OpTest静态图部分用program_guard去进行保护
  2. 修复当kernelsignature 与python api 输出个数不一致导致输出dict创建失败的问题。

@Charles-hit Charles-hit reopened this Feb 23, 2023
JiabinYang
JiabinYang approved these changes Feb 23, 2023
View reviewed changes
Copy link
Contributor

@JiabinYang JiabinYang left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@Charles-hit Charles-hit merged commit 91992da into PaddlePaddle:develop Feb 23, 2023
@Charles-hit Charles-hit deleted the fix_outputs_dict branch February 28, 2023 07:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@JiabinYang JiabinYang JiabinYang approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Charles-hit @JiabinYang