Add configurable Dask frame execution with SLURM-backed HPC support#354
Conversation
…d-parallelisation
…d-parallelisation
jimboid
left a comment
There was a problem hiding this comment.
This PR looks very complete. The code additions to core functionality look like they are in line with the parallel version I wrote in the water-entropy code. It looks like the code has been sufficiently adapted to this code base with complete and thorough treatment of parameter handling. Documentation looks complete and detailed, the tests look detailed and complete like on the WE implementation. I don't see any surprises with the HPC set up logic, however we should not leave this to chance. In such a PR I would like to see the following data/screenshots or have someone verify the following points (since unit tests only go so far when there is an element of infrastructure involved):
- Performance characterisation difference for serial implementation and parallel implementation for a local run. Total execution time is sufficient for say 20 - 50 frames.
- Testing that the submission script on a known hpc platform is generated correctly for any automatic submission mode
- Testing that the auto submission and conda detection machinery is working correctly and that the dask scheduler is submitted correctly.
- That the performance data for a single node is inline and likely much stronger than those in point 1.
jimboid
left a comment
There was a problem hiding this comment.
Scaling behaviour is not as immediately strong as for WE, but this suggests that as we discussed there are other parts of the code that might need to be optimised. Please go ahead and merge








Summary
This PR adds configurable parallel frame-by-frame execution to CodeEntropy using Dask, with support for both local Dask workers and SLURM-backed HPC execution. It also adds runtime submission support for SLURM master jobs, validates the new parallel/HPC configuration options, and updates the documentation to explain the new arguments and parallel execution modes.
Changes
Parallel frame execution:
LevelDAG.Local Dask and SLURM/HPC configuration:
parallel_frames,use_dask,dask_workers, anddask_threads_per_worker.hpc,submit,hpc_queue,hpc_nodes,hpc_cores,hpc_processes,hpc_memory,hpc_walltime,hpc_account,hpc_qos,hpc_constraint,conda_path,conda_exec, andconda_env.HPCDaskManagerfor configuring SLURM-backed Dask clusters and generating/submitting master SLURM scripts.submit=True, allowing CodeEntropy to submit a master SLURM job and exit before starting local analysis.Tests and documentation:
LevelDAGtests into a single test file to reduce duplication.Impact