-
Notifications
You must be signed in to change notification settings - Fork 5.6k
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.
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
【PaddlePaddle Hackathon 3 No.14】为 Paddle 新增 remainder_ API #45266
Conversation
你的PR提交成功,感谢你对开源项目的贡献! |
|
✅ This PR's description meets the template requirements! |
@thunder95 麻烦任务 leader 确认一下 |
@zrr1999 大佬 为啥需要我确认?我怎么确认? |
因为提案是你交的呀~所以后面其他开发者完成了这个任务也会请求你来 review 一下呢,是写在活动详情里的: |
@Ligoml 那应该在怎么地方确认, 在这里回复一下就好了么? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
好的谢啦,我也是在那个文档里看到要艾特一下 leader确认的 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
已approve |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM for docs
对应的中文文档需要再补充一些内容
好的,已经修改完毕 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM for docs,中文文档需要稍微修改下~
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM for docs and unittest~
PR types
New features
PR changes
APIs
Describe