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

Long videos are not having their duration correctly fixed #794

Closed
rafaellehmkuhl opened this issue Mar 1, 2024 · 4 comments · Fixed by #852
Closed

Long videos are not having their duration correctly fixed #794

rafaellehmkuhl opened this issue Mar 1, 2024 · 4 comments · Fixed by #852
Assignees
@joaoantoniocardoso @rafaellehmkuhl
Labels
bug Something isn't working investigate P1 - Important Priority High priority task ux Has to do with the user experience video
Milestone
From Beta to Stable

Comments

@rafaellehmkuhl
Copy link
Member

Matthey from Carcinus has experienced that with a 2.5h video, and I can confirm I've seen this happen as well.

The duration fix on shorter videos appears to be working fine thought.
@rafaellehmkuhl rafaellehmkuhl added bug Something isn't working P1 - Important Priority High priority task ux Has to do with the user experience video labels Mar 1, 2024
@rafaellehmkuhl rafaellehmkuhl added this to the FromBetaToStable milestone Mar 1, 2024
@rafaellehmkuhl
Copy link
Member Author

This package is likely relevant here.

@joaoantoniocardoso joaoantoniocardoso mentioned this issue Mar 8, 2024
Merged
@rafaellehmkuhl
Copy link
Member Author

@joaoantoniocardoso mentioned that there's a big chance this was fixed with a change he made in MCM. If so, we can close.

@rafaellehmkuhl
Copy link
Member Author

Fixed by @joaoantoniocardoso in mavlink-camera-manager itself.

@rafaellehmkuhl
Copy link
Member Author

Unfortunately, it's still not fixed. It can be the fix-webm-duration package ruining things. I will take a look.

@rafaellehmkuhl rafaellehmkuhl reopened this Apr 1, 2024
@rafaellehmkuhl rafaellehmkuhl self-assigned this Apr 1, 2024
This was referenced Apr 1, 2024
Closed
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@joaoantoniocardoso joaoantoniocardoso

@rafaellehmkuhl rafaellehmkuhl

Labels
bug Something isn't working investigate P1 - Important Priority High priority task ux Has to do with the user experience video
Projects
Cockpit - Roadmap to Stable
Status: Done
Milestone
From Beta to Stable
3 participants
@joaoantoniocardoso @rafaellehmkuhl @JoaoMario109