Skip to content

Commit

Permalink
Don't skip the assignment if a notebook raises an exception
Browse files Browse the repository at this point in the history
  • Loading branch information
tuncbkose committed Mar 15, 2023
1 parent 3a44ef4 commit 9a12d17
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion nbgrader/converters/base.py
Original file line number Diff line number Diff line change
Expand Up @@ -391,7 +391,7 @@ def _handle_failure(gd: typing.Dict[str, str]) -> None:
self.convert_single_notebook(notebook_filename)

# Exceptions that shouldn't interrupt the entire conversion process should go here
# Those that should go in outer try/except
# Those that should interrupt go in the outer try/except
except UnresponsiveKernelError:
self.log.error(
"While processing assignment %s, the kernel became "
Expand All @@ -405,6 +405,7 @@ def _handle_failure(gd: typing.Dict[str, str]) -> None:
errors.append((gd['assignment_id'], gd['student_id']))
_handle_failure(gd)

# Raise unhandled exceptions for the outer try/except
except Exception as e:
raise e

Expand Down

0 comments on commit 9a12d17

Please sign in to comment.