QuizGenerator 0.4.4__py3-none-any.whl → 0.5.1__py3-none-any.whl

QuizGenerator/premade_questions/cst334/persistence_questions.py

@@ -54,7 +54,15 @@ class HardDriveAccessTime(IOQuestion, TableQuestionMixin, BodyTemplatesMixin):
       ),
     })
   
-  def get_body(self, *args, **kwargs) -> ContentAST.Section:
+  def _get_body(self, *args, **kwargs):
+    """Build question body and collect answers."""
+    answers = [
+      self.answers["answer__rotational_delay"],
+      self.answers["answer__access_delay"],
+      self.answers["answer__transfer_delay"],
+      self.answers["answer__disk_access_delay"],
+    ]
+
     # Create parameter info table using mixin
     parameter_info = {
       "Hard Drive Rotation Speed": f"{self.hard_drive_rotation_speed}RPM",
@@ -84,8 +92,8 @@ class HardDriveAccessTime(IOQuestion, TableQuestionMixin, BodyTemplatesMixin):
     intro_text = "Given the information below, please calculate the following values."
 
     instructions = (
-      f"Make sure your answers are rounded to {Answer.DEFAULT_ROUNDING_DIGITS} decimal points "
-      f"(even if they are whole numbers), and do so after you finish all your calculations! "
+      f"Make sure that if you round your answers you use the unrounded values for your final calculations, "
+      f"otherwise you may introduce error into your calculations."
       f"(i.e. don't use your rounded answers to calculate your overall answer)"
     )
 
@@ -96,9 +104,14 @@ class HardDriveAccessTime(IOQuestion, TableQuestionMixin, BodyTemplatesMixin):
       additional_instructions=instructions
     )
 
+    return body, answers
+
+  def get_body(self, *args, **kwargs) -> ContentAST.Section:
+    """Build question body (backward compatible interface)."""
+    body, _ = self._get_body(*args, **kwargs)
     return body
-  
-  def get_explanation(self) -> ContentAST.Section:
+
+  def _get_explanation(self):
     explanation = ContentAST.Section()
     
     explanation.add_element(
@@ -150,6 +163,11 @@ class HardDriveAccessTime(IOQuestion, TableQuestionMixin, BodyTemplatesMixin):
         f"= {self.number_of_reads} \\cdot {self.access_delay:0.2f} + {self.transfer_delay:0.2f} "
         f"= {self.disk_access_delay:0.2f}ms")
     ])
+    return explanation, []
+
+  def get_explanation(self) -> ContentAST.Section:
+    """Build question explanation (backward compatible interface)."""
+    explanation, _ = self._get_explanation()
     return explanation
 
 
@@ -178,7 +196,15 @@ class INodeAccesses(IOQuestion, TableQuestionMixin, BodyTemplatesMixin):
       "answer__inode_index_in_block": Answer.integer("answer__inode_index_in_block", self.inode_index_in_block),
     })
   
-  def get_body(self) -> ContentAST.Section:
+  def _get_body(self):
+    """Build question body and collect answers."""
+    answers = [
+      self.answers["answer__inode_address"],
+      self.answers["answer__inode_block"],
+      self.answers["answer__inode_address_in_block"],
+      self.answers["answer__inode_index_in_block"],
+    ]
+
     # Create parameter info table using mixin
     parameter_info = {
       "Block Size": f"{self.block_size} Bytes",
@@ -206,21 +232,21 @@ class INodeAccesses(IOQuestion, TableQuestionMixin, BodyTemplatesMixin):
     # Use mixin to create complete body with both tables
     intro_text = "Given the information below, please calculate the following values."
 
-    instructions = (
-      "(hint: they should all be round numbers). "
-      "Remember, demonstrating you know the equations and what goes into them is generally sufficient."
-    )
-
     body = self.create_parameter_calculation_body(
       intro_text=intro_text,
       parameter_table=parameter_table,
       answer_table=answer_table,
-      additional_instructions=instructions
+      # additional_instructions=instructions
     )
 
+    return body, answers
+
+  def get_body(self) -> ContentAST.Section:
+    """Build question body (backward compatible interface)."""
+    body, _ = self._get_body()
     return body
-  
-  def get_explanation(self) -> ContentAST.Section:
+
+  def _get_explanation(self):
     explanation = ContentAST.Section()
     
     explanation.add_element(
@@ -292,7 +318,12 @@ class INodeAccesses(IOQuestion, TableQuestionMixin, BodyTemplatesMixin):
         f"{self.inode_index_in_block}"
       ]
     ))
-    
+
+    return explanation, []
+
+  def get_explanation(self) -> ContentAST.Section:
+    """Build question explanation (backward compatible interface)."""
+    explanation, _ = self._get_explanation()
     return explanation
 
 
@@ -330,40 +361,42 @@ class VSFS_states(IOQuestion):
       f"{operations[-1]['cmd']}",
       kind=Answer.AnswerKind.MULTIPLE_DROPDOWN,
       correct=True,
-      baffles=list(set([op['cmd'] for op in operations[:-1] if op != operations[-1]['cmd']]))
+      baffles=list(set([op['cmd'] for op in operations[:-1] if op != operations[-1]['cmd']])),
+      label="Command"
     )
   
-  def get_body(self) -> ContentAST.Section:
+  def _get_body(self):
+    """Build question body and collect answers."""
+    answers = [self.answers["answer__cmd"]]
+
     body = ContentAST.Section()
-    
+
     body.add_element(ContentAST.Paragraph(["What operation happens between these two states?"]))
-    
+
     body.add_element(
       ContentAST.Code(
         self.start_state,
         make_small=True
       )
     )
-    
-    body.add_element(
-      ContentAST.AnswerBlock(
-        ContentAST.Answer(
-          self.answers["answer__cmd"],
-          label="Command"
-        )
-      )
-    )
-    
+
+    body.add_element(ContentAST.AnswerBlock(self.answers["answer__cmd"]))
+
     body.add_element(
       ContentAST.Code(
         self.end_state,
         make_small=True
       )
     )
-    
+
+    return body, answers
+
+  def get_body(self) -> ContentAST.Section:
+    """Build question body (backward compatible interface)."""
+    body, _ = self._get_body()
     return body
-  
-  def get_explanation(self) -> ContentAST.Section:
+
+  def _get_explanation(self):
     explanation = ContentAST.Section()
     
     log.debug(f"self.start_state: {self.start_state}")
@@ -446,6 +479,11 @@ class VSFS_states(IOQuestion):
         highlight_changes(self.start_state, self.end_state)
       )
     )
-    
+
+    return explanation, []
+
+  def get_explanation(self) -> ContentAST.Section:
+    """Build question explanation (backward compatible interface)."""
+    explanation, _ = self._get_explanation()
     return explanation