-
-
Notifications
You must be signed in to change notification settings - Fork 825
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
Improve reduction core #2697
Improve reduction core #2697
Conversation
Sorry for my late review. Could you resolve conflicts? |
ed46fe3
to
86c0564
Compare
I rebased. |
jenkins, test this please. |
Successfully created a job for commit 86c0564: |
Jenkins CI test (for commit 86c0564, target branch master) succeeded! |
This PR is just for code simplification? I compared the performance between this branch and by using cupy-perf, but I saw no performance improvement. |
This PR focuses simplify. |
Co-Authored-By: Akifumi Imanishi <akifumi.imanishi@gmail.com>
jenkins, test this please. |
Successfully created a job for commit a339f3a: |
Jenkins CI test (for commit a339f3a, target branch master) succeeded! |
LGTM. |
This PR adds
_get_reduction_args
to remove duplicated code.Please merge #2689 before this PR.