A-New-BellHope / bellhopcuda

CUDA and C++ port of BELLHOP / BELLHOP3D underwater acoustics simulator
GNU General Public License v3.0
60 stars 7 forks source link

Support CUDA 12.4 #29

Closed oldstylejoe closed 4 months ago

oldstylejoe commented 5 months ago

First reported by Denton Woods: _It looks like Nvidia changed something in the CUDA Toolkit with version 12.2 on Windows. I have tried 12.2, 12.3, and 12.4, and they all fail to compile bellhopcuda, while 12.1 seems to work. Unfortunately, I'm required to use 12.2 or higher at work. I have also tried it on a Linux machine with 12.3, and it compiles. I get screens full of compilation errors relating to chrono conversions (e.g., C:\Program Files\NVIDIA GPU Computing Toolkit\CUDA\v12.2\include\cuda\std\detail\libcxx\include\chrono(906): error : class "cuda::std::__4::commontype<double, double>" has no member "type"). I'm guessing that you are using nvcc's chrono for timing the parts of the run. I'm attaching part of the error log in case it's helpful (the full thing was ~40k lines at 6 MB). This almost seems like an nvcc issue, but hopefully there's a workaround.

oldstylejoe commented 4 months ago

Fixed by #31