Inconsistent handling of queue parameter
The Simulation.run()
function takes a parameter queue
, but it is not used on Merlin6. Instead, an environment variable SLURM_PARTITION
is used.
This should be more consistent.
I would generally advise against the use of environment variables in Simulation
. Currently, it is not obvious where a setting is coming from, which environment variables may/have to be set, and what their meaning is.
I suggest to read the env variables in the command line script, and pass their values to the constructor of the class or the run method.