Skip to content

Commit bffaef4

Browse files
committed
changed job_num to task_id in current_jobs_parsed.
we use "in" to check and since we are checking an int against a list of strings, we will always get false.
1 parent 62c83c6 commit bffaef4

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

nipype/pipeline/plugins/sge.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -196,12 +196,12 @@ def _parse_qstat_job_list(self, xml_job_list):
196196
job_queue_state, job_time, job_queue_name, job_slots)
197197
sge_debug_print("Updating job: {0}".format(
198198
self._task_dictionary[task_id]))
199-
current_jobs_parsed.append(job_num)
199+
current_jobs_parsed.append(task_id)
200200
# Changed from job_num as "in" is used to check which does not cast
201201
else:
202202
# Any Job that was not explicitly added with qsub command is
203203
# out of scope
204-
self._out_of_scope_jobs.append(int(task_id))
204+
self._out_of_scope_jobs.append(task_id)
205205

206206
# To ensure that every job is in the dictionary has a state reported
207207
# by the SGE environment, it is necessary to explicitly check jobs

0 commit comments

Comments
 (0)