Inconsistent handling of queue parameter
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.