Update app.py

app.py

@@ -596,120 +596,107 @@ def generate_timetable(course_df, room_df, output_dir, room_slots, timetable):
 
 # --- Final Working Process Function ---
 def process(course_file, room_file):
+    import traceback
     import shutil
     import zipfile
     from collections import defaultdict
     import pandas as pd
     import os
 
-    # --- Read Excel Files ---
-    course_df = pd.read_excel(course_file.name)
-    room_df = pd.read_excel(room_file.name)
+    try:
+        print("✅ Starting process...")
 
-    # ✅ Rename columns for clarity
-    room_df.columns = [
-        "Class Room", "Class Capacity",
-        "Computer Lab", "Computer Capacity",
-        "Seminar Halls", "Seminar Capacity"
-    ]
+        # --- Read Excel Files ---
+        print("📄 Reading course and room files...")
+        course_df = pd.read_excel(course_file.name)
+        room_df = pd.read_excel(room_file.name)
 
-    # --- Create output directories ---
-    output_dir = "generated_routines"
-    output_1st_year_dir = "generated_1st_year_LT"
+        # ✅ Rename columns for clarity
+        room_df.columns = [
+            "Class Room", "Class Capacity",
+            "Computer Lab", "Computer Capacity",
+            "Seminar Halls", "Seminar Capacity"
+        ]
 
-    for path in [output_dir, output_1st_year_dir]:
-        if os.path.exists(path):
-            shutil.rmtree(path)
-        os.makedirs(path)
+        # --- Create output directories ---
+        output_dir = "generated_routines"
+        output_1st_year_dir = "generated_1st_year_LT"
 
-    # --- Initialize room availability map ---
-    room_slots = defaultdict(lambda: defaultdict(lambda: True))
+        for path in [output_dir, output_1st_year_dir]:
+            if os.path.exists(path):
+                shutil.rmtree(path)
+            os.makedirs(path)
 
-    # ✅ Block 2–5 PM slots by default for all rooms (for 1st year labs)
-    for day in days:
-        for room in room_slots:
-            for slot in ["2-3", "3-4", "4-5"]:
-                room_slots[room][f"{day}-{slot}"] = False
+        # --- Initialize room availability map ---
+        room_slots = defaultdict(lambda: defaultdict(lambda: True))
+
+        for day in days:
+            for room in room_slots:
+                for slot in ["2-3", "3-4", "4-5"]:
+                    room_slots[room][f"{day}-{slot}"] = False
+
+        # --- Initialize Central Timetable ---
+        timetable = defaultdict(lambda: defaultdict(lambda: defaultdict(list)))
+
+        # ✅ Step 1: Generate lectures for 2nd–5th year
+        print("📚 Generating timetable for 2nd–5th years...")
+        first_year_lectures = generate_timetable(course_df, room_df, output_dir, room_slots, timetable)
+
+        # ✅ Step 2: Schedule BS-MS 1st year labs and tutorials
+        print("👨‍🔬 Scheduling 1st year labs/tutorials...")
+        tutorial_groups, lab_groups, unscheduled_1st = schedule_bs_ms_first_year(
+            course_df,
+            room_df,
+            room_slots,
+            time_slots,
+            days,
+            output_1st_year_dir,
+            first_year_lectures_by_day_slot=first_year_lectures,
+            timetable=timetable
+        )
+
+        # ✅ Step 3: Reports
+        print("📊 Generating reports...")
+        generate_available_halls_report(room_slots)
+        generate_central_timetable_excel(timetable)
+        unused_hall_path = "Unused_Halls_Report.xlsx"
+        generate_unused_halls_report(room_slots, output_path=unused_hall_path)
+
+        # ✅ Step 4: Zipping folders
+        print("🗂️ Zipping timetables...")
+        zip_path_main = "Final_Timetable.zip"
+        with zipfile.ZipFile(zip_path_main, 'w') as zipf:
+            for foldername, _, filenames in os.walk(output_dir):
+                for filename in filenames:
+                    filepath = os.path.join(foldername, filename)
+                    arcname = os.path.relpath(filepath, output_dir)
+                    zipf.write(filepath, arcname)
+
+        zip_path_1st = "1st_Year_Labs_Tutorials.zip"
+        with zipfile.ZipFile(zip_path_1st, 'w') as zipf:
+            for foldername, _, filenames in os.walk(output_1st_year_dir):
+                for filename in filenames:
+                    filepath = os.path.join(foldername, filename)
+                    arcname = os.path.relpath(filepath, output_1st_year_dir)
+                    zipf.write(filepath, arcname)
+
+        print("✅ All done successfully.")
+
+        return (
+            zip_path_main,
+            zip_path_1st,
+            "Unscheduled_LTP_Report.xlsx" if os.path.exists("Unscheduled_LTP_Report.xlsx") else None,
+            "Available_Halls.xlsx" if os.path.exists("Available_Halls.xlsx") else None,
+            "Central_Timetable.xlsx" if os.path.exists("Central_Timetable.xlsx") else None,
+            unused_hall_path if os.path.exists(unused_hall_path) else None,
+            ""
+        )
+
+    except Exception as e:
+        err_msg = f"🚨 Error: {str(e)}\n\n" + traceback.format_exc()
+        print(err_msg)
+        return None, None, None, None, None, None, err_msg
 
-    # ✅ Build master room data
-    room_data = []
-    for i in range(len(room_df)):
-        # Class Room
-        if pd.notna(room_df["Class Room"].iloc[i]) and pd.notna(room_df["Class Capacity"].iloc[i]):
-            room_data.append((
-                str(room_df["Class Room"].iloc[i]).strip(),
-                int(room_df["Class Capacity"].iloc[i]),
-                "Class Room"
-            ))
-
-        # Computer Lab
-        if pd.notna(room_df["Computer Lab"].iloc[i]) and pd.notna(room_df["Computer Capacity"].iloc[i]):
-            room_data.append((
-                str(room_df["Computer Lab"].iloc[i]).strip(),
-                int(room_df["Computer Capacity"].iloc[i]),
-                "Computer Lab"
-            ))
-
-        # Seminar Halls
-        if pd.notna(room_df["Seminar Halls"].iloc[i]) and pd.notna(room_df["Seminar Capacity"].iloc[i]):
-            room_data.append((
-                str(room_df["Seminar Halls"].iloc[i]).strip(),
-                int(room_df["Seminar Capacity"].iloc[i]),
-                "Seminar Halls"
-            ))
-
-    # --- Initialize Central Timetable ---
-    timetable = defaultdict(lambda: defaultdict(lambda: defaultdict(list)))
-
-    # Step 1: Generate lectures for 2nd–5th year
-    first_year_lectures = generate_timetable(course_df, room_df, output_dir, room_slots, timetable)
-
-    # Step 2: Schedule BS-MS 1st year labs and tutorials
-    tutorial_groups, lab_groups, unscheduled_1st = schedule_bs_ms_first_year(
-        course_df,
-        room_df,
-        room_slots,
-        time_slots,
-        days,
-        output_1st_year_dir,
-        first_year_lectures_by_day_slot=first_year_lectures,
-        timetable=timetable
-    )
-
-    # Step 3: Generate all reports
-    generate_available_halls_report(room_slots)
-    generate_central_timetable_excel(timetable)
-
-    # ✅ NEW: Generate unused halls report
-    unused_hall_path = "Unused_Halls_Report.xlsx"
-    generate_unused_halls_report(room_slots, output_path=unused_hall_path)
-
-    # Step 4: Zip Timetables for download
-    zip_path_main = "Final_Timetable.zip"
-    with zipfile.ZipFile(zip_path_main, 'w') as zipf:
-        for foldername, _, filenames in os.walk(output_dir):
-            for filename in filenames:
-                filepath = os.path.join(foldername, filename)
-                arcname = os.path.relpath(filepath, output_dir)
-                zipf.write(filepath, arcname)
-
-    zip_path_1st = "1st_Year_Labs_Tutorials.zip"
-    with zipfile.ZipFile(zip_path_1st, 'w') as zipf:
-        for foldername, _, filenames in os.walk(output_1st_year_dir):
-            for filename in filenames:
-                filepath = os.path.join(foldername, filename)
-                arcname = os.path.relpath(filepath, output_1st_year_dir)
-                zipf.write(filepath, arcname)
-
-    # Step 5: Return all generated files
-    return (
-        zip_path_main,
-        zip_path_1st,
-        ("Unscheduled_LTP_Report.xlsx" if os.path.exists("Unscheduled_LTP_Report.xlsx") else None),
-        ("Available_Halls.xlsx" if os.path.exists("Available_Halls.xlsx") else None),
-        ("Central_Timetable.xlsx" if os.path.exists("Central_Timetable.xlsx") else None),
-        (unused_hall_path if os.path.exists(unused_hall_path) else None)
-    )