fix: Do not (unnecessarily!) copy files twice in Autograde converter

natkam · natkam · commit a8534a7d9385 · 2025-11-21T19:33:12.000+01:00
diff --git a/grader_service/convert/converters/autograde.py b/grader_service/convert/converters/autograde.py
@@ -74,8 +74,6 @@ def convert_notebooks(self) -> None:
         # check for missing notebooks and give them a score of zero if they do not exist
         json_path = os.path.join(self._output_directory, "gradebook.json")
         with Gradebook(json_path) as gb:
-            self.copy_unmatched_files(gb)
-
             glob_notebooks = {
                 self.init_single_notebook_resources(n)["unique_key"]: n for n in self.notebooks
             }

Original file line number	Diff line number	Diff line change
`@@ -74,8 +74,6 @@ def convert_notebooks(self) -> None:`
`74`	`74`	`# check for missing notebooks and give them a score of zero if they do not exist`
`75`	`75`	`json_path = os.path.join(self._output_directory, "gradebook.json")`
`76`	`76`	`with Gradebook(json_path) as gb:`
`77`		`- self.copy_unmatched_files(gb)`
`78`		`-`
`79`	`77`	`glob_notebooks = {`
`80`	`78`	`self.init_single_notebook_resources(n)["unique_key"]: n for n in self.notebooks`
`81`	`79`	`}`