PyKubeGrader 0.2.0__tar.gz → 0.2.2__tar.gz

{pykubegrader-0.2.0/src/PyKubeGrader.egg-info → pykubegrader-0.2.2}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.1
 Name: PyKubeGrader
-Version: 0.2.0
+Version: 0.2.2
 Summary: Add a short description here!
 Home-page: https://github.com/pyscaffold/pyscaffold/
 Author: jagar2

{pykubegrader-0.2.0 → pykubegrader-0.2.2/src/PyKubeGrader.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.1
 Name: PyKubeGrader
-Version: 0.2.0
+Version: 0.2.2
 Summary: Add a short description here!
 Home-page: https://github.com/pyscaffold/pyscaffold/
 Author: jagar2

{pykubegrader-0.2.0 → pykubegrader-0.2.2}/src/pykubegrader/build/build_folder.py

@@ -503,6 +503,9 @@ class NotebookProcessor:
             NotebookProcessor.add_initialization_code(
                 temp_notebook_path, self.week, self.assignment_type
             )
+            NotebookProcessor.replace_temp_in_notebook(
+                temp_notebook_path, temp_notebook_path
+            )
             return None, 0
 
     @staticmethod
@@ -535,7 +538,7 @@ class NotebookProcessor:
         index, cell = find_first_code_cell(notebook_path)
         cell = cell["source"]
         import_text = "from pykubegrader.initialize import initialize_assignment\n"
-        cell += f'\nresponses = initialize_assignment("{os.path.splitext(os.path.basename(notebook_path))[0]}", "{week}", "{assignment_type}" )\n'
+        import_text += f'\nresponses = initialize_assignment("{os.path.splitext(os.path.basename(notebook_path))[0]}", "{week}", "{assignment_type}" )\n'
         cell = f"{import_text}\n" + cell
         replace_cell_source(notebook_path, index, cell)
 
@@ -1398,6 +1401,8 @@ def clean_notebook(notebook_path):
                     cell.metadata["editable"] = cell.metadata.get("editable", False)
                     cell.metadata["deletable"] = cell.metadata.get("deletable", False)
                 if cell.cell_type == "code":
+                    if "grader.check(" in cell.source:
+                        continue
                     cell.metadata["tags"] = cell.metadata.get("tags", [])
                     if "skip-execution" not in cell.metadata["tags"]:
                         cell.metadata["tags"].append("skip-execution")
@@ -1563,7 +1568,7 @@ def generate_mcq_file(data_dict, output_file="mc_questions.py"):
                     )
                     f.write("    def __init__(self):\n")
                     f.write("        super().__init__(\n")
-                    f.write(f"            title=f'{q_value['question_text']}',\n")
+                    f.write(f'            title=f"{q_value['question_text']}",\n')
                     f.write("            style=MCQ,\n")
                     f.write(
                         f"            question_number={q_value['question number']},\n"

{pykubegrader-0.2.0 → pykubegrader-0.2.2}/src/pykubegrader/telemetry.py

@@ -10,6 +10,7 @@ import requests
 from IPython.core.interactiveshell import ExecutionInfo
 from requests import Response
 from requests.auth import HTTPBasicAuth
+from requests.exceptions import RequestException
 
 #
 # Logging setup
@@ -122,7 +123,6 @@ def update_responses(key: str, value) -> dict:
 #
 
 
-# TODO: Improve error handling
 def score_question(
     term: str = "winter_2025",
     base_url: str = "https://engr-131-api.eastus.cloudapp.azure.com",
@@ -140,16 +140,24 @@ def score_question(
         "responses": responses,
     }
 
-    res = requests.post(url, json=payload, auth=HTTPBasicAuth("student", "capture"))
-
-    res_data: dict[str, tuple[float, float]] = res.json()
-
-    for question, (points_earned, max_points) in res_data.items():
-        log_variable(
-            assignment_name=responses["assignment"],
-            value=f"{points_earned}, {max_points}",
-            info_type=question,
-        )
+    try:
+        res = requests.post(url, json=payload, auth=HTTPBasicAuth("student", "capture"))
+        res.raise_for_status()
+
+        res_data: dict[str, tuple[float, float]] = res.json()
+
+        for question, (points_earned, max_points) in res_data.items():
+            log_variable(
+                assignment_name=responses["assignment"],
+                value=f"{points_earned}, {max_points}",
+                info_type=question,
+            )
+    except RequestException as e:
+        raise RuntimeError("Failed to access question-scoring endpoint") from e
+    except ValueError as e:
+        raise ValueError("Failed to parse question-scoring JSON response") from e
+    except Exception as e:
+        raise RuntimeError("Failed to score question") from e
 
 
 def submit_question(