Fix video playback: Add ffmpeg params (yuv420p/faststart), cleanup image composition, add imageio-ffmpeg

modules/story_reels/services/story_creator.py

@@ -566,53 +566,54 @@ class StoryCreator:
                     duration = remaining
             
             # Create image clip
-            clip = ImageClip(image_path).set_duration(duration)
-            
-            # Resize to portrait (1080x1920)
-            clip = clip.resize(height=TARGET_HEIGHT)
-            
-            # Scene position-based effects
-            # Hook (first 2 clips): Zoom OUT (start big, end normal) - grabs attention
-            # Middle clips: Subtle zoom IN (Ken Burns)
-            # Outro (last clip): Static with fade out
-            
-            if i < 2:
-                # HOOK: Zoom OUT effect (1.1 → 1.0) - dynamic attention grabber
-                def make_zoom_out(t, clip_duration=duration):
-                    zoom = 1.1 - (0.1 * (t / clip_duration))  # 1.1 to 1.0
-                    return zoom
-                clip = clip.resize(lambda t: make_zoom_out(t))
-                
-            elif i < total_scenes - 1:
-                # MIDDLE: Ken Burns zoom IN (1.0 → 1.05)
-                def make_zoom_in(t, clip_duration=duration):
-                    zoom = 1.0 + (ZOOM_FACTOR - 1.0) * (t / clip_duration)
-                    return zoom
-                clip = clip.resize(lambda t: make_zoom_in(t))
+            # Create video clips from images
+            img_clip = ImageClip(str(image_path))
+            
+            # Resize logic:
+            # 1. Resize to cover screen height (maintaining aspect ratio)
+            # 2. Center crop to target width
+            
+            # Calculate resize factor to cover height
+            w, h = img_clip.size
+            ratio = TARGET_HEIGHT / h
+            new_width = int(w * ratio)
+            
+            if new_width < TARGET_WIDTH:
+                # If width is still too small, resize by width
+                ratio = TARGET_WIDTH / w
+                # new_height = int(h * ratio)
+                clip = img_clip.resize(width=TARGET_WIDTH)
+            else:
+                clip = img_clip.resize(height=TARGET_HEIGHT)
                 
-            # Last clip stays static (no zoom)
-            
-            # Center crop after zoom (to maintain 1080x1920)
-            clip = clip.resize(width=TARGET_WIDTH, height=TARGET_HEIGHT)
+            # Center Crop
+            clip = clip.crop(
+                x1=(clip.w - TARGET_WIDTH) // 2,
+                y1=0,
+                width=TARGET_WIDTH,
+                height=TARGET_HEIGHT
+            )
             
-            # Transitions: crossfade for smooth scene changes (NOT on first 2 clips)
-            # Hook clips: NO crossfade, clean direct cut
-            if i >= 2 and duration > CROSSFADE_DURATION:
-                clip = clip.crossfadein(CROSSFADE_DURATION)
+            # Set duration
+            clip = clip.set_duration(duration)
             
-            # NO fade in for Hook (first 2 clips) - start immediately visible!
-            # Only fade out at the very end
-            if i == total_scenes - 1:
-                clip = clip.fadeout(FADE_DURATION)
+            # Simple Zoom Effect (Ken Burns)
+            # We apply a slight zoom (1.0 -> 1.05) to all clips for movement
+            if duration > 0:
+               clip = clip.resize(lambda t: 1 + 0.04 * t / duration) 
+               # Re-crop to ensure no borders appear during zoom
+               clip = clip.crop(x_center=clip.w/2, y_center=clip.h/2, width=TARGET_WIDTH, height=TARGET_HEIGHT)
+
+            # Transitions
+            if i > 0:
+                clip = clip.crossfadein(crossfade_duration)
             
             clips.append(clip)
             total_video_duration += duration
         
-        # Concatenate with crossfade transitions
-        if len(clips) > 1:
-            video = concatenate_videoclips(clips, method="compose", padding=-CROSSFADE_DURATION)
-        else:
-            video = clips[0]
+        # Concatenate clips
+        # method="compose" is safer for crossfades
+        video = concatenate_videoclips(clips, method="compose", padding=-crossfade_duration)
         
         # Final safety: match video length to audio exactly
         if abs(video.duration - audio_duration) > 0.1:
@@ -677,7 +678,7 @@ class StoryCreator:
                 video = CompositeVideoClip([video] + caption_clips)
                 logger.info(f"Added {len(caption_clips)} caption clips")
         
-        # Write final video
+        # Write final video (with browser-compatible settings)
         logger.info(f"Writing video with effects: crossfade={CROSSFADE_DURATION}s, zoom={ZOOM_FACTOR}x")
         video.write_videofile(
             str(output_path),
@@ -685,7 +686,13 @@ class StoryCreator:
             codec='libx264',
             audio_codec='aac',
             threads=4,
-            preset='medium'
+            preset='medium',
+            ffmpeg_params=[
+                '-pix_fmt', 'yuv420p',      # Browser compatible pixel format
+                '-movflags', '+faststart',   # Enable streaming/progressive download
+                '-profile:v', 'baseline',    # Maximum compatibility
+                '-level', '3.0'              # Compatible with most devices
+            ]
         )
         
         # Cleanup

requirements.txt

@@ -24,3 +24,4 @@ groq
 # Utilities
 python-multipart
 huggingface_hub
+imageio-ffmpeg>=0.4.9