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.

Sign up for GitHub

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

Jump to bottom

Add metadata helper to generate metadata and keyframes preview image #59

Merged
merged 4 commits into from
Oct 26, 2023
Merged

Add metadata helper to generate metadata and keyframes preview image #59

merged 4 commits into from
Oct 26, 2023

Conversation

EverettSummer
Copy link
Member

@EverettSummer EverettSummer commented Oct 15, 2023 •
edited
Loading

This PR contains the following features:

  1. move the thumbnail of video file generation to video manager,
  2. get duration, width, height of video file
  3. generate a preview of frames every 2 seconds, and composite several NxN tile png with all preview images, N and number of images are determined by the number of frames.

@EverettSummer EverettSummer added the Ready To Test Labeled only a PR is safe to test label Oct 26, 2023
@EverettSummer EverettSummer added Ready To Test Labeled only a PR is safe to test and removed Ready To Test Labeled only a PR is safe to test labels Oct 26, 2023
@EverettSummer EverettSummer added Ready To Test Labeled only a PR is safe to test and removed Ready To Test Labeled only a PR is safe to test labels Oct 26, 2023
@EverettSummer EverettSummer merged commit 5f5bdc6 into master Oct 26, 2023
0 of 2 checks passed
@EverettSummer EverettSummer mentioned this pull request Oct 26, 2023
Open
@EverettSummer EverettSummer deleted the meta-data branch October 26, 2023 19:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@lordfriend lordfriend lordfriend approved these changes

@acgotaku acgotaku Awaiting requested review from acgotaku

Assignees
No one assigned
Labels
Ready To Test Labeled only a PR is safe to test
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@EverettSummer @lordfriend