Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

combine_videos doesn't work at all #578

Open
1 task done
umutbozdag opened this issue Jan 25, 2025 · 16 comments
Open
1 task done

combine_videos doesn't work at all #578

umutbozdag opened this issue Jan 25, 2025 · 16 comments
Labels
bug Something isn't working

Comments

@umutbozdag
Copy link

umutbozdag commented Jan 25, 2025

是否已存在类似问题?

  • 我已搜索现有问题

当前行为

There are many logs like below:
webui | ## combining video: 1 => ./storage/tasks/37ba57c7-5a5f-450e-a959-a745410c4f6c/combined-1.mp4 webui | 2025-01-25 14:28:40 | INFO | "./app/services/video.py:61": combine_videos - max duration of audio: 43.44 seconds webui | 2025-01-25 14:28:40 | INFO | "./app/services/video.py:65": combine_videos - each clip will be maximum 3 seconds long webui | 2025-01-25 14:29:10 | INFO | "./app/services/video.py:141": combine_videos - Using transition mode: VideoTransitionMode.fade_in webui | 2025-01-25 14:29:11 | INFO | "./app/services/video.py:141": combine_videos - Using transition mode: VideoTransitionMode.fade_in webui | 2025-01-25 14:29:11 | INFO | "./app/services/video.py:141": combine_videos - Using transition mode: VideoTransitionMode.fade_in webui | 2025-01-25 14:29:11 | INFO | "./app/services/video.py:136": combine_videos - resizing video to 1080 x 1920, clip size: 720 x 1280 webui | 2025-01-25 14:29:11 | INFO | "./app/services/video.py:141": combine_videos - Using transition mode: VideoTransitionMode.fade_in webui | 2025-01-25 14:29:11 | INFO | "./app/services/video.py:141": combine_videos - Using transition mode: VideoTransitionMode.fade_in webui | 2025-01-25 14:29:11 | INFO | "./app/services/video.py:141": combine_videos - Using transition mode: VideoTransitionMode.fade_in webui | 2025-01-25 14:29:12 | INFO | "./app/services/video.py:136": combine_videos - resizing video to 1080 x 1920, clip size: 720 x 1280

then program exits itself with webui exited with code 0 and nothing happens, there is no combined-1.mp4 file.

How to solve this?

预期行为

I should see combined-1.mp4 file.

重现步骤

See above.

堆栈追踪/日志

See above.

Python 版本

v3.0.0

操作系统

macOS 14+

MoneyPrinterTurbo 版本

v1.2.2

其他信息

No response

@umutbozdag umutbozdag added the bug Something isn't working label Jan 25, 2025
@n0sys
Copy link

n0sys commented Jan 25, 2025

same issue .. any idea why ?

@Cattpot
Copy link

Cattpot commented Jan 27, 2025

same issue

Edit: I used the docker containers

@slashedzer0
Copy link

same issue, using both docker and manual deployments

(using github codespaces)

@slashedzer0
Copy link

slashedzer0 commented Jan 29, 2025

same issue, using both docker and manual deployments

(using github codespaces)

update: apparently turning off the subtitles fix the issue

@Maximepzv
Copy link

Maximepzv commented Feb 9, 2025

Same issue here using docker.
Turning off subtitles didn't help but then got some additionnals wanings :

ffmpeg_reader.py:157: UserWarning: In file /MoneyPrinterTurbo/storage/cache_videos/vid-37606542c6e29699beab9385968d0404.mp4, 6220800 bytes wanted but 0 bytes read at frame index 360 (out of a total 360 frames), at time 15.00/15.02 sec. Using the last valid frame instead.

Can provide further informations if needed to hep fixing this issue.

@badbojus
Copy link

same issue here... anyone got this working???

@jianan46
Copy link

same issue. macos 15.3

@anhadlamba30
Copy link

Same issue, mac 15.3

@sergicx
Copy link

sergicx commented Feb 19, 2025

Had the same issue, disabled the transitions and fixed

@badbojus
Copy link

Had the same issue, disabled the transitions and fixed

have disabled transitioons and subtitles. still same problem.

@QuantumMedic
Copy link

Same issue, mac 15.3

@AlexZhou1995
Copy link

AlexZhou1995 commented Feb 20, 2025

Same issue, mac 14.7
using docker

@activete
Copy link

activete commented Feb 20, 2025

I’m facing the same issue when generating an 8-minute video. Is there a limit on how long the video can be? When I generated a 30-second video, I got a properly combined final version, but for longer videos, the combined version isn’t saved. The individual parts are available in the task/project folder, but not the full version. Is there a maximum length limit for the transcript or video?"

Let me know if you want any tweaks! 😊

using docker:

my final terminal windows is :
webui | 2025-02-20 09:57:54 | INFO | "./app/services/video.py:141": combine_videos - Using transition mode: VideoTransitionMode.none
webui | 2025-02-20 09:57:54 | INFO | "./app/services/video.py:141": combine_videos - Using transition mode: VideoTransitionMode.none
webui | 2025-02-20 09:57:55 | INFO | "./app/services/video.py:141": combine_videos - Using transition mode: VideoTransitionMode.none
webui | 2025-02-20 09:57:55 | INFO | "./app/services/video.py:141": combine_videos - Using transition mode: VideoTransitionMode.none
webui | 2025-02-20 09:57:55 | INFO | "./app/services/video.py:141": combine_videos - Using transition mode: VideoTransitionMode.none
webui | 2025-02-20 09:57:55 | INFO | "./app/services/video.py:141": combine_videos - Using transition mode: VideoTransitionMode.none
webui exited with code 0
webui |
webui | You can now view your Streamlit app in your browser.
webui |
webui | URL: http://127.0.0.1:8501
webui |

@Darknesser
Copy link

遇到同样问题,docker内存分配的不够,合成视频内存占用很高,从6G调到12G之后成功了

Image

@100-heon
Copy link

i think should allocate minimun 16G ram

@elmsec
Copy link

elmsec commented Feb 22, 2025

  1. Allocate as much RAM as possible (>=14G)
  2. increase the "maximum duration of video clips" value to >4, as this lowers the number of clips to combine.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests