Skip to content

Commit

Permalink
fix confusion due to overwritten variable
Browse files Browse the repository at this point in the history
  • Loading branch information
NeoLegends committed Aug 20, 2024
1 parent 1f08daa commit e496821
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions sisyphus/manager.py
Original file line number Diff line number Diff line change
Expand Up @@ -577,9 +577,9 @@ def maybe_clear_state(state, always_clear, action):
return True

def handle_job_failure(self, prev_jobs: Dict[str, List[Job]], cur_jobs: Dict[str, List[Job]]):
prev_jobs = set(prev_jobs.get(gs.STATE_ERROR, []))
prev_errored_jobs = set(prev_jobs.get(gs.STATE_ERROR, []))
for job in cur_jobs.get(gs.STATE_ERROR, []):
if job not in prev_jobs:
if job not in prev_errored_jobs:
gs.on_job_failure(job)

@tools.default_handle_exception_interrupt_main_thread
Expand Down

0 comments on commit e496821

Please sign in to comment.