fixed expected 1 but got 0 and keyError 6 also added new ui and scrollable

app.py

@@ -77,20 +77,21 @@ def extract_subjects_and_marks_for_gradio(roll_no):
         headers = studentwise_data[0]
         roll_no = roll_no.strip().upper()
         
-        # Find the student row using cached data
+        # Replace indexing with loop search:
         student_row = None
         for row in studentwise_data[1:]:  # Skip header row
-            # Check multiple possible roll number columns (0, 1, 2)
-            for col_idx in [0, 1, 2]:
-                if col_idx < len(row) and row[col_idx].strip().upper() == roll_no:
+            # Check multiple columns for roll number
+            for i, cell in enumerate(row[:5]):  # Check first 5 columns
+                if cell.strip().upper() == roll_no:
                     student_row = row
                     break
-            if student_row:
+            if student_row is not None:
                 break
-
-        if not student_row:
+        
+        if student_row is None:
             return f"❌ Student with Roll No '{roll_no}' not found."
 
+        # Rest of the function remains the same...
         def get_value_if_not_empty(col_name):
             """Helper function to get value only if it's not empty"""
             if col_name in headers:
@@ -142,154 +143,150 @@ def extract_subjects_and_marks_for_gradio(roll_no):
             subject['total_marks'] = subject['ip1_marks'] + subject['ip2_marks']
 
         # Calculate grand totals
-        total_theory_ip1 = sum(s['ip1_points'] for s in theory_subjects)
-        total_theory_ip2 = sum(s['ip2_points'] for s in theory_subjects)
-        total_lab_ip1 = sum(s['ip1_points'] for s in lab_subjects)
-        total_lab_ip2 = sum(s['ip2_points'] for s in lab_subjects)
-        
-        grand_total_ip1 = total_theory_ip1 + total_lab_ip1
-        grand_total_ip2 = total_theory_ip2 + total_lab_ip2
-        grand_total_points = grand_total_ip1 + grand_total_ip2
-
-        # Calculate marks totals
-        total_theory_marks_ip1 = sum(s['ip1_marks'] for s in theory_subjects)
-        total_theory_marks_ip2 = sum(s['ip2_marks'] for s in theory_subjects)
-        total_lab_marks_ip1 = sum(s['ip1_marks'] for s in lab_subjects)
-        total_lab_marks_ip2 = sum(s['ip2_marks'] for s in lab_subjects)
+        total_theory_ip1_points = sum(s['ip1_points'] for s in theory_subjects)
+        total_theory_ip2_points = sum(s['ip2_points'] for s in theory_subjects)
+        total_lab_ip1_points = sum(s['ip1_points'] for s in lab_subjects)
+        total_lab_ip2_points = sum(s['ip2_points'] for s in lab_subjects)
         
-        grand_total_marks_ip1 = total_theory_marks_ip1 + total_lab_marks_ip1
-        grand_total_marks_ip2 = total_theory_marks_ip2 + total_lab_marks_ip2
-        grand_total_marks = grand_total_marks_ip1 + grand_total_marks_ip2
-
-        total_subjects = len(theory_subjects) + len(lab_subjects)
-
-        # SIMPLE TABLE FORMATTING WITHOUT CONFUSING SYMBOLS
-        def format_table_row(subject, ip1, ip2, total):
-            """Format a table row with clean alignment"""
-            # Fixed column widths
-            subject_col = 48
-            ip1_col = 10
-            ip2_col = 10
-            total_col = 12
-            
-            # Format subject column (left-aligned)
-            subject_str = str(subject)[:subject_col].ljust(subject_col)
-            
-            # Format numeric columns (right-aligned with fixed width)
-            ip1_str = str(ip1).rjust(ip1_col)
-            ip2_str = str(ip2).rjust(ip2_col)
-            total_str = str(total).rjust(total_col)
-            
-            return f"{subject_str} {ip1_str} {ip2_str} {total_str}"
+        total_theory_ip1_marks = sum(s['ip1_marks'] for s in theory_subjects)
+        total_theory_ip2_marks = sum(s['ip2_marks'] for s in theory_subjects)
+        total_lab_ip1_marks = sum(s['ip1_marks'] for s in lab_subjects)
+        total_lab_ip2_marks = sum(s['ip2_marks'] for s in lab_subjects)
 
-        # Build output with clean formatting
+        # Build output with clean card-style formatting
         output = []
         output.append("")
-        output.append("🏆 REWARD POINTS DISTRIBUTION")
-        output.append("-" * 88)
-        
-        # Header row
-        header_row = format_table_row("SUBJECTS", "IP - 1", "IP - 2", "TOTAL")
-        output.append(header_row)
-        output.append("-" * 88)
+        output.append("🏆 INNOVATIVE PRACTICE (IP) SUMMARY")
+        output.append("=" * 80)
         
-        # Theory courses section
+        # Theory subjects section
         if theory_subjects:
-            theory_header = format_table_row(f"THEORY COURSES - ({len(theory_subjects)} COURSES)", "", "", "")
-            output.append(theory_header)
+            output.append(f"\n📚 THEORY SUBJECTS ({len(theory_subjects)} subjects)")
+            output.append("-" * 50)
             
             for subject in theory_subjects:
-                ip1_val = f"{subject['ip1_points']:.2f}" if subject['ip1_points'] > 0 else ""
-                ip2_val = f"{subject['ip2_points']:.2f}" if subject['ip2_points'] > 0 else ""
-                total_val = f"{subject['total_points']:.2f}"
-                subject_row = format_table_row(subject['code'], ip1_val, ip2_val, total_val)
-                output.append(subject_row)
-        
-        # Add-on courses
-        addon_row = format_table_row("ADD-ON / HONOR / MINOR COURSES", "", "", "")
-        output.append(addon_row)
+                output.append(f"\n🔹 {subject['code']}")
+                
+                # Points section
+                if subject['ip1_points'] > 0 or subject['ip2_points'] > 0:
+                    points_line = "   Reward Points: "
+                    if subject['ip1_points'] > 0:
+                        points_line += f"IP-1: {subject['ip1_points']:.2f}"
+                    if subject['ip2_points'] > 0:
+                        if subject['ip1_points'] > 0:
+                            points_line += f" | IP-2: {subject['ip2_points']:.2f}"
+                        else:
+                            points_line += f"IP-2: {subject['ip2_points']:.2f}"
+                    points_line += f" | Total: {subject['total_points']:.2f}"
+                    output.append(points_line)
+                
+                # Marks section
+                if subject['ip1_marks'] > 0 or subject['ip2_marks'] > 0:
+                    marks_line = "   Internal Marks: "
+                    if subject['ip1_marks'] > 0:
+                        marks_line += f"IP-1: {subject['ip1_marks']:.2f}"
+                    if subject['ip2_marks'] > 0:
+                        if subject['ip1_marks'] > 0:
+                            marks_line += f" | IP-2: {subject['ip2_marks']:.2f}"
+                        else:
+                            marks_line += f"IP-2: {subject['ip2_marks']:.2f}"
+                    marks_line += f" | Total: {subject['total_marks']:.2f}"
+                    output.append(marks_line)
         
-        # Lab courses section
+        # Lab subjects section
         if lab_subjects:
-            lab_header = format_table_row(f"LAB. COURSES - ({len(lab_subjects)} COURSES)", "", "", "")
-            output.append(lab_header)
+            output.append(f"\n🧪 LAB SUBJECTS ({len(lab_subjects)} subjects)")
+            output.append("-" * 50)
             
             for subject in lab_subjects:
-                ip1_val = f"{subject['ip1_points']:.2f}" if subject['ip1_points'] > 0 else ""
-                ip2_val = f"{subject['ip2_points']:.2f}" if subject['ip2_points'] > 0 else ""
-                total_val = f"{subject['total_points']:.2f}"
-                subject_row = format_table_row(subject['code'], ip1_val, ip2_val, total_val)
-                output.append(subject_row)
-        else:
-            lab_header = format_table_row("LAB. COURSES - (0 COURSES)", "", "", "")
-            output.append(lab_header)
-        
-        # Total row
-        grand_ip1_val = f"{grand_total_ip1:,.2f}" if grand_total_ip1 > 0 else "0.00"
-        grand_ip2_val = f"{grand_total_ip2:,.2f}" if grand_total_ip2 > 0 else "0.00"
-        grand_total_val = f"{grand_total_points:,.2f}"
-        total_row = format_table_row(f"TOTAL - ({total_subjects} SUBJECTS)", grand_ip1_val, grand_ip2_val, grand_total_val)
-        output.append("-" * 88)
-        output.append(total_row)
-        
-        output.append("")
-        
-        # INTERNAL MARKS DISTRIBUTION
-        output.append("🎯 INTERNAL MARKS DISTRIBUTION")
-        output.append("-" * 88)
+                output.append(f"\n🔹 {subject['code']}")
+                
+                # Points section
+                if subject['ip1_points'] > 0 or subject['ip2_points'] > 0:
+                    points_line = "   Reward Points: "
+                    if subject['ip1_points'] > 0:
+                        points_line += f"IP-1: {subject['ip1_points']:.2f}"
+                    if subject['ip2_points'] > 0:
+                        if subject['ip1_points'] > 0:
+                            points_line += f" | IP-2: {subject['ip2_points']:.2f}"
+                        else:
+                            points_line += f"IP-2: {subject['ip2_points']:.2f}"
+                    points_line += f" | Total: {subject['total_points']:.2f}"
+                    output.append(points_line)
+                
+                # Marks section
+                if subject['ip1_marks'] > 0 or subject['ip2_marks'] > 0:
+                    marks_line = "   Internal Marks: "
+                    if subject['ip1_marks'] > 0:
+                        marks_line += f"IP-1: {subject['ip1_marks']:.2f}"
+                    if subject['ip2_marks'] > 0:
+                        if subject['ip1_marks'] > 0:
+                            marks_line += f" | IP-2: {subject['ip2_marks']:.2f}"
+                        else:
+                            marks_line += f"IP-2: {subject['ip2_marks']:.2f}"
+                    marks_line += f" | Total: {subject['total_marks']:.2f}"
+                    output.append(marks_line)
         
-        # Header for marks
-        marks_header = format_table_row("SUBJECTS", "IP - 1", "IP - 2", "TOTAL")
-        output.append(marks_header)
-        output.append("-" * 88)
+        # Summary section
+        output.append("\n" + "=" * 80)
+        output.append("📊 OVERALL SUMMARY")
+        output.append("=" * 80)
         
-        # Theory courses with marks
+        # Reward Points Summary
+        output.append("\n🏅 REWARD POINTS BREAKDOWN:")
         if theory_subjects:
-            theory_marks_header = format_table_row(f"THEORY COURSES - ({len(theory_subjects)} COURSES)", "", "", "")
-            output.append(theory_marks_header)
-            
-            for subject in theory_subjects:
-                ip1_marks = f"{subject['ip1_marks']:.2f}" if subject['ip1_marks'] > 0 else ""
-                ip2_marks = f"{subject['ip2_marks']:.2f}" if subject['ip2_marks'] > 0 else ""
-                total_marks = f"{subject['total_marks']:.2f}"
-                marks_row = format_table_row(subject['code'], ip1_marks, ip2_marks, total_marks)
-                output.append(marks_row)
+            theory_total = total_theory_ip1_points + total_theory_ip2_points
+            output.append(f"   Theory Subjects: {theory_total:.2f} points")
+            if total_theory_ip1_points > 0:
+                output.append(f"     ➤ IP-1: {total_theory_ip1_points:.2f}")
+            if total_theory_ip2_points > 0:
+                output.append(f"     ➤ IP-2: {total_theory_ip2_points:.2f}")
         
-        # Add-on courses with marks
-        addon_marks_row = format_table_row("ADD-ON / HONOR / MINOR COURSES", "", "", "")
-        output.append(addon_marks_row)
+        if lab_subjects:
+            lab_total = total_lab_ip1_points + total_lab_ip2_points
+            output.append(f"   Lab Subjects: {lab_total:.2f} points")
+            if total_lab_ip1_points > 0:
+                output.append(f"     ➤ IP-1: {total_lab_ip1_points:.2f}")
+            if total_lab_ip2_points > 0:
+                output.append(f"     ➤ IP-2: {total_lab_ip2_points:.2f}")
+        
+        grand_total_points = (total_theory_ip1_points + total_theory_ip2_points + 
+                             total_lab_ip1_points + total_lab_ip2_points)
+        output.append(f"\n🎯 TOTAL REWARD POINTS: {grand_total_points:.2f}")
+        
+        # Internal Marks Summary
+        output.append("\n📝 INTERNAL MARKS BREAKDOWN:")
+        if theory_subjects:
+            theory_marks_total = total_theory_ip1_marks + total_theory_ip2_marks
+            output.append(f"   Theory Subjects: {theory_marks_total:.2f} marks")
+            if total_theory_ip1_marks > 0:
+                output.append(f"     ➤ IP-1: {total_theory_ip1_marks:.2f}")
+            if total_theory_ip2_marks > 0:
+                output.append(f"     ➤ IP-2: {total_theory_ip2_marks:.2f}")
         
-        # Lab courses with marks
         if lab_subjects:
-            lab_marks_header = format_table_row(f"LAB. COURSES - ({len(lab_subjects)} COURSES)", "", "", "")
-            output.append(lab_marks_header)
-            
-            for subject in lab_subjects:
-                ip1_marks = f"{subject['ip1_marks']:.2f}" if subject['ip1_marks'] > 0 else ""
-                ip2_marks = f"{subject['ip2_marks']:.2f}" if subject['ip2_marks'] > 0 else ""
-                total_marks = f"{subject['total_marks']:.2f}"
-                marks_row = format_table_row(subject['code'], ip1_marks, ip2_marks, total_marks)
-                output.append(marks_row)
-        else:
-            lab_marks_header = format_table_row("LAB. COURSES - (0 COURSES)", "", "", "")
-            output.append(lab_marks_header)
+            lab_marks_total = total_lab_ip1_marks + total_lab_ip2_marks
+            output.append(f"   Lab Subjects: {lab_marks_total:.2f} marks")
+            if total_lab_ip1_marks > 0:
+                output.append(f"     ➤ IP-1: {total_lab_ip1_marks:.2f}")
+            if total_lab_ip2_marks > 0:
+                output.append(f"     ➤ IP-2: {total_lab_ip2_marks:.2f}")
+        
+        grand_total_marks = (total_theory_ip1_marks + total_theory_ip2_marks + 
+                            total_lab_ip1_marks + total_lab_ip2_marks)
+        output.append(f"\n📊 TOTAL INTERNAL MARKS: {grand_total_marks:.2f}")
         
-        # Total marks row
-        marks_ip1_total = f"{grand_total_marks_ip1:.2f}" if grand_total_marks_ip1 > 0 else "0.00"
-        marks_ip2_total = f"{grand_total_marks_ip2:.2f}" if grand_total_marks_ip2 > 0 else "0.00"
-        marks_total_total = f"{grand_total_marks:.2f}"
-        marks_total_row = format_table_row(f"TOTAL - ({total_subjects} SUBJECTS)", marks_ip1_total, marks_ip2_total, marks_total_total)
-        output.append("-" * 88)
-        output.append(marks_total_row)
+        total_subjects = len(theory_subjects) + len(lab_subjects)
+        output.append(f"\n📚 TOTAL SUBJECTS: {total_subjects}")
         
-        output.append("=" * 88)
+        output.append("\n" + "=" * 80)
         
         # Log the search
         now_ist = datetime.now(ist).strftime("%Y-%m-%d %H:%M:%S")
         print(f"IP Details Searched - Roll No: {roll_no} | Student: {student_name} | Time (IST): {now_ist}")
         
         return "\n".join(output)
-        
+    
     except Exception as e:
         error_msg = f"❌ Error extracting subject details: {str(e)}"
         print(error_msg)
@@ -1248,12 +1245,13 @@ with gr.Blocks(
     title="Student Reward Points Check", 
     theme=gr.themes.Soft(),
 ) as app:
-    gr.Markdown("# 🎓 Student Reward Points Checker")
-    gr.Markdown("### Search for Student Details such as Reward Points, Redemption Dates and Innovative Practice (IP) Details")
-    gr.Markdown("### எல்லா புகழும் இறைவனுக்கே ✝ 🕉 ☪")
-    gr.Markdown("## 📝 Use Desktop Site Feature (Desktop Mode) in browser for Good UI and UX")
+    gr.Markdown("## 🎓 Student Reward Points Checker")
+    gr.Markdown("##### Search for Student Details such as Reward Points, Redemption Dates and Innovative Practice (IP) Details")
+    gr.Markdown("##### எல்லா புகழும் இறைவனுக்கே ✝ 🕉 ☪")
+    gr.Markdown("💻 **Mode**: Use Desktop Mode in browser for Good UI and UX")
     gr.Markdown("🕒 **Auto-Updates**: Data automatically refreshes when there is a change in Reward Points Sheet")
-    gr.Markdown("### Fill this form for any Issue/Feedback: [Issue/Feedback Form](https://docs.google.com/forms/d/e/1FAIpQLScnl0udcN2pUDENHl45HIj5HZbvDuwZ0g2eepBbp8tJYg-NvQ/viewform)")    
+    gr.Markdown("📝 **Issue/Feedback Form** : [Issue/Feedback Form](https://docs.google.com/forms/d/e/1FAIpQLScnl0udcN2pUDENHl45HIj5HZbvDuwZ0g2eepBbp8tJYg-NvQ/viewform)")    
+    
     with gr.Tabs():
         with gr.TabItem("🔍 Student Search"):
             with gr.Row():
@@ -1287,14 +1285,17 @@ with gr.Blocks(
 
             subject_output = gr.Textbox(
                 label="Innovative Practice (IP) Details",
-                lines=65,
-                max_lines=75,
+                lines=50,
+                max_lines=60,
                 show_copy_button=True,
                 autoscroll=False
             )
             
         with gr.TabItem("ℹ️ System Information"):
-            system_btn = gr.Button("📊 Get System Information", variant="secondary")
+            with gr.Row():
+                with gr.Column():
+                    system_btn = gr.Button("📊 Get System Information", variant="secondary", size="lg")
+            
             system_output = gr.Textbox(
                 label="System Information",
                 lines=50,
@@ -1305,15 +1306,45 @@ with gr.Blocks(
                 show_label=True
             )
     
-    # Event handlers
-    search_btn.click(fn=search_student, inputs=roll_input, outputs=result_output)
-    roll_input.submit(fn=search_student, inputs=roll_input, outputs=result_output)
+    # Event handlers - Fixed order and proper input/output mapping
     
-    # NEW: Subject details event handlers
-    subject_search_btn.click(fn=extract_subjects_and_marks_for_gradio, inputs=subject_roll_input, outputs=subject_output)
-    subject_roll_input.submit(fn=extract_subjects_and_marks_for_gradio, inputs=subject_roll_input, outputs=subject_output)
+    # 1. Student search handlers
+    search_btn.click(
+        fn=search_student, 
+        inputs=[roll_input], 
+        outputs=[result_output],
+        api_name="search_student"
+    )
     
-    system_btn.click(fn=get_system_info, outputs=system_output)
+    roll_input.submit(
+        fn=search_student, 
+        inputs=[roll_input], 
+        outputs=[result_output],
+        api_name="search_student_submit"
+    )
+    
+    # 2. IP Details handlers - Fixed input parameter
+    subject_search_btn.click(
+        fn=extract_subjects_and_marks_for_gradio, 
+        inputs=[subject_roll_input], 
+        outputs=[subject_output],
+        api_name="get_ip_details"
+    )
+    
+    subject_roll_input.submit(
+        fn=extract_subjects_and_marks_for_gradio, 
+        inputs=[subject_roll_input], 
+        outputs=[subject_output],
+        api_name="get_ip_details_submit"
+    )
+    
+    # 3. System info handler - Fixed to not require inputs
+    system_btn.click(
+        fn=get_system_info, 
+        inputs=[], 
+        outputs=[system_output],
+        api_name="get_system_info"
+    )
     
     # Footer section
     gr.Markdown("---")
@@ -1323,7 +1354,7 @@ with gr.Blocks(
                 """
                 <div style="text-align: center; margin-top: 20px; padding: 20px; background: linear-gradient(135deg, #667eea 0%, #764ba2 100%); border-radius: 10px; color: white;">
                     <h3 style="margin: 0; color: white;">💻 Developed with ❤️ by</h3>
-                    <a href="https://praneshjs.vercel.app/" target="_blank" style="text-decoration: none;">
+                    <a href="https://pranesh-2005.github.io/My-Portfolio/" target="_blank" style="text-decoration: none;">
                         <h2 style="margin: 5px 0; color: #ffd700; cursor: pointer; transition: color 0.3s ease;">PRANESH S</h2>
                     </a>
                     <div style="margin: 15px 0;">
@@ -1346,19 +1377,21 @@ with gr.Blocks(
                 elem_id="footer"
             )
 
-    # System info initialization function
+    # System info initialization function - Fixed to handle errors gracefully
     def initialize_system_info():
-        """Initialize system information display"""
+        """Initialize system information display with error handling"""
         try:
             return get_system_info()
         except Exception as e:
-            print(f"⚠️ Error initializing system info: {str(e)}")
-            return "⚠️ System information will be available after data loads completely."
+            error_msg = f"⚠️ Error initializing system info: {str(e)}"
+            print(error_msg)
+            return "⚠️ System information will be available after data loads completely. Please click 'Get System Information' button to retry."
     
-    # Load system info on startup
+    # Load system info on startup - Fixed to use proper event handling
     app.load(
         fn=initialize_system_info,
-        outputs=system_output
+        inputs=[],
+        outputs=[system_output]
     )
 
 # Launch the app