added IP details sections to view about Innovative Practice Details

app.py

@@ -58,6 +58,244 @@ def authorize():
     
     return gspread.authorize(creds)
 
+# NEW FUNCTION: Extract subjects and marks
+def extract_subjects_and_marks_for_gradio(roll_no):
+    """
+    Extract subjects with their redeemed points and marks for Gradio interface
+    Uses cached studentwise_data instead of making fresh API calls
+    """
+    if not roll_no.strip():
+        return "❌ Please enter a roll number"
+    
+    try:
+        # Get cached data instead of making API calls
+        combined_df, studentwise_data, details_info, reward_points_df = get_cached_data()
+        
+        if not studentwise_data or len(studentwise_data) < 2:
+            return "❌ Studentwise data not available in cache"
+        
+        headers = studentwise_data[0]
+        roll_no = roll_no.strip().upper()
+        
+        # Find the student row using cached data
+        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:
+                    student_row = row
+                    break
+            if student_row:
+                break
+
+        if not student_row:
+            return f"❌ Student with Roll No '{roll_no}' not found."
+
+        def get_value_if_not_empty(col_name):
+            """Helper function to get value only if it's not empty"""
+            if col_name in headers:
+                idx = headers.index(col_name)
+                value = student_row[idx] if idx < len(student_row) else ''
+                return value.strip() if value.strip() else None
+            return None
+
+        def get_numeric_value(col_name):
+            """Helper function to get numeric value, return 0 if empty or invalid"""
+            value = get_value_if_not_empty(col_name)
+            try:
+                return float(value) if value else 0.0
+            except (ValueError, TypeError):
+                return 0.0
+
+        # Get student basic info
+        student_name = get_value_if_not_empty("Student Name") or "Unknown"
+        
+        # Collect theory subjects (optimized loops)
+        theory_subjects = []
+        for i in range(1, 10):
+            subject_code = get_value_if_not_empty(f"TS{i}")
+            if subject_code:
+                theory_subjects.append({
+                    'code': subject_code,
+                    'ip1_points': get_numeric_value(f"IP1TS{i}R"),
+                    'ip2_points': get_numeric_value(f"IP2TS{i}R"),
+                    'ip1_marks': get_numeric_value(f"IP1TS{i}M"),
+                    'ip2_marks': get_numeric_value(f"IP2TS{i}M")
+                })
+
+        # Collect lab subjects (optimized loops)
+        lab_subjects = []
+        for i in range(1, 3):
+            subject_code = get_value_if_not_empty(f"LS{i}")
+            if subject_code:
+                lab_subjects.append({
+                    'code': subject_code,
+                    'ip1_points': get_numeric_value(f"IP1LS{i}R"),
+                    'ip2_points': get_numeric_value(f"IP2LS{i}R"),
+                    'ip1_marks': get_numeric_value(f"IP1LS{i}M"),
+                    'ip2_marks': get_numeric_value(f"IP2LS{i}M")
+                })
+
+        # Calculate totals using list comprehensions (faster)
+        for subject in theory_subjects + lab_subjects:
+            subject['total_points'] = subject['ip1_points'] + subject['ip2_points']
+            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)
+        
+        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}"
+
+        # Build output with clean 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)
+        
+        # Theory courses section
+        if theory_subjects:
+            theory_header = format_table_row(f"THEORY COURSES - ({len(theory_subjects)} COURSES)", "", "", "")
+            output.append(theory_header)
+            
+            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)
+        
+        # Lab courses section
+        if lab_subjects:
+            lab_header = format_table_row(f"LAB. COURSES - ({len(lab_subjects)} COURSES)", "", "", "")
+            output.append(lab_header)
+            
+            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)
+        
+        # Header for marks
+        marks_header = format_table_row("SUBJECTS", "IP - 1", "IP - 2", "TOTAL")
+        output.append(marks_header)
+        output.append("-" * 88)
+        
+        # Theory courses with marks
+        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)
+        
+        # Add-on courses with marks
+        addon_marks_row = format_table_row("ADD-ON / HONOR / MINOR COURSES", "", "", "")
+        output.append(addon_marks_row)
+        
+        # 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)
+        
+        # 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)
+        
+        output.append("=" * 88)
+        
+        # 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)
+        return error_msg
+
+
 # Function to get data from a specific sheet
 def get_sheet_data(spreadsheet, gid, sheet_name):
     try:
@@ -674,6 +912,7 @@ def details_sheet_watcher():
 
         time.sleep(check_interval)
 
+
 def get_detailed_student_points(roll_no, studentwise_data):
     """Get detailed points breakdown from studentwise data"""
     if not studentwise_data or len(studentwise_data) < 3:
@@ -955,10 +1194,10 @@ def get_system_info():
     output.append("=" * 80)
     output.append("SYSTEM INFORMATION")
     output.append("=" * 80)
-    
+
     # Average Points
     if 'average_points' in details_info:
-        output.append("\n🎯 AVERAGE REWARD POINTS BY YEAR:")
+        output.append("\nAVERAGE REWARD POINTS BY YEAR:")
         output.append("-" * 40)
         for year, points in details_info['average_points'].items():
             if points:
@@ -971,21 +1210,21 @@ def get_system_info():
     
     # Redemption Dates
     if 'ip1_redemption' in details_info:
-        output.append("\n📅 IP 1 REDEMPTION DATES:")
+        output.append("\nIP 1 REDEMPTION DATES:")
         output.append("-" * 40)
         for semester, date in details_info['ip1_redemption'].items():
             if date and date != '-':
                 output.append(f"{semester:<10}: {date}")
     
     if 'ip2_redemption' in details_info:
-        output.append("\n📅 IP 2 REDEMPTION DATES:")
+        output.append("\nIP 2 REDEMPTION DATES:")
         output.append("-" * 40)
         for semester, date in details_info['ip2_redemption'].items():
             if date and date != '-':
                 output.append(f"{semester:<10}: {date}")
     
     if 'last_updated' in details_info:
-        output.append(f"\n🕒 LAST UPDATED:")
+        output.append(f"\nLAST UPDATED:")
         output.append("-" * 40)
         output.append(details_info['last_updated'])
     
@@ -1004,19 +1243,17 @@ def get_system_info():
     
     return "\n".join(output)
 
-# Create Gradio interface - SIMPLE LAYOUT WITHOUT ANALYTICS DISPLAY
+# Create Gradio interface
 with gr.Blocks(
     title="Student Reward Points Check", 
     theme=gr.themes.Soft(),
 ) as app:
-    
-    # Simple Header
-    gr.Markdown("# 🎓 Student Reward Points Check")
-    gr.Markdown("### Search for student details including reward points and redemption dates")
+    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("🕒 **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("### Fill this form for any Issue/Feedback: [Issue/Feedback Form](https://docs.google.com/forms/d/e/1FAIpQLScnl0udcN2pUDENHl45HIj5HZbvDuwZ0g2eepBbp8tJYg-NvQ/viewform)")    
     with gr.Tabs():
         with gr.TabItem("🔍 Student Search"):
             with gr.Row():
@@ -1036,6 +1273,25 @@ with gr.Blocks(
                 show_copy_button=True,
                 autoscroll=False
             )
+        
+        with gr.TabItem("📚 Innovative Practice (IP) Details"):
+            with gr.Row():
+                with gr.Column(scale=3):
+                    subject_roll_input = gr.Textbox(
+                        label="Enter Roll Number for Innovative Practice (IP) Details",
+                        placeholder="e.g., 7376222AL181",
+                        value=""
+                    )
+                with gr.Column(scale=1):
+                    subject_search_btn = gr.Button("📚 Get Innovative Practice (IP) Details", variant="primary")
+
+            subject_output = gr.Textbox(
+                label="Innovative Practice (IP) Details",
+                lines=65,
+                max_lines=75,
+                show_copy_button=True,
+                autoscroll=False
+            )
             
         with gr.TabItem("ℹ️ System Information"):
             system_btn = gr.Button("📊 Get System Information", variant="secondary")
@@ -1049,9 +1305,14 @@ with gr.Blocks(
                 show_label=True
             )
     
-    # Event handlers - simplified without analytics updates
+    # 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)
+    
+    # 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)
+    
     system_btn.click(fn=get_system_info, outputs=system_output)
     
     # Footer section
@@ -1062,7 +1323,9 @@ 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>
-                    <h2 style="margin: 5px 0; color: #ffd700;">PRANESH S</h2>
+                    <a href="https://praneshjs.vercel.app/" 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;">
                         <a href="https://github.com/Pranesh-2005" target="_blank" style="color: #ffd700; text-decoration: none; margin: 0 10px; font-size: 16px;">
                             🐱 GitHub
@@ -1082,7 +1345,8 @@ with gr.Blocks(
                 """,
                 elem_id="footer"
             )
-    # FIXED: System info initialization function
+
+    # System info initialization function
     def initialize_system_info():
         """Initialize system information display"""
         try:
@@ -1090,12 +1354,13 @@ with gr.Blocks(
         except Exception as e:
             print(f"⚠️ Error initializing system info: {str(e)}")
             return "⚠️ System information will be available after data loads completely."
-
-    # Load system info on startup (separate from admin mode)
+    
+    # Load system info on startup
     app.load(
         fn=initialize_system_info,
         outputs=system_output
     )
+
 # Launch the app
 if __name__ == "__main__":
     print("🚀 Launching Gradio interface...")