Qa/follow set naming (#867)
* Follow set naming * Follow set naming * Disable type hints * Uniform order
This commit is contained in:
@@ -82,11 +82,11 @@ def delete_jobs(halt_on_error : bool) -> bool:
|
||||
|
||||
def find_jobs(job_status : JobStatus) -> JobSet:
|
||||
job_ids = find_job_ids(job_status)
|
||||
jobs : JobSet = {}
|
||||
job_set : JobSet = {}
|
||||
|
||||
for job_id in job_ids:
|
||||
jobs[job_id] = read_job_file(job_id)
|
||||
return jobs
|
||||
job_set[job_id] = read_job_file(job_id)
|
||||
return job_set
|
||||
|
||||
|
||||
def find_job_ids(job_status : JobStatus) -> List[str]:
|
||||
@@ -188,7 +188,6 @@ def set_step_status(job_id : str, step_index : int, step_status : JobStepStatus)
|
||||
|
||||
if job:
|
||||
steps = job.get('steps')
|
||||
|
||||
if has_step(job_id, step_index):
|
||||
steps[step_index]['status'] = step_status
|
||||
return update_job_file(job_id, job)
|
||||
|
||||
@@ -101,12 +101,12 @@ def clean_steps(job_id: str) -> bool:
|
||||
|
||||
def collect_output_set(job_id : str) -> JobOutputSet:
|
||||
steps = job_manager.get_steps(job_id)
|
||||
output_set : JobOutputSet = {}
|
||||
job_output_set : JobOutputSet = {}
|
||||
|
||||
for index, step in enumerate(steps):
|
||||
output_path = step.get('args').get('output_path')
|
||||
|
||||
if output_path:
|
||||
step_output_path = job_manager.get_step_output_path(job_id, index, output_path)
|
||||
output_set.setdefault(output_path, []).append(step_output_path)
|
||||
return output_set
|
||||
job_output_set.setdefault(output_path, []).append(step_output_path)
|
||||
return job_output_set
|
||||
|
||||
Reference in New Issue
Block a user