abacus-develop icon indicating copy to clipboard operation
abacus-develop copied to clipboard

[Test] zdot_real_op (and some other math ops) benchmark on cpu

Open Critsium-xy opened this issue 1 year ago • 3 comments

Finished zdot_real_op benchmark on cpu. Please check whether the way generating testing cases is appropriate. I will do google benchmark for other ops related to after this PR get checked.

Modified file: source/module_hsolver/kernels/test/CMakeLists.txt

Added file: source/module_hsolver/kernels/test/perf_math_kernel.cpp

How to run this benchmark: Under folder {PATH_TO_ABACUS}/abacus-develop/build , do this command ctest -R Perf_Hsolver_Kernels After this test finished, do this command to check benchmark report cat {PATH_TO_ABACUS}/abacus-develop/build/Testing/Temporary/LastTest.log

resolve #3891

image

Critsium-xy avatar Mar 28 '24 12:03 Critsium-xy

I finished some other ops that should be able to use the same method to generate test cases. But the theme or this PR is unchanged.

Critsium-xy avatar Mar 29 '24 05:03 Critsium-xy

The title of this PR should be 'Test' but not 'Feature'.

dyzheng avatar Apr 01 '24 07:04 dyzheng

LGTM, maybe you can use squash to make your commit clear. A clear, readable commit log also helps the developers and users to understand new features. And this doc may help you, refer to: https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/incorporating-changes-from-a-pull-request/about-pull-request-merges https://gist.github.com/lpranam/4ae996b0a4bc37448dc80356efbca7fa

jieli-matrix avatar May 11 '24 07:05 jieli-matrix