PyKubeGrader 0.1.18__tar.gz → 0.1.19__tar.gz

{pykubegrader-0.1.18/src/PyKubeGrader.egg-info → pykubegrader-0.1.19}/PKG-INFO

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

{pykubegrader-0.1.18 → pykubegrader-0.1.19/src/PyKubeGrader.egg-info}/PKG-INFO

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

{pykubegrader-0.1.18 → pykubegrader-0.1.19}/src/pykubegrader/build/build_folder.py

@@ -64,7 +64,7 @@ class NotebookProcessor:
             __name__
         )  # Create a logger instance specific to this module
         self.logger = logger  # Assign the logger instance to the class for use in instance methods
-
+        
         self.total_point_log = {}
 
     def process_notebooks(self):
@@ -120,12 +120,10 @@ class NotebookProcessor:
 
                 # Process the notebook if it meets the criteria
                 self._process_single_notebook(notebook_path)
-
+                
         # Write the dictionary to a JSON file
         with open(f"{self.solutions_folder}/total_points.json", "w") as json_file:
-            json.dump(
-                self.total_point_log, json_file, indent=4
-            )  # `indent=4` for pretty formatting
+            json.dump(self.total_point_log, json_file, indent=4)  # `indent=4` for pretty formatting
 
     def _print_and_log(self, message):
         """
@@ -178,7 +176,7 @@ class NotebookProcessor:
         Returns:
             None
         """
-
+        
         self.select_many_total_points = 0
         self.mcq_total_points = 0
         self.tf_total_points = 0
@@ -290,16 +288,11 @@ class NotebookProcessor:
                 os.path.join(student_path, question_file_name_sanitized),
                 os.path.join(questions_folder_jbook, question_file_name_sanitized),
             )
-
-        total_points = (
-            self.select_many_total_points
-            + self.mcq_total_points
-            + self.tf_total_points
-            + self.otter_total_points
-        )
-
-        self.total_point_log.update({notebook_name: total_points})
-
+            
+        total_points = self.select_many_total_points + self.mcq_total_points + self.tf_total_points + self.otter_total_points
+            
+        self.total_point_log.update({notebook_name: total_points})    
+            
     def free_response_parser(
         self, temp_notebook_path, notebook_subfolder, notebook_name
     ):
@@ -316,7 +309,7 @@ class NotebookProcessor:
 
             shutil.copy("./keys/.client_private_key.bin", client_private_key)
             shutil.copy("./keys/.server_public_key.bin", server_public_key)
-
+            
             client_private_key = os.path.join(
                 notebook_subfolder,
                 ".client_private_key.bin",
@@ -329,9 +322,7 @@ class NotebookProcessor:
             shutil.copy("./keys/.client_private_key.bin", client_private_key)
             shutil.copy("./keys/.server_public_key.bin", server_public_key)
 
-            out = FastAPINotebookBuilder(
-                notebook_path=temp_notebook_path, assignment_tag=self.assignment_tag
-            )
+            out = FastAPINotebookBuilder(notebook_path=temp_notebook_path, assignment_tag=self.assignment_tag)
 
             debug_notebook = os.path.join(
                 notebook_subfolder,
@@ -409,6 +400,7 @@ class NotebookProcessor:
         replace_cell_source(notebook_path, index, cell)
 
     def multiple_choice_parser(self, temp_notebook_path, new_notebook_path):
+        
         ### Parse the notebook for multiple choice questions
         if self.has_assignment(temp_notebook_path, "# BEGIN MULTIPLE CHOICE"):
             self._print_and_log(
@@ -428,9 +420,7 @@ class NotebookProcessor:
 
             for data_ in data:
                 # Generate the solution file
-                self.mcq_total_points = self.generate_solution_MCQ(
-                    data, output_file=solution_path
-                )
+                self.mcq_total_points = self.generate_solution_MCQ(data, output_file=solution_path)
 
                 question_path = (
                     f"{new_notebook_path.replace(".ipynb", "")}_questions.py"
@@ -470,9 +460,7 @@ class NotebookProcessor:
 
             # for data_ in data:
             # Generate the solution file
-            self.tf_total_points = self.generate_solution_MCQ(
-                data, output_file=solution_path
-            )
+            self.tf_total_points = self.generate_solution_MCQ(data, output_file=solution_path)
 
             question_path = f"{new_notebook_path.replace(".ipynb", "")}_questions.py"
 
@@ -508,9 +496,7 @@ class NotebookProcessor:
 
             # for data_ in data:
             # Generate the solution file
-            self.select_many_total_points = self.generate_solution_MCQ(
-                data, output_file=solution_path
-            )
+            self.select_many_total_points = self.generate_solution_MCQ(data, output_file=solution_path)
 
             question_path = f"{new_notebook_path.replace(".ipynb", "")}_questions.py"
 
@@ -716,7 +702,7 @@ class NotebookProcessor:
                 solutions.update(existing_module.solutions)
             if hasattr(existing_module, "total_points"):
                 total_points.extend(existing_module.total_points)
-
+        
         question_points = 0
         # Process new question data and update solutions and total_points
         for question_set in data_list:
@@ -729,14 +715,14 @@ class NotebookProcessor:
         # Write updated total_points and solutions back to the file
         with open(output_file, "w", encoding="utf-8") as f:
             f.write("from typing import Any\n\n")
-            f.write(f"total_points: list[float] = {total_points}\n\n")
+            f.write(f"total_points: float = {total_points}\n\n")
 
             f.write("solutions: dict[str, Any] = {\n")
             for key, solution in solutions.items():
                 # For safety, we assume solutions are strings, but if not, repr would be safer
                 f.write(f'    "{key}": {repr(solution)},\n')
             f.write("}\n")
-
+            
         return question_points
 
     def extract_MCQ(ipynb_file):
@@ -1453,7 +1439,7 @@ def generate_mcq_file(data_dict, output_file="mc_questions.py"):
             keys = []
             for i, (q_key, q_value) in enumerate(question_dict.items()):
                 # Write keys
-                keys.append(f"q{q_value['subquestion_number']}-{q_value['name']}")
+                keys.append(f"q{q_value['question number']}-q{q_value['subquestion_number']}-{q_value['name']}")
 
             f.write(f"            keys={keys},\n")
 
@@ -1522,7 +1508,7 @@ def generate_select_many_file(data_dict, output_file="select_many_questions.py")
             keys = []
             for i, (q_key, q_value) in enumerate(question_dict.items()):
                 # Write keys
-                keys.append(f"q{q_value['subquestion_number']}-{q_value['name']}")
+                keys.append(f"q{q_value['question number']}-q{q_value['subquestion_number']}-{q_value['name']}")
 
             f.write(f"            keys={keys},\n")
 
@@ -1597,7 +1583,7 @@ def generate_tf_file(data_dict, output_file="tf_questions.py"):
             keys = []
             for i, (q_key, q_value) in enumerate(question_dict.items()):
                 # Write keys
-                keys.append(f"q{q_value['subquestion_number']}-{q_value['name']}")
+                keys.append(f"q{q_value['question number']}-q{q_value['subquestion_number']}-{q_value['name']}")
 
             f.write(f"            keys={keys},\n")
 
@@ -1685,18 +1671,16 @@ def main():
     parser.add_argument(
         "root_folder", type=str, help="Path to the root folder to process"
     )
-
+    
     parser.add_argument(
         "--assignment-tag",
         type=str,
         help="assignment-tag used for calculating grades",
         default="Reading-Week-X",
     )
-
+    
     args = parser.parse_args()
-    processor = NotebookProcessor(
-        root_folder=args.root_folder, assignment_tag=args.assignment_tag
-    )
+    processor = NotebookProcessor(root_folder=args.root_folder, assignment_tag=args.assignment_tag)
     processor.process_notebooks()