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

修改 masked_fill RFC 算子输出名称 #637

Merged
merged 15 commits into from
Sep 21, 2023
Merged

修改 masked_fill RFC 算子输出名称 #637

merged 15 commits into from
Sep 21, 2023

Conversation

AndSonder
Copy link
Contributor

@AndSonder AndSonder commented Sep 19, 2023 •
edited
Loading

修改RFC中算子的输入名称

相关PR:

@paddle-bot
Copy link

paddle-bot bot commented Sep 19, 2023

你的PR提交成功，感谢你对开源项目的贡献!
请检查PR提交格式和内容是否完备，具体请参考示例和模版。
Your PR has been submitted. Thanks for your contribution!
Please check its format and content. For this, you can refer to Template and Demo.

@AndSonder AndSonder changed the title 【Hackathon 5th No.2】 为 Paddle 新增 masked_fill API （修改输入名称） 【Hackathon 5th No.3】 为 Paddle 新增 masked_fill API （修改输入名称） Sep 19, 2023
@AndSonder AndSonder mentioned this pull request Sep 19, 2023
Merged
@AndSonder AndSonder changed the title 【Hackathon 5th No.3】 为 Paddle 新增 masked_fill API （修改输入名称） 修改 masked_fill RFC 算子输出名称 Sep 19, 2023
@zoooo0820 zoooo0820 self-assigned this Sep 19, 2023
zoooo0820
zoooo0820 reviewed Sep 19, 2023
View reviewed changes
Copy link
Contributor

@zoooo0820 zoooo0820 left a comment

Choose a reason for hiding this comment

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

辛苦解下冲突，直接在之前合入的文件上修改就好

@luotao1 luotao1 self-assigned this Sep 19, 2023
@AndSonder
Copy link
Contributor Author

辛苦解下冲突，直接在之前合入的文件上修改就好

done

zoooo0820
zoooo0820 approved these changes Sep 21, 2023
View reviewed changes
Copy link
Contributor

@zoooo0820 zoooo0820 left a comment

Choose a reason for hiding this comment

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

LGTM

@zoooo0820 zoooo0820 merged commit b755340 into PaddlePaddle:master Sep 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@zoooo0820 zoooo0820 zoooo0820 approved these changes

Assignees

@luotao1 luotao1

@zoooo0820 zoooo0820

Labels
contributor
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@AndSonder @zoooo0820 @luotao1