Skip to content

fix_seed_op #58552

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
Merged
merged 2 commits into from
Nov 2, 2023
Merged

fix_seed_op #58552

merged 2 commits into from
Nov 2, 2023

Conversation

@xingmingyyj
Copy link
Contributor

@xingmingyyj xingmingyyj commented Nov 1, 2023 •
edited
Loading

PR types

Others

PR changes

Others

Description

修复seed_op
旧Ir下定义的GetExpectedKernelType为

  phi::KernelKey GetExpectedKernelType(
      const framework::ExecutionContext& ctx) const override {
    return phi::KernelKey(framework::proto::VarType::INT32, ctx.GetPlace());
  }

在新Ir下也改为INT32类型
因为seed op的Out具有随机性所以加入test/white_list/new_ir_op_test_no_check_list

@paddle-bot paddle-bot bot added the contributor External developers label Nov 1, 2023
@kangguangli kangguangli merged commit 8488ccb into PaddlePaddle:develop Nov 2, 2023
@xingmingyyj xingmingyyj deleted the fix_seed_op branch November 2, 2023 03:54
@xingmingyyj xingmingyyj mentioned this pull request Nov 2, 2023
Closed
zeroRains pushed a commit to zeroRains/Paddle that referenced this pull request Nov 8, 2023
@xingmingyyj @zeroRains
fix_seed_op (PaddlePaddle#58552)
804f0b6 
* fix_seed_op

* fix_seed_op
danleifeng pushed a commit to danleifeng/Paddle that referenced this pull request Nov 14, 2023
@xingmingyyj
fix_seed_op (PaddlePaddle#58552)
30c900f 
* fix_seed_op

* fix_seed_op
@xingmingyyj xingmingyyj mentioned this pull request Nov 27, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@kangguangli kangguangli kangguangli approved these changes

Assignees

No one assigned

Labels

contributor External developers

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@xingmingyyj @kangguangli