ROCm / rpp

AMD ROCm Performance Primitives (RPP) library is a comprehensive high-performance computer vision library for AMD processors with HIP/OpenCL/CPU back-ends.
https://rocm.docs.amd.com/projects/rpp/en/latest/
MIT License
53 stars 38 forks source link

[Issue]: Location of half.hpp #292

Closed trixirt closed 5 months ago

trixirt commented 8 months ago

Problem Description

Same issue as MIOpen

I am packaging rpp for Fedora, the addition of the half/ makes it impossible to set the location of half.hpp in configury.

Operating System

Fedora Rawhide

CPU

x86_64

GPU

AMD Instinct MI210

Other

No response

ROCm Version

ROCm 6.0.0

ROCm Component

rpp

Steps to Reproduce

No response

(Optional for Linux users) Output of /opt/rocm/bin/rocminfo --support

No response

Additional Information

No response

kiritigowda commented 8 months ago

@trixirt we have FindHalf to locate half headers. If you add additional location here - https://github.com/ROCm/rpp/blob/2d7adde4c09d7a9b334b6858758531ad7fa8091e/cmake/FindHALF.cmake#L35 should find the library on your setup.

If you provide Half default on Fedora, I can add it or you can create a PR to make it available for all.

kiritigowda commented 7 months ago

@trixirt were you able to resolve this issue?

AngryLoki commented 6 months ago

@trixirt , I think distro maintainers should install half.hpp into /usr/include/half/. I encountered this issue in Gentoo, and I will update ebuilds there in that way.

Reasoning: https://github.com/ROCm/half is a project under ROCm umbrella. MIOpen, MIVisionX, AMDMIGraphX, rpp, MIFin, rocAL and maybe other ROCm projects just decided to include it as <half/half.hpp>. I see this header in other projects, but again, mostly as <half/half.hpp>.

kiritigowda commented 5 months ago

@trixirt --- With the cmake you can also use the -D HALF_DIRor ENV Variable HALF_DIR to set path to include. If you still face this issue reopen.

-D HALF_DIR /PATH/TO/half/half.hpp

https://github.com/ROCm/rpp/blob/b6f334a5f779ea5327cbe398794aece30305dc37/cmake/FindHALF.cmake#L31

https://github.com/ROCm/rpp/blob/b6f334a5f779ea5327cbe398794aece30305dc37/cmake/FindHALF.cmake#L28