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

Video processing can take a long time for long videos #879

Closed
rafaellehmkuhl opened this issue Apr 10, 2024 · 1 comment · Fixed by #919
Closed

Video processing can take a long time for long videos #879

rafaellehmkuhl opened this issue Apr 10, 2024 · 1 comment · Fixed by #919
Assignees
@rafaellehmkuhl
Labels
enhancement New feature or request investigate performance ux Has to do with the user experience video
Milestone
From Beta to Stable

Comments

@rafaellehmkuhl
Copy link
Member

We should try to reduce it as much as possible.

If impossible to reduce it, we should investigate the possibility of doing it on a way it does not interfere with the usage of Cockpit.
Working with workers can be a solution.

Related to #878.
@rafaellehmkuhl rafaellehmkuhl added enhancement New feature or request investigate ux Has to do with the user experience performance video labels Apr 10, 2024
@rafaellehmkuhl rafaellehmkuhl added this to the From Beta to Stable milestone Apr 10, 2024
@rafaellehmkuhl rafaellehmkuhl changed the title Video processing takes too much time Video processing can take a long time for long videos Apr 16, 2024
@rafaellehmkuhl rafaellehmkuhl self-assigned this Apr 18, 2024
@rafaellehmkuhl
Copy link
Member Author

Fixed on #919.

Tested with a 49min video and it took 15 seconds to process (from former ~40s), of which about two thirds were from the webm duration fix (so more improvements will take a lot more effort).

@github-project-automation github-project-automation bot moved this from Backlog to Done in Cockpit - Roadmap to Stable May 1, 2024
@rafaellehmkuhl rafaellehmkuhl linked a pull request May 1, 2024 that will close this issue
sensors-logging: Find log by day prioritizing those from the last day #919
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@rafaellehmkuhl rafaellehmkuhl

Labels
enhancement New feature or request investigate performance ux Has to do with the user experience video
Projects
No open projects
Cockpit - Roadmap to Stable
Status: Done
Milestone
From Beta to Stable
Development

Successfully merging a pull request may close this issue.

sensors-logging: Find log by day prioritizing those from the last day rafaellehmkuhl/cockpit-public
1 participant
@rafaellehmkuhl