search

samridh-dev / votess

A fast, multi-target voronoi tessellator using the SYCL framework
GNU General Public License v3.0
3 stars 1 forks source link

oneapi setvars.sh #1

Open cbyrohl opened 2 months ago

cbyrohl commented 2 months ago

Hi Sam,

copying Dylan's issues over from the private repo:

The documentation currently says this:

If compiled with the dpc++ compiler, the following command must be run in the terminal instance:

source /opt/intel/oneapi/setvars.sh

but this may vary system by system. On my computer, I have no file. Suggest to remove, perhaps just provide a link to some oneapi documentation instead which explains what this is doing/why it is necessary.

As a result,

$ bash make.sh make.sh: line 6: /opt/intel/oneapi/setvars.sh: No such file or directory

samridh-dev commented 2 months ago

I edited the README.md file to include installation steps on both intel oneAPI and AdaptiveCpp. I also added links to to setting environment variables via setvars.sh if using the intel compiler.

cbyrohl commented 2 months ago

Not all distributions, particularly on HPC resources, have the "setvars.sh" script located at /opt/intel/oneapi/setvars.sh. Furthermore, the README implies one can use the AdaptiveCpp compiler instead of Intel oneAPI. Thus, I would recommend you move the setvars.sh out of the make.sh script. Instead provide a README subsection on environment preparation (for the Intel compiler in this case) or add optional environment scripts for the respective compilers and add some link to the respective oneapi documentation when the setvars.sh is not found at set location.