FujiwaraChoki / MoneyPrinter

Automate Creation of YouTube Shorts using MoviePy.
MIT License
10.07k stars 1.35k forks source link

[BUG] "No videos found to download." #243

Closed Serback-Stable closed 5 months ago

Serback-Stable commented 6 months ago

Good morning, this is my first time using github so I'm going to use the stock bug template, if I'm missing something please tell me, I'll add it asap.

Describe the bug Basically after configuring both the front end and the backend and launching them, I'm getting this problem: "No videos found to download" before that, it tries to search for this terms, but it can't find anything (0 videos):

Generated 5 search terms: movies stock videos, cinematic stock footage, film clips, movie scenes, Hollywood footage => "movies stock videos" found 0 Videos => "cinematic stock footage" found 0 Videos => "film clips" found 0 Videos => "movie scenes" found 0 Videos => "Hollywood footage" found 0 Videos [-] No videos found to download. Also, I'm not getting any kind of mp4 output

To Reproduce Steps to reproduce the behavior:

  1. Go to '...'
  2. Generate output
  3. Get this error

Expected behavior MP4 File output

Screenshots 1 screenshot that shows 2 failed attemps.

Desktop (please complete the following information):

Serback-Stable commented 6 months ago

I forgot to add this, everytime I try to generate a file, it shows this, I do not know if it is normal: INFO:undetected_chromedriver.patcher:patching driver executable C:\Users\Enzo\appdata\roaming\undetected_chromedriver\undetected_chromedriver.exe

Krisseck commented 5 months ago

The video urls have changed in Pexels API.

In file Backend/search.py, line 48, change the string com/external to loc=external

I've made a fork: https://github.com/Krisseck/MoneyPrinterPlus

Serback-Stable commented 5 months ago

The video urls have changed in Pexels API.

In file Backend/search.py, line 48, change the string com/external to loc=external

I'm considering making a fork

Unfortunatly it did not change much, It can now rarely make 1 video, but for most imputes no videos get found. I think I'll try with MoneyPrinter V2, but if you have any other idea on how to fix it, it would be nice.

Troptrap commented 5 months ago

The best way to grab Pexels videos is not from the API itself, but constructing the url using ID of video. The end result looking like this: https://www.pexels.com/video/4916733/download/ where the numbers are the ID

Troptrap commented 5 months ago

I forgot to add this, everytime I try to generate a file, it shows this, I do not know if it is normal: INFO:undetected_chromedriver.patcher:patching driver executable C:\Users\Enzo\appdata\roaming\undetected_chromedriver\undetected_chromedriver.exe

This one is normal behavior. It comes from g4f (gpt for free) which uses a headless browser in background. That message is telling you that it uses a special driver for browser. It needs that to scrape websites like a human

ipharksai commented 5 months ago

Hey @FujiwaraChoki - just as a clarification for me, i keep running into these problems and all these issues are just closed by you with no further comment. Does that mean the issue is solved and what we experience is a new issue? Should we open new issues?

FujiwaraChoki commented 5 months ago

You're talking about separate problems in the same issue, and when once gets solved or clarified, I will close the issue. As @Troptrap mentioned, this has nothing to do with MoneyPrinter, rather g4f, if updating to the latest version doesn't help, then open an issue there, not here.

@ipharksai

Also, I'm not a full-time maintainer, so stuff like this depends on the community 💯

ipharksai commented 5 months ago

Just to clarify, I was not criticizing you, I'm new to all that and just kept running into issues and was not sure if I should open new issues or not.

Sorry if that came over as rude!

FujiwaraChoki commented 5 months ago

Just to clarify, I was not criticizing you, I'm new to all that and just kept running into issues and was not sure if I should open new issues or not.

Sorry if that came over as rude!

I didn't take it as rudeness, I was merely explaining ..