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

feat(10): refactor processing lib to MoviePy #12

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

paulhchoi
Copy link
Member

Closes #10

Description

This PR should take care of the incorrect mediafile generation by dumping the temp folder each run. It also adds:

  • refactoring the video encoding to use moviepy instead of ffmpeg to apply crossfades, which should just run the encoding in a single pass now
  • shifting the intro and outro.mp4's to the data file to allow us delete the tmpdir without incurring a re-download of those files (which would almost never be intentional)

How to Test

Run the script as usual for both audio and video processes and see that it generates as expected.

@paulhchoi paulhchoi self-assigned this Jun 17, 2024
@paulhchoi paulhchoi requested review from 13jkim and dhy6785 June 17, 2024 01:07
@dhy6785
Copy link

dhy6785 commented Jun 17, 2024

@paulhchoi nice

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Sermon-Processor Bug: Incorrect Media File Generation
2 participants