Skip to content

Conversation

@timerring
Copy link
Owner

Description

Add the subtitles to the videos via OpenAI/whisper model.
Refactor the whole project structure.

Related Issues

fix #60

Who Can Review?

@timerring

TODO

  • If condition permits, I hope to infer larger parameters models rather than using the GTX 1650 with 4GB VRAM.

Checklist

  • Code has been reviewed
  • Code complies with the project's code standards and best practices
  • Code has passed all tests
  • Code does not affect the normal use of existing features
  • Code has been commented properly
  • Documentation has been updated (if applicable)
  • Demo/checkpoint has been attached (if applicable)

@timerring timerring self-assigned this Nov 28, 2024
Copy link
Owner Author

@timerring timerring left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Wait for the test.

Copy link
Owner Author

@timerring timerring left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pass test! GOOD.

@timerring timerring merged commit fa49703 into main Nov 29, 2024
@timerring timerring deleted the feat branch November 29, 2024 03:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Feature] Add subtitles to the video

2 participants