search

buddy-compiler / buddy-benchmark

Benchmark Framework for Buddy Projects
Apache License 2.0
44 stars 34 forks source link

[DIP] Add benchmark for Buddy & OpenCV rotation operation. #63

Open taiqzheng opened 1 year ago

taiqzheng commented 1 year ago

For Buddy Rotate2D operation, use command ./image-processing-rotate-benchmark <image path> <Rotate option> <RotateAngle>. Here Rotate option available are DEGREE, RADIAN. RotateAngle accepts a float number for Rotate option. Examples:
./image-processing-rotate-benchmark ../../benchmarks/ImageProcessing/Images/YuTu.png RADIAN 1.57 OR ./image-processing-rotate-benchmark ../../benchmarks/ImageProcessing/Images/YuTu.png DEGREE 45

For OpenCV rotate(), it only supports 90, 180 and 270 degree. Only accepted when using DEGREE option and an integer( Num % 90 = 0). Like: ./image-processing-rotate-benchmark ../../benchmarks/ImageProcessing/Images/YuTu.png DEGREE 270

taiqzheng commented 1 year ago

Please reformat all the c++ file with clang-format.

The code have been reformat with clang-format -i -style=file command.

taiqzheng commented 1 year ago

Will delete the OpenCV enum class after exams(before the weekend).

meshtag commented 1 year ago

I expect the rotation operation to be improved from this PR. Shall we move ahead and review this once that patch lands?

Or do you want me to move ahead with the review process now?

Also, please mark the PR as "ready for review" once you think it is ready.

taiqzheng commented 1 year ago

Shall we move ahead and review this once that patch lands?

Or do you want me to move ahead with the review process now?

If the acceleration PR changes the API, then it will be better to review that PR first.