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

[Hackathon 5th No.49][pir] add OpResult.append() #59220

Merged
merged 2 commits into from
Nov 24, 2023
Merged

[Hackathon 5th No.49][pir] add OpResult.append() #59220

merged 2 commits into from
Nov 24, 2023

Conversation

gouzil
Copy link
Member

@gouzil gouzil commented Nov 21, 2023 •
edited
Loading

@luotao1 luotao1 mentioned this pull request Nov 22, 2023
Open
@paddle-bot paddle-bot bot added the contributor External developers label Nov 22, 2023
@gouzil gouzil requested a review from 0x45f November 23, 2023 02:10
@gouzil gouzil mentioned this pull request Nov 23, 2023
Merged
0x45f
0x45f approved these changes Nov 24, 2023
View reviewed changes
python/paddle/pir/math_op_patch.py
self.type()
)
)
from paddle.tensor.array import array_length, array_write
Copy link
Contributor

Choose a reason for hiding this comment

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

在文件前面import是会有循环import 的问题吗

@0x45f 0x45f merged commit 7a1795c into PaddlePaddle:develop Nov 24, 2023
28 checks passed
SecretXV pushed a commit to SecretXV/Paddle that referenced this pull request Nov 28, 2023
@gouzil @SecretXV
[Hackathon 5th No.49][pir] add OpResult.append() (PaddlePaddle#59220)
8f7e19f
@gouzil gouzil mentioned this pull request Nov 29, 2023
Open
@gouzil gouzil deleted the pir_method_append branch January 25, 2024 14:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@0x45f 0x45f 0x45f approved these changes

Assignees

@luotao1 luotao1

@0x45f 0x45f

Labels
contributor External developers PaddlePaddle Hackathon
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@gouzil @0x45f @luotao1