Skip to content

Commit

Permalink
Add preliminary support for UGE (an open source fork of SGE)
Browse files Browse the repository at this point in the history
SGE's parallel environment specification is space-delimited and UGE is comma delimited.
This handles fix handles both. Other than that it seems to work fine.

Thanks to Andrew Oler for spotting this issue, figuring out what wrong, suggesting
the fix and testing the fix.
  • Loading branch information
roryk committed Jan 25, 2016
1 parent a71c94e commit 5097996
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions cluster_helper/cluster.py
Original file line number Diff line number Diff line change
Expand Up @@ -16,6 +16,7 @@
import shutil
import subprocess
import time
import re
from distutils.version import LooseVersion
import sys

Expand Down Expand Up @@ -402,8 +403,7 @@ def _queue_can_access_pe(pe_name, queue):
queue_config = _parseSGEConf(subprocess.check_output(["qconf", "-sq", queue]))
except:
return False
for test_pe_name in queue_config["pe_list"].split():
test_pe_name = test_pe_name.split(",")[0].strip()
for test_pe_name in re.split('\W+|,', queue_config["pe_list"]):
if test_pe_name == pe_name:
return True
return False
Expand Down

0 comments on commit 5097996

Please sign in to comment.