-
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
(this PR is just for my personal self-learning) [PaddlePaddle Hackathon 2 No.22] add paddle.index_add to Paddle #42475
Conversation
你的PR提交成功,感谢你对开源项目的贡献! |
add "data = np.copy(data)" in the first line in the function "def index_add_np(data, axis, index, add_value):" in unit test file
Caution: add "x = np.copy(x)" at the first line of the function that use numpy to simulate the calculation of paddle op in unit test file. Because if you don't use the copy of input at first , then the calculation in the function may change the elements of numpy.ndarray in place, so after calling this function, the input numpy.ndarray is modified to be equal to the output numpy.ndarray TODO: #42454 |
|
|
#42804 |
…ry.cc (when did I delete the two lines???)
感谢Paddle 专家的指导意见,我将暂停本PR的开发工作,在以后有时间时再继续开发 |
Sorry to inform you that 24d3827's CIs have passed for more than 7 days. To prevent PR conflicts, you need to re-run all CIs manually. |
Since you haven't replied for more than a year, we have closed this issue/pr. |
很抱歉,经过我们的反复讨论,你的PR暂未达到合入标准,请阅读飞桨原生算子开发规范,你可以重新提交新的PR,我们先将此PR关闭,感谢你的贡献。 |
PR types
New features
PR changes
APIs
Describe
【PaddlePaddle Hackathon 第二期】任务总览
task 22: add paddle.index_add to Paddle
#40311
RFC: PaddlePaddle/community#127
This PR is gotten by modifying this PR: #42454
(with least modification)
I stop working on this PR at 2022/5/17 18:00 . I will continue to work on this PR after 2022/6/10 when I have time.