Update app.py

app.py

@@ -1,4 +1,4 @@
-# app.py - Enhanced PDF & Video Generator (HD Video with Names Below Images)
+# app.py - PDF Generator ONLY (No MoviePy - HF Spaces Compatible)
 
 import gradio as gr
 from fpdf import FPDF
@@ -6,237 +6,273 @@ import os
 import tempfile
 import re
 from datetime import datetime
-from moviepy.editor import ImageClip, concatenate_videoclips, TextClip, CompositeVideoClip
-from PIL import Image  # For image processing if needed
+from pathlib import Path
+import io
 
-class SimplePDF(FPDF):
+class ProfessionalPDF(FPDF):
+    def header(self):
+        self.set_font('Arial', 'B', 12)
+        self.cell(0, 10, 'Professional Document', 0, 1, 'C')
+    
     def footer(self):
         self.set_y(-15)
         self.set_font('Arial', 'I', 8)
-        self.cell(0, 10, f'Page {self.page_no()}', 0, 0, 'C')
+        self.cell(0, 10, f'Page {self.page_no()} | Generated {datetime.now().strftime("%Y-%m-%d")}', 0, 0, 'C')
 
-def clean_text(text, max_length=100):
-    """Clean text for PDF and video compatibility"""
+def clean_text_for_pdf(text, max_length=100):
+    """Clean text to avoid PDF encoding issues"""
     if not text:
-        return ""
+        return "Untitled"
+    
+    # Replace problematic characters
     replacements = {
         '–': '-', '—': '-', '“': '"', '”': '"', '‘': "'", '’': "'",
-        '•': '-', '°': '°', '™': 'TM', '®': 'R', '©': 'C',
-        '€': 'EUR', '£': 'GBP', '¥': 'JPY'
+        '•': '*', '°': 'deg', '™': 'TM', '®': 'R', '©': 'C',
+        '€': 'EUR', '£': 'GBP', '¥': 'JPY', '\n': ' '
     }
+    
     for old, new in replacements.items():
-        text = text.replace(old, new)
-    text = re.sub(r'[^\w\s\-.,!?\(\)\[\]:;0-9°]', '', text)
-    text = ' '.join(text.split())[:max_length]
-    return text.strip()
+        text = str(text).replace(old, new)
+    
+    # Remove remaining special chars
+    text = re.sub(r'[^\w\s\-.,!?\(\)\[\]:;0-9]', '', text)
+    text = re.sub(r'\s+', ' ', text)  # Normalize whitespace
+    return text.strip()[:max_length]
 
-def generate_outputs(title, plan, names, image_files, duration_per_slide):
-    # Validation
+def generate_pdf(title, plan, names, image_files):
+    # Fast validation
     if not title or not title.strip():
-        return None, None, "❌ Title required!"
+        return None, "❌ Title is required!"
     
     names_list = [n.strip() for n in names.split(',') if n.strip()]
     
     if not names_list:
-        return None, None, "❌ At least one name needed!"
+        return None, "❌ Enter at least one name!"
+    
+    if len(names_list) > 30:
+        return None, "❌ Maximum 30 names allowed!"
     
     if not image_files:
-        return None, None, "❌ Images required!"
+        return None, "❌ Upload images (matching names count)!"
     
     if len(names_list) != len(image_files):
-        return None, None, f"❌ {len(names_list)} names vs {len(image_files)} images"
-    
-    # Limit to 30 items max
-    if len(names_list) > 30:
-        return None, None, "❌ Max 30 items allowed!"
+        return None, f"❌ Mismatch: {len(names_list)} names, {len(image_files)} images"
     
     try:
-        # PDF Generation
-        pdf = SimplePDF()
-        pdf.set_margins(15, 15, 15)
+        pdf = ProfessionalPDF()
+        pdf.set_margins(20, 20, 20)
+        pdf.set_auto_page_break(auto=True, margin=20)
         
-        clean_title = clean_text(title)
-        clean_plan = clean_text(plan, 400)
+        # Clean inputs
+        clean_title = clean_text_for_pdf(title, 80)
+        clean_plan = clean_text_for_pdf(plan, 800)
         
+        # COVER PAGE
         pdf.add_page()
-        pdf.set_font('Arial', 'B', 20)
-        pdf.cell(0, 15, clean_title, ln=1, align='C')
+        pdf.set_font('Arial', 'B', 28)
+        pdf.set_text_color(0, 51, 102)  # Dark blue
+        pdf.cell(0, 20, clean_title, ln=1, align='C')
         
-        if clean_plan:
-            pdf.ln(10)
-            pdf.set_font('Arial', '', 11)
-            pdf.multi_cell(0, 7, clean_plan)
+        pdf.ln(15)
+        pdf.set_font('Arial', '', 14)
+        pdf.set_text_color(0, 0, 0)
+        pdf.multi_cell(0, 10, clean_plan)
         
-        pdf.ln(10)
-        pdf.set_font('Arial', 'I', 9)
-        pdf.cell(0, 8, f"Generated: {datetime.now().strftime('%Y-%m-%d %H:%M')}", ln=1, align='C')
-        pdf.cell(0, 8, f"Participants: {len(names_list)}", ln=1, align='C')
+        pdf.ln(20)
+        pdf.set_font('Arial', 'B', 12)
+        pdf.cell(0, 10, f"TOTAL PARTICIPANTS: {len(names_list)}", ln=1, align='C')
+        pdf.cell(0, 10, f"GENERATED: {datetime.now().strftime('%Y-%m-%d %H:%M')}", ln=1, align='C')
         
+        # INDIVIDUAL PAGES (Image UP, Name DOWN)
         for i, (name, img_path) in enumerate(zip(names_list, image_files)):
             pdf.add_page()
-            clean_name = clean_text(name, 40)
-            pdf.set_font('Arial', 'B', 16)
-            pdf.cell(0, 12, clean_name.upper(), ln=1, align='C')
             
-            pdf.set_font('Arial', '', 11)
-            pdf.cell(0, 8, f"ID: {i+1}/{len(names_list)}", ln=1, align='C')
+            # Clean name
+            clean_name = clean_text_for_pdf(name, 50)
             
-            image_added = False
-            if os.path.exists(img_path):
-                try:
-                    img_width = 100
-                    x_pos = (pdf.w - img_width) / 2
-                    pdf.image(img_path, x=x_pos, y=40, w=img_width)
-                    image_added = True
-                except:
-                    pass
-            
-            if not image_added:
-                pdf.set_font('Arial', '', 12)
-                pdf.cell(0, 10, "[IMAGE PLACEHOLDER]", ln=1, align='C')
-        
-        pdf_path = f"pdf_{int(datetime.now().timestamp())}.pdf"
-        pdf.output(pdf_path)
-        
-        # Video Generation - Full HD 1920x1080
-        clips = []
-        hd_width, hd_height = 1920, 1080
-        text_height = 150  # Space for name below image
-        
-        for i, (name, img_path) in enumerate(zip(names_list, image_files)):
-            clean_name = clean_text(name, 40)
+            # HEADER WITH ID
+            pdf.set_font('Arial', 'B', 18)
+            pdf.set_text_color(51, 51, 153)  # Blue
+            pdf.cell(0, 15, f"PARTICIPANT #{i+1}", ln=1, align='C')
             
-            # Load image
-            if os.path.exists(img_path):
-                try:
-                    # Create image clip
-                    img_clip = ImageClip(img_path).resize(height=hd_height - text_height - 50)  # Resize to fit above text
-                    img_clip = img_clip.set_position(("center", 25))  # Position at top with margin
+            # IMAGE SECTION (UPPER PART)
+            pdf.ln(10)
+            try:
+                if os.path.exists(img_path):
+                    # Calculate centered position
+                    img_width = 140
+                    page_width = pdf.w - 40  # Account for margins
+                    x_pos = (page_width - img_width) / 2 + 20
                     
-                    # Text clip for name below
-                    text_clip = TextClip(
-                        f"{clean_name.upper()}\nID: {i+1}",
-                        fontsize=80, color='white', font='Arial-Bold',
-                        size=(hd_width, text_height), bg_color='black', method='caption'
-                    ).set_position(("center", hd_height - text_height - 25))
+                    pdf.image(img_path, x=x_pos, y=pdf.get_y(), w=img_width, h=180)
                     
-                    # Composite
-                    composite = CompositeVideoClip([img_clip, text_clip], size=(hd_width, hd_height))
-                    composite = composite.set_duration(duration_per_slide)
-                    clips.append(composite)
-                except Exception as e:
-                    # Fallback placeholder clip
-                    text_clip = TextClip(
-                        f"{clean_name.upper()}\nID: {i+1}\n[Image Not Available]",
-                        fontsize=60, color='white', bg_color='black'
-                    ).set_duration(duration_per_slide)
-                    clips.append(text_clip)
-            else:
-                # Placeholder if no image
-                text_clip = TextClip(
-                    f"{clean_name.upper()}\nID: {i+1}\n[No Image]",
-                    fontsize=60, color='white', bg_color='black'
-                ).set_duration(duration_per_slide)
-                clips.append(text_clip)
-        
-        # Concatenate all clips into video
-        video = concatenate_videoclips(clips, method="compose")
+                    # Image border
+                    pdf.set_draw_color(200, 200, 200)
+                    pdf.line(x_pos, pdf.get_y(), x_pos + img_width, pdf.get_y())
+                    pdf.line(x_pos + img_width, pdf.get_y(), x_pos + img_width, pdf.get_y() + 180)
+                    pdf.line(x_pos + img_width, pdf.get_y() + 180, x_pos, pdf.get_y() + 180)
+                    pdf.line(x_pos, pdf.get_y() + 180, x_pos, pdf.get_y())
+                    
+                else:
+                    # Placeholder rectangle
+                    img_width = 140
+                    x_pos = (pdf.w - img_width) / 2
+                    pdf.set_fill_color(240, 240, 240)
+                    pdf.rect(x_pos, pdf.get_y(), img_width, 180, 'F')
+                    pdf.set_font('Arial', '', 12)
+                    pdf.set_xy(x_pos + 10, pdf.get_y() + 80)
+                    pdf.multi_cell(img_width - 20, 10, "IMAGE\nNOT\nFOUND", align='C')
+                    
+            except Exception:
+                # Fallback placeholder
+                img_width = 140
+                x_pos = (pdf.w - img_width) / 2
+                pdf.set_fill_color(240, 240, 240)
+                pdf.rect(x_pos, pdf.get_y(), img_width, 180, 'F')
+                pdf.set_font('Arial', '', 12)
+                pdf.set_xy(x_pos + 10, pdf.get_y() + 80)
+                pdf.multi_cell(img_width - 20, 10, "IMAGE\nERROR", align='C')
+            
+            # NAME SECTION (LOWER PART - BELOW IMAGE)
+            pdf.ln(200)  # Move below image area
+            pdf.set_font('Arial', 'B', 22)
+            pdf.set_text_color(0, 0, 0)
+            pdf.cell(0, 20, clean_name.upper(), ln=1, align='C')
+            
+            pdf.set_font('Arial', '', 12)
+            pdf.set_text_color(100, 100, 100)
+            pdf.cell(0, 10, f"Participant ID: {i+1} of {len(names_list)}", ln=1, align='C')
+            pdf.cell(0, 10, f"Generated: {datetime.now().strftime('%d/%m/%Y')}", ln=1, align='C')
         
-        # Add title intro slide if title
-        if clean_title:
-            title_clip = TextClip(
-                clean_title.upper(),
-                fontsize=100, color='white', bg_color='blue',
-                size=(hd_width, hd_height)
-            ).set_duration(3)  # 3 sec intro
-            video = concatenate_videoclips([title_clip, video])
+        # Output to file directly
+        timestamp = int(datetime.now().timestamp())
+        pdf_path = f"professional_pdf_{timestamp}.pdf"
+        pdf.output(pdf_path)
         
-        video_path = f"video_{int(datetime.now().timestamp())}.mp4"
-        video.write_videofile(video_path, fps=24, codec='libx264', audio=False)  # HD silent video
+        success_msg = f"""
+        ✅ **PDF Generated Successfully!**
+        📄 **Title**: {clean_title[:50]}...
+        👥 **{len(names_list)} Participants** (IDs 1-{len(names_list)})
+        📑 **{len(names_list) + 1} Pages** (Cover + Participants)
+        🖼️ **Images**: UP | **Names**: DOWN layout
+        💾 Ready for download!
+        """
         
-        success_msg = f"✅ Generated!\n📄 PDF & 🎥 HD Video (1920x1080)\n👥 {len(names_list)} items (IDs 1 to {len(names_list)})"
-        return pdf_path, video_path, success_msg
+        return pdf_path, success_msg
         
     except Exception as e:
-        return None, None, f"❌ Error: {str(e)}"
+        return None, f"❌ Error: {str(e)}\nPlease check your inputs and try again."
 
-# Enhanced UI/UX with Better Layout & Instructions
+# Professional UI/UX with Full HD View Support
 with gr.Blocks(
-    theme=gr.themes.Soft(primary_hue="blue", secondary_hue="gray", neutral_hue="slate"),
+    title="Professional PDF Generator",
+    theme=gr.themes.Soft(
+        primary_hue="blue",
+        secondary_hue="gray",
+        neutral_hue="stone"
+    ),
     css="""
-    .gradio-container {max-width: 1200px; margin: auto; padding: 20px;}
-    .input-label {font-weight: bold; font-size: 1.2em;}
-    .output-section {margin-top: 20px; padding: 15px; background: #f0f4f8; border-radius: 10px;}
-    .status {font-size: 1.1em; color: #333;}
+    .gradio-container {max-width: 1400px !important; margin: auto;}
+    .header {background: linear-gradient(135deg, #667eea 0%, #764ba2 100%); color: white; padding: 30px; border-radius: 15px; text-align: center;}
+    .input-section {background: #f8fafc; padding: 20px; border-radius: 10px; margin: 10px 0; border-left: 4px solid #3b82f6;}
+    .status-success {background: #dcfce7; color: #166534; padding: 15px; border-radius: 8px; border-left: 4px solid #22c55e;}
+    .status-error {background: #fef2f2; color: #dc2626; padding: 15px; border-radius: 8px; border-left: 4px solid #ef4444;}
+    .btn-primary {background: linear-gradient(45deg, #3b82f6, #1d4ed8) !important;}
+    .output-zone {background: #f1f5f9; padding: 20px; border-radius: 10px; margin-top: 20px;}
     """
 ) as demo:
-    gr.Markdown(
-        """
-        # 🚀 HD PDF & Video Generator
-        **Create Professional PDF & Full HD Video Slideshow**
-        
-        ### Instructions:
-        - **Title**: Document/Video title (required)
-        - **Plan**: Optional description
-        - **Names**: Comma-separated (e.g., "John Doe, Jane Smith") - Max 30
-        - **Images**: Upload matching images (uploaded below names input for UX)
-        - **Slide Duration**: Seconds per slide in video
-        - Video: Full HD 1920x1080, name below image, IDs 1 to N
-        - Click Generate for PDF & Video downloads
-        """
-    )
     
-    with gr.Row(equal_height=True):
+    # Header
+    with gr.Row():
+        gr.Markdown(
+            """
+            <div class="header">
+                <h1>📄 Professional PDF Generator</h1>
+                <p><strong>Image UP • Name DOWN • HD Quality • 1-30 Participants</strong></p>
+            </div>
+            """,
+            elem_classes="header"
+        )
+    
+    gr.Markdown("""
+    ### 🚀 How to Use:
+    1. **Title**: Enter document title
+    2. **Names**: Comma-separated (max 30): "John Doe, Jane Smith"
+    3. **Images**: Upload matching images
+    4. **Layout**: Each page = Large image (UP) + Name/ID (DOWN)
+    5. **Output**: Professional PDF with cover page
+    """)
+    
+    # Inputs
+    with gr.Row():
         with gr.Column(scale=2):
             title_input = gr.Textbox(
-                label="📛 Title (Required)",
-                placeholder="Enter title here...",
-                lines=1,
-                elem_classes="input-label"
-            )
-            plan_input = gr.Textbox(
-                label="📋 Plan/Description (Optional)",
-                placeholder="Enter details here...",
-                lines=4
+                label="📛 Document Title *",
+                placeholder="Enter your document title...",
+                lines=2,
+                elem_classes="input-section"
             )
+            
             names_input = gr.Textbox(
-                label="👥 Names (Comma-Separated, Required)",
-                placeholder="Name1, Name2, Name3 (up to 30)",
-                lines=3
+                label="👥 Names (Comma Separated) *",
+                placeholder="John Doe, Jane Smith, Bob Wilson (Max 30)",
+                lines=4,
+                elem_classes="input-section"
             )
-            images_input = gr.File(
-                label="🖼️ Upload Images (Match Names Count)",
-                file_count="multiple",
-                file_types=["image"]
+            
+            plan_input = gr.Textbox(
+                label="📝 Description/Plan (Optional)",
+                placeholder="Add description, plan, or notes for cover page...",
+                lines=3,
+                elem_classes="input-section"
             )
         
         with gr.Column(scale=1):
-            duration_input = gr.Slider(
-                minimum=1,
-                maximum=30,
-                step=1,
-                value=5,
-                label="⏱️ Duration per Slide (Seconds)"
+            images_input = gr.File(
+                label="🖼️ Upload Images *",
+                file_count="multiple",
+                file_types=[".jpg", ".jpeg", ".png", ".webp"],
+                elem_classes="input-section"
             )
-            generate_btn = gr.Button("🚀 Generate PDF & Video", variant="primary", size="lg")
-            clear_btn = gr.Button("🗑️ Clear All", variant="secondary")
     
-    with gr.Row(elem_classes="output-section"):
-        pdf_output = gr.File(label="📄 Download PDF")
-        video_output = gr.File(label="🎥 Download HD Video (MP4)")
+    # Controls
+    with gr.Row():
+        generate_btn = gr.Button("🚀 Generate Professional PDF", variant="primary", size="lg")
+        clear_btn = gr.Button("🗑️ Clear All", variant="secondary")
     
-    status_output = gr.Markdown("Ready to generate...", elem_classes="status")
+    # Outputs
+    with gr.Row(elem_classes="output-zone"):
+        pdf_output = gr.File(label="📥 Download Professional PDF")
+    
+    status_output = gr.Markdown("Ready to generate professional PDF...", elem_classes="status-success")
+    
+    # Events
+    def clear_all():
+        return "", "", "", None, "Form cleared! Ready to create new PDF."
     
-    # Event handlers
     generate_btn.click(
-        fn=generate_outputs,
-        inputs=[title_input, plan_input, names_input, images_input, duration_input],
-        outputs=[pdf_output, video_output, status_output]
+        fn=generate_pdf,
+        inputs=[title_input, plan_input, names_input, images_input],
+        outputs=[pdf_output, status_output]
     )
     
     clear_btn.click(
-        fn=lambda: ("", "", "", None, 5, None, None, "Cleared!"),
-        outputs=[title_input, plan_input, names_input, images_input, duration_input, pdf_output, video_output, status_output]
+        fn=clear_all,
+        outputs=[title_input, plan_input, names_input, images_input, status_output]
+    )
+    
+    # Real-time name counter
+    def count_names(names):
+        names_list = [n.strip() for n in names.split(',') if n.strip()]
+        count = len(names_list)
+        if count > 30:
+            return f"⚠️ {count} names (MAX 30 allowed!)"
+        return f"✅ {count} names ready"
+    
+    names_input.change(
+        fn=count_names,
+        inputs=[names_input],
+        outputs=[status_output]
     )
 
 if __name__ == "__main__":