Update app.py

app.py

@@ -118,12 +118,13 @@ class UniversalTranscriptParser:
         gpa_data = {}
         gpa_match = re.search(r'Cum\. GPA\s*\|\s*([\d\.]+)', text)
         if gpa_match:
-            gpa_data['unweighted'] = gpa_match.group(1)
-            gpa_data['weighted'] = gpa_match.group(1)  # Homeschool often has same weighted/unweighted
+            gpa_value = gpa_match.group(1)
+            gpa_data['unweighted'] = gpa_value
+            gpa_data['weighted'] = gpa_value  # Homeschool often has same weighted/unweighted
         
         return {
             'student_info': student_info,
-            'courses': {'All': courses},  # Homeschool doesn't separate by grade in same way
+            'courses': {'All': courses},
             'gpa': gpa_data,
             'grade_level': current_grade.replace('th Grade', '') if current_grade else "Unknown"
         }
@@ -234,8 +235,8 @@ class UniversalTranscriptParser:
         
         # Extract GPA information
         gpa_data = {
-            'weighted': extract_gpa(text, 'Weighted GPA'),
-            'unweighted': extract_gpa(text, 'Un-weighted GPA')
+            'weighted': self._extract_gpa(text, 'Weighted GPA'),
+            'unweighted': self._extract_gpa(text, 'Un-weighted GPA')
         }
         
         # Extract current grade level
@@ -248,6 +249,12 @@ class UniversalTranscriptParser:
             'grade_level': grade_level
         }
 
+    def _extract_gpa(self, text: str, gpa_type: str) -> str:
+        """Helper method to extract GPA values"""
+        pattern = rf'{gpa_type}\s*([\d\.]+)'
+        match = re.search(pattern, text)
+        return match.group(1) if match else "Not Available"
+
     def _compile_miami_dade_patterns(self):
         return {
             'student': re.compile(r'Current Grade:\s*(\d+).*YOG\s*(\d{4})'),
@@ -274,11 +281,6 @@ class UniversalTranscriptParser:
             )
         }
 
-def extract_gpa(text, gpa_type):
-    pattern = rf'{gpa_type}\s*([\d\.]+)'
-    match = re.search(pattern, text)
-    return match.group(1) if match else "N/A"
-
 def parse_transcript(file):
     parser = UniversalTranscriptParser()
     
@@ -290,11 +292,15 @@ def parse_transcript(file):
         
         parsed_data = parser.parse_transcript(text)
         
-        # Only show GPA in the output
-        output_text = f"Transcript Processed Successfully!\n\n"
-        output_text += f"GPA Information:\n"
-        output_text += f"- Weighted: {parsed_data['gpa'].get('weighted', 'N/A')}\n"
-        output_text += f"- Unweighted: {parsed_data['gpa'].get('unweighted', 'N/A')}"
+        # Create clear GPA output
+        gpa_data = parsed_data.get('gpa', {})
+        weighted_gpa = gpa_data.get('weighted', 'Not Available')
+        unweighted_gpa = gpa_data.get('unweighted', 'Not Available')
+        
+        output_text = "Transcript Processed Successfully!\n\n"
+        output_text += "GPA Information:\n"
+        output_text += f"- Weighted GPA: {weighted_gpa}\n"
+        output_text += f"- Unweighted GPA: {unweighted_gpa}\n"
         
         return output_text, parsed_data
     else:
@@ -449,18 +455,23 @@ def save_profile(name, age, interests, transcript, learning_style,
     with open(json_path, "w") as f:
         json.dump(data, f, indent=2)
 
+    # Create profile summary with clear GPA display
+    gpa = transcript.get('gpa', {})
     markdown_summary = f"""### Student Profile: {name}
 **Age:** {age}  
 **Interests:** {interests}  
 **Learning Style:** {learning_style}  
+
 #### GPA Information:
-- Weighted: {transcript['gpa'].get('weighted', 'N/A')}
-- Unweighted: {transcript['gpa'].get('unweighted', 'N/A')}
+- Weighted GPA: {gpa.get('weighted', 'Not Available')}
+- Unweighted GPA: {gpa.get('unweighted', 'Not Available')}
+
 #### Favorites:
 - Movie: {favorites['movie']} ({favorites['movie_reason']})
 - Show: {favorites['show']} ({favorites['show_reason']})
 - Book: {favorites['book']} ({favorites['book_reason']})
 - Character: {favorites['character']} ({favorites['character_reason']})
+
 #### Blog:
 {blog if blog else "_No blog provided_"}
 """
@@ -498,16 +509,29 @@ def generate_response(message, history):
     # Common responses
     greetings = ["hi", "hello", "hey"]
     study_help = ["study", "learn", "prepare", "exam"]
-    grade_help = ["gpa", "grade point average", "grades"]
+    grade_help = ["gpa", "grade", "weighted", "unweighted", "grades"]
     course_help = ["courses", "classes", "subjects"]
     
     if any(greet in message.lower() for greet in greetings):
         return f"Hello {profile.get('name', 'there')}! How can I help you today?"
     
     elif any(word in message.lower() for word in grade_help):
-        return (f"Your GPA information:\n"
-               f"- Weighted: {gpa.get('weighted', 'N/A')}\n"
-               f"- Unweighted: {gpa.get('unweighted', 'N/A')}")
+        response = "Your GPA Information:\n"
+        response += f"- Weighted GPA: {gpa.get('weighted', 'Not Available')}\n"
+        response += f"- Unweighted GPA: {gpa.get('unweighted', 'Not Available')}\n"
+        
+        # Add interpretation if available
+        weighted = gpa.get('weighted')
+        if weighted and weighted.replace('.', '').isdigit():
+            weighted_num = float(weighted)
+            if weighted_num >= 3.5:
+                response += "\nExcellent GPA! You're doing great!"
+            elif weighted_num >= 3.0:
+                response += "\nGood GPA! Keep up the good work!"
+            else:
+                response += "\nConsider focusing on improving your grades."
+        
+        return response
     
     elif any(word in message.lower() for word in study_help):
         # Analyze course performance to give personalized advice
@@ -631,5 +655,4 @@ with gr.Blocks() as app:
 
 if __name__ == "__main__":
     app.launch()
-