Build updates and improvements #10
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR makes a few updates to the CMake files and docker build scripts based on recent issues/suggestions:
Update Docker build to NVHPC SDK 23.7: This is related to CMakeList.txt doesn't support CUDA Arch 89 #8 as
cc89support was introduced in the 23.7 version of the compilers. Updating to this version so that the docker build will support this use case.Build fixes from cmake cannot locate MPI fortran (from NVHPC 23.7) #3:
CMP0057policyg++for compiling CXX files to support__rdtsc()usage in torchFixes docker build uses large amount of memory when running with more than 4 cores #9:
-j$(nproc)in Dockerfilemakecommands to limit parallel build processes to number of cores. Can be further limited by enforcing CPU core limits indocker buildcommand if neccessary.