fixed formating issues and expected 1 but got 0 issue

app.py

@@ -811,103 +811,103 @@ def auto_refresh_worker():
             time.sleep(3600)
 
 def details_sheet_watcher():
-    """Background watcher: checks every 30 seconds if 'POINTS LAST UPDATED' changed - optimized timing"""
+    """Background watcher: checks every 30 seconds (drift-free, IST logs) if 'POINTS LAST UPDATED' changed"""
+    import time
+    from datetime import datetime
+
     last_seen_update = None
     consecutive_errors = 0
     max_errors = 3
 
     watcher_client = None
     watcher_spreadsheet = None
+    watcher_sheet = None
     last_connection_time = None
     connection_duration = 2700  # 45 minutes
 
+    # Specific cell coordinates for "POINTS LAST UPDATED"
+    TARGET_ROW = 16
+    TARGET_COL = 2
+    CELL_RANGE = f"R{TARGET_ROW}C{TARGET_COL}"  # Row 16, Column 2
+
     check_interval = 30  # seconds
+    next_check = time.time()
 
-    print(f"👀 Starting optimized details sheet watcher (checks every {check_interval} seconds)...")
+    print(f"👀 Starting optimized details sheet watcher (monitoring cell R{TARGET_ROW}C{TARGET_COL} every {check_interval} seconds)...")
 
     while True:
+        start_time = time.time()
         try:
             if data_cache["is_loading"]:
-                print("⏳ Watcher: Skipping check - data loading in progress")
-                time.sleep(check_interval)
-                continue
-
-            current_time = datetime.now()
-            if (watcher_client is None or 
-                watcher_spreadsheet is None or
-                last_connection_time is None or 
-                (current_time - last_connection_time).total_seconds() > connection_duration):
+                print("⏳ Watcher: Skipping check – data loading in progress")
+            else:
+                current_time = datetime.now()
+                if (watcher_client is None or 
+                    watcher_spreadsheet is None or
+                    watcher_sheet is None or
+                    last_connection_time is None or 
+                    (current_time - last_connection_time).total_seconds() > connection_duration):
 
-                try:
                     print("🔄 Watcher: Refreshing connection...")
                     watcher_client = authorize()
                     watcher_spreadsheet = watcher_client.open_by_key(os.getenv('GOOGLE_SHEET_ID'))
+                    watcher_sheet = watcher_spreadsheet.get_worksheet_by_id(847680829)  # Details sheet GID
                     last_connection_time = current_time
-                    print("✅ Watcher: Connection refreshed")
-                except Exception as auth_error:
-                    print(f"⚠️ Watcher connection error: {str(auth_error)[:100]}...")
-                    consecutive_errors += 1
-                    watcher_client = None
-                    watcher_spreadsheet = None
-                    time.sleep(check_interval)
-                    continue
+                    print(f"✅ Watcher: Connection refreshed (monitoring cell R{TARGET_ROW}C{TARGET_COL})")
 
-            try:
-                details_info = get_details_info(watcher_spreadsheet)
-            except Exception as sheet_error:
-                print(f"⚠️ Watcher sheet error: {str(sheet_error)[:100]}...")
-                consecutive_errors += 1
-                watcher_client = None
-                watcher_spreadsheet = None
-                time.sleep(check_interval)
-                continue
-
-            if details_info and 'last_updated' in details_info:
-                current_update = details_info['last_updated'].strip()
-
-                if last_seen_update is None:
-                    last_seen_update = current_update
-                    print(f"🕒 Watcher: Monitoring established")
-                    print(f"    Current: {current_update[:80]}...")
-                elif current_update != last_seen_update:
-                    print(f"🔄 CHANGE DETECTED!")
-                    print(f"    Old: {last_seen_update[:60]}...")
-                    print(f"    New: {current_update[:60]}...")
-                    last_seen_update = current_update
-
-                    if not data_cache["is_loading"]:
-                        print("🔄 Reloading data directly...")
-                        try:
+                # Get only the specific cell content
+                try:
+                    # Use batch_get to get specific cell efficiently
+                    cell_value = watcher_sheet.cell(TARGET_ROW, TARGET_COL).value
+                    current_update = str(cell_value).strip() if cell_value else ""
+                    
+                    now_ist = datetime.now(ist).strftime("%Y-%m-%d %H:%M:%S")
+
+                    if last_seen_update is None:
+                        last_seen_update = current_update
+                        print(f"🕒 Watcher: Monitoring established ({now_ist})")
+                        print(f"    Target cell R{TARGET_ROW}C{TARGET_COL}: {current_update[:80]}...")
+                    elif current_update != last_seen_update:
+                        print(f"🔄 CHANGE DETECTED! ({now_ist})")
+                        print(f"    Old: {last_seen_update[:60]}...")
+                        print(f"    New: {current_update[:60]}...")
+                        last_seen_update = current_update
+
+                        if not data_cache["is_loading"]:
+                            print("🔄 Reloading data directly...")
                             load_all_data()
-                            print("✅ Data reload completed successfully")
-                        except Exception as reload_error:
-                            print(f"❌ Reload error: {str(reload_error)[:100]}...")
+                            print(f"✅ Data reload completed successfully ({now_ist})")
+                        else:
+                            print(f"⏳ Data already loading, skipping reload ({now_ist})")
                     else:
-                        print("⏳ Data already loading, skipping reload")
-                else:
-                    # Show heartbeat every 5 minutes instead of 10
-                    if datetime.now().minute % 5 == 0 and datetime.now(ist).second < check_interval:
-                        print(f"✅ Watcher: No changes detected ({datetime.now(ist).strftime('%H:%M')})")
-            else:
-                print("⚠️ Watcher: Could not extract last_updated info")
-                consecutive_errors += 1
+                        # Log heartbeat every 5 minutes in IST
+                        current_ist = datetime.now(ist)
+                        if current_ist.minute % 5 == 0 and current_ist.second < check_interval:
+                            print(f"✅ Watcher: No changes detected in R{TARGET_ROW}C{TARGET_COL} ({current_ist.strftime('%H:%M:%S')})")
+                            
+                except Exception as cell_error:
+                    print(f"⚠️ Error reading cell R{TARGET_ROW}C{TARGET_COL}: {str(cell_error)[:100]}")
+                    consecutive_errors += 1
 
-            if details_info and consecutive_errors > 0:
+            if consecutive_errors > 0 and not any([watcher_client is None, watcher_spreadsheet is None, watcher_sheet is None]):
                 print(f"✅ Watcher: Connection restored (cleared {consecutive_errors} errors)")
                 consecutive_errors = 0
 
         except Exception as e:
             consecutive_errors += 1
-            print(f"⚠️ Watcher error #{consecutive_errors}: {str(e)[:100]}...")
+            print(f"⚠️ Watcher error #{consecutive_errors}: {str(e)[:120]}")
             watcher_client = None
             watcher_spreadsheet = None
+            watcher_sheet = None
             if consecutive_errors >= max_errors:
-                error_wait = 300  # 5 minutes
-                print(f"❌ Too many watcher errors, waiting {error_wait//60} minutes...")
-                time.sleep(error_wait)
+                print("❌ Too many watcher errors, waiting 5 minutes before retry...")
+                time.sleep(300)
                 consecutive_errors = 0
 
-        time.sleep(check_interval)
+        # Drift-free timing — ensures consistent 30s intervals
+        next_check += check_interval
+        sleep_time = max(0, next_check - time.time())
+        time.sleep(sleep_time)
 
 
 def get_detailed_student_points(roll_no, studentwise_data):
@@ -1306,44 +1306,39 @@ with gr.Blocks(
                 show_label=True
             )
     
-    # Event handlers - Fixed order and proper input/output mapping
+    # Event handlers - FIXED: Proper input/output mapping
     
     # 1. Student search handlers
     search_btn.click(
         fn=search_student, 
-        inputs=[roll_input], 
-        outputs=[result_output],
-        api_name="search_student"
+        inputs=roll_input,  # Changed from [roll_input] to roll_input
+        outputs=result_output
     )
     
     roll_input.submit(
         fn=search_student, 
-        inputs=[roll_input], 
-        outputs=[result_output],
-        api_name="search_student_submit"
+        inputs=roll_input,  # Changed from [roll_input] to roll_input
+        outputs=result_output
     )
     
-    # 2. IP Details handlers - Fixed input parameter
+    # 2. IP Details handlers - FIXED: Proper 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"
+        inputs=subject_roll_input,  # Changed from [subject_roll_input] to subject_roll_input
+        outputs=subject_output
     )
     
     subject_roll_input.submit(
         fn=extract_subjects_and_marks_for_gradio, 
-        inputs=[subject_roll_input], 
-        outputs=[subject_output],
-        api_name="get_ip_details_submit"
+        inputs=subject_roll_input,  # Changed from [subject_roll_input] to subject_roll_input
+        outputs=subject_output
     )
     
-    # 3. System info handler - Fixed to not require inputs
+    # 3. System info handler - Correct (no inputs needed)
     system_btn.click(
         fn=get_system_info, 
         inputs=[], 
-        outputs=[system_output],
-        api_name="get_system_info"
+        outputs=system_output
     )
     
     # Footer section
@@ -1354,7 +1349,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://pranesh-2005.github.io/My-Portfolio/" target="_blank" style="text-decoration: none;">
+                    <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;">
@@ -1387,11 +1382,11 @@ with gr.Blocks(
             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 - Fixed to use proper event handling
+    # Load system info on startup
     app.load(
         fn=initialize_system_info,
         inputs=[],
-        outputs=[system_output]
+        outputs=system_output
     )
 
 # Launch the app