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

Add video metadata display #231

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

Conversation

onyedikachi-david
Copy link
Contributor

@onyedikachi-david onyedikachi-david commented Dec 27, 2024

/claim #224
Fixes: #224

image

Copy link

vercel bot commented Dec 27, 2024

@onyedikachi-david is attempting to deploy a commit to the Cap Software Inc Team on Vercel.

A member of the Team first needs to authorize it.

@richiemcilroy
Copy link
Collaborator

Hey @onyedikachi-david. Thanks for this. Need to change it so that it does the following:

  • The time estimation should be the longest of either camera.mp4 or display.mp4.
  • Need to have an estimation of the rendered file size, not the current file size of the source file

let me know if you need anything else from me

onyedikachi-david and others added 3 commits January 13, 2025 16:03
- Use longest duration between camera.mp4 and display.mp4

- Replace source file size with encoder-based size estimation (8-10 Mbps video + 128 Kbps audio)
@onyedikachi-david
Copy link
Contributor Author

Hey @onyedikachi-david. Thanks for this. Need to change it so that it does the following:

  • The time estimation should be the longest of either camera.mp4 or display.mp4.
  • Need to have an estimation of the rendered file size, not the current file size of the source file

let me know if you need anything else from me

Hi, I have implemented them @richiemcilroy

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

Successfully merging this pull request may close these issues.

Time/size estimation for export
2 participants