search

JuliaParallel / ClusterManagers.jl

Other
232 stars 74 forks source link

Fix for SGE manager #159

Closed Moelf closed 3 years ago

Moelf commented 3 years ago

Fix #158

(which was fixed broken by #153

bjarthur commented 3 years ago

the idea behind #153 was that you need to add the -q flag yourself to the addprocs_sge. removing all such flags from the code provides maximum flexibility to the user to adapt to different SGE installations.

bjarthur commented 3 years ago

see https://github.com/JuliaParallel/ClusterManagers.jl/issues/151#issuecomment-731230616

Moelf commented 3 years ago

@bjarthur but the SGEManager takes the queue field, this is broken right now.

checkout: #158

The most broken part is that a simple addprocs_sge(3) won't work, this is extremely confusing.