increased max_errors in details_sheet_watcher and removed new feature banner

app.py

@@ -862,62 +862,77 @@ def auto_refresh_worker():
             time.sleep(3600)
 
 def details_sheet_watcher():
-    """Background watcher: checks every 30 seconds (drift-free, IST logs) if 'POINTS LAST UPDATED' changed"""
+    """Background watcher: checks every 30 seconds (drift-free, IST logs) if 'POINTS LAST UPDATED' changed safely"""
     import time
     from datetime import datetime
 
     last_seen_update = None
     consecutive_errors = 0
-    max_errors = 3
+    max_errors = 10
 
     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
-
+    connection_duration = 3600  # 1 hour
     check_interval = 30  # seconds
     next_check = time.time()
 
-    print(f"👀 Starting optimized details sheet watcher (monitoring cell R{TARGET_ROW}C{TARGET_COL} every {check_interval} seconds)...")
+    TARGET_ROW, TARGET_COL = 16, 2
+
+    print(f"👀 Starting optimized details sheet watcher (R{TARGET_ROW}C{TARGET_COL}, every {check_interval}s)...")
+
+    global last_auth_refresh
+    last_auth_refresh = 0
 
     while True:
-        start_time = time.time()
         try:
             if data_cache["is_loading"]:
                 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):
-
-                    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(f"✅ Watcher: Connection refreshed (monitoring cell R{TARGET_ROW}C{TARGET_COL})")
-
-                # Get only the specific cell content
+
+                # Refresh only if 24 hours passed or connection lost
+                should_refresh = (
+                    watcher_client is None
+                    or watcher_sheet is None
+                    or last_connection_time is None
+                    or (current_time - last_connection_time).total_seconds() > connection_duration
+                )
+
+                if should_refresh:
+                    # Enforce 5-min cooldown between authorizations
+                    if time.time() - last_auth_refresh < 300:
+                        print("⏳ Skipping token refresh — recently done.")
+                    else:
+                        print("🔄 Watcher: Refreshing connection...")
+                        try:
+                            watcher_client = authorize()
+                            watcher_spreadsheet = watcher_client.open_by_key(os.getenv("GOOGLE_SHEET_ID"))
+                            watcher_sheet = watcher_spreadsheet.get_worksheet_by_id(847680829)
+                            last_connection_time = current_time
+                            last_auth_refresh = time.time()
+                            print(f"✅ Watcher: Connection refreshed (monitoring cell R{TARGET_ROW}C{TARGET_COL})")
+                        except Exception as e:
+                            print(f"⚠️ Watcher: Failed to refresh connection — {str(e)[:100]}")
+                            time.sleep(60)
+                            continue
+
+                # Skip check if no valid sheet
+                if not watcher_sheet:
+                    time.sleep(check_interval)
+                    continue
+
                 try:
-                    # Use batch_get to get specific cell efficiently
+                    # Fetch only specific cell value
                     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]}...")
+                        print(f"🕒 Watcher started at {now_ist}")
+                        print(f"    Target 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]}...")
@@ -931,34 +946,34 @@ def details_sheet_watcher():
                         else:
                             print(f"⏳ Data already loading, skipping reload ({now_ist})")
                     else:
-                        # 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]}")
+                    print(f"⚠️ Error reading cell R{TARGET_ROW}C{TARGET_COL}: {str(cell_error)[:200]}")
                     consecutive_errors += 1
 
-            if consecutive_errors > 0 and not any([watcher_client is None, watcher_spreadsheet is None, watcher_sheet is None]):
+            # Reset error counter if stable
+            if consecutive_errors > 0:
                 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)[:120]}")
+            print(f"⚠️ Watcher error #{consecutive_errors}: {str(e)[:200]}")
             watcher_client = None
             watcher_spreadsheet = None
             watcher_sheet = None
+
             if consecutive_errors >= max_errors:
                 print("❌ Too many watcher errors, waiting 5 minutes before retry...")
                 time.sleep(300)
                 consecutive_errors = 0
 
-        # Drift-free timing — ensures consistent 30s intervals
+        # Drift-free sleep to maintain 30s sync
         next_check += check_interval
-        sleep_time = max(0, next_check - time.time())
-        time.sleep(sleep_time)
+        time.sleep(max(0, next_check - time.time()))
 
 
 def get_detailed_student_points(roll_no, studentwise_data):
@@ -1371,42 +1386,15 @@ with gr.Blocks(
     title="Student Reward Points Check", 
     theme=gr.themes.Soft(),
 ) as app:
-    gr.HTML(
-        """
-        <style>
-        @keyframes fadeOut {
-            0% { opacity: 1; }
-            90% { opacity: 1; }
-            100% { opacity: 0; display: none; }
-        }
-        
-        .auto-hide-banner {
-            animation: fadeOut 10s forwards;
-        }
-        </style>
-        
-        <div class="auto-hide-banner" style='background-color:#FFF3CD;
-                    padding:12px;
-                    border-radius:10px;
-                    text-align:center;
-                    font-size:16px;
-                    color:#856404;
-                    font-weight:500;
-                    margin-bottom:10px;'>
-            🆕 <b>New Feature Added!</b> Check out the new <b>📚 Innovative Practice (IP) Details</b> tab to view subject-wise IP points and marks instantly!<br>
-            Check out by clicking the 3 dots and select the Innovative Practice (IP) Details Tab and Enter your roll no 🚀
-        </div>
-        """
-    )
     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("""
-    # ***Available Features*** :
-    # **🔍 Student Search Tab**,
-    # **📚 IP Details Tab**,
-    # **ℹ️ System Info Tab**
-    # """)
+    gr.Markdown("""
+    ***Available Features*** :
+    **🔍 Student Search Tab**,
+    **📚 IP Details Tab**,
+    **ℹ️ System Info Tab**
+    """)
     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("📝 **Issue/Feedback Form** : [Issue/Feedback Form](https://docs.google.com/forms/d/e/1FAIpQLScnl0udcN2pUDENHl45HIj5HZbvDuwZ0g2eepBbp8tJYg-NvQ/viewform)")