ExCALIBUR-NEPTUNE / NESO

MIT License
4 stars 4 forks source link

Feature/composite interaction base #223

Closed will-saunders-ukaea closed 1 month ago

will-saunders-ukaea commented 9 months ago

Description

Please include a summary of changes, motivation and context for this PR.

Fixes # (issue)

Type of change

Please delete options that are not relevant.

Testing

Please describe the tests that you ran to verify your changes and provide instructions for reproducibility. Please also list any relevant details for your test configuration.

Test Configuration:

Checklist:

######### DELETE THIS AND BELOW #########

How-to:

  1. Format C++ source code with clang-format
find ./src ./include ./test -iname \*.hpp -o -iname \*.cpp | xargs clang-format -i
  1. Format cmake files with cmake-format
cmake-format -i CMakeLists.txt src/nektar/CMakeLists.txt test/CMakeLists.txt
  1. Format python with black
find ./python -iname \*.py | xargs black
will-saunders-ukaea commented 1 month ago

I have updated the NP side such that this points at the dev branch rather than an I think this is ready for another review/tests running by someone else.

JamesEdgeley commented 1 month ago

I get some errors like this in the build phase for the dev branch of neso-particles using spack:

     509    In file included from /home/jedge/NESO/neso-particles/include/neso_particles/external_interfaces/common/../../compute_target.hpp:15:
     510    In file included from /home/jedge/NESO/neso-particles/include/neso_particles/containers/../loop/../sycl_typedefs.hpp:7:
     511    In file included from /home/jedge/.spack/opt/spack/linux-ubuntu22.04-icelake/gcc-11.4.0/intel-oneapi-compilers-2022.2.1-p55yrly2lxt7mhzrgsg35vdiancszqrd/compiler/2022.2.1/linux/bin-l
            lvm/../include/sycl/CL/sycl.hpp:16:
     512    In file included from /home/jedge/.spack/opt/spack/linux-ubuntu22.04-icelake/gcc-11.4.0/intel-oneapi-compilers-2022.2.1-p55yrly2lxt7mhzrgsg35vdiancszqrd/compiler/2022.2.1/linux/bin-l
            lvm/../include/sycl/CL/sycl/backend.hpp:18:
     513    In file included from /home/jedge/.spack/opt/spack/linux-ubuntu22.04-icelake/gcc-11.4.0/intel-oneapi-compilers-2022.2.1-p55yrly2lxt7mhzrgsg35vdiancszqrd/compiler/2022.2.1/linux/bin-l
            lvm/../include/sycl/CL/sycl/detail/backend_traits_opencl.hpp:26:
     514    In file included from /home/jedge/.spack/opt/spack/linux-ubuntu22.04-icelake/gcc-11.4.0/intel-oneapi-compilers-2022.2.1-p55yrly2lxt7mhzrgsg35vdiancszqrd/compiler/2022.2.1/linux/bin-l
            lvm/../include/sycl/CL/sycl/queue.hpp:20:
  >> 515    /home/jedge/.spack/opt/spack/linux-ubuntu22.04-icelake/gcc-11.4.0/intel-oneapi-compilers-2022.2.1-p55yrly2lxt7mhzrgsg35vdiancszqrd/compiler/2022.2.1/linux/bin-llvm/../include/sycl/CL
            /sycl/handler.hpp:1242:50: error: '__usmfill<RemotePoint>' is invalid; kernel name should be forward declarable at namespace scope
     516        kernel_parallel_for<KernelName, ElementType>(KernelFunc);

Am I using a compatible compiler?

will-saunders-ukaea commented 1 month ago

I'm using

intel-oneapi-compilers/2024.1.0-gcc-12.3.0-pcvjv7b
intel-oneapi-runtime/2024.1.0-oneapi-2024.1.0-ci4wevz
intel-oneapi-mpi/2021.12.1-oneapi-2024.1.0-ld7q7bt
intel-tbb/2021.9.0-oneapi-2024.1.0-fmgspbl
intel-oneapi-mkl/2024.0.0-oneapi-2024.1.0-3afo4xc

Intel were making noises about removing access to toolchains over a year (I think) hence I updated.

JamesEdgeley commented 1 month ago

Everything working now