Move magic constants into internal

src/benchcab/internal.py

@@ -12,6 +12,12 @@
 
 CONFIG_REQUIRED_KEYS = ["realisations", "modules"]
 
+# CMake module used for compilation:
+CMAKE_MODULE = "cmake/3.24.2"
+
+# Number of parallel jobs used when compiling with CMake:
+CMAKE_BUILD_PARALLEL_LEVEL = 4
+
 # Parameters for job script:
 QSUB_FNAME = "benchmark_cable_qsub.sh"
 FLUXSITE_DEFAULT_PBS: PBSConfig = {

src/benchcab/model.py

@@ -121,12 +121,13 @@ def custom_build(self, modules: list[str]):
     def build(self, modules: list[str], mpi=False):
         """Build CABLE with CMake."""
         path_to_repo = internal.SRC_DIR / self.name
-        nproc_default = 4
         cmake_args = ["-DCMAKE_BUILD_TYPE=Release"]
         if mpi:
             cmake_args.append('-DCABLE_MPI="ON"')
 
-        with chdir(path_to_repo), self.modules_handler.load(["cmake/3.24.2", *modules]):
+        with chdir(path_to_repo), self.modules_handler.load(
+            [internal.CMAKE_MODULE, *modules]
+        ):
             env = os.environ.copy()
             # This is required so that the netcdf-fortran library is discoverable by
             # pkg-config:
@@ -141,7 +142,7 @@ def build(self, modules: list[str], mpi=False):
                     "CMAKE_PREFIX_PATH", f"{env['OPENMPI_BASE']}/include/Intel", env=env
                 )
 
-            env["CMAKE_BUILD_PARALLEL_LEVEL"] = str(nproc_default)
+            env["CMAKE_BUILD_PARALLEL_LEVEL"] = str(internal.CMAKE_BUILD_PARALLEL_LEVEL)
 
             self.subprocess_handler.run_cmd(
                 "cmake -S . -B build " + " ".join(cmake_args), env=env