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

All exported videos have errors due to incorrect ffmpeg setting #402

Closed
wallobit opened this issue Jun 9, 2020 · 2 comments
Closed

All exported videos have errors due to incorrect ffmpeg setting #402

wallobit opened this issue Jun 9, 2020 · 2 comments

Comments

@wallobit
Copy link

wallobit commented Jun 9, 2020 •
edited
Loading

The ffmpeg parameter "-movflags use_metadata_tags" leads to the following errors in the exported videos:

  1. Existing tags such as comments are not displayed, but are still in the file. This can be demonstrated by copying the exported data again with "ffmpeg -i inputfile -c copy outputfile". Then they are back. Self-created tags like "MyFavorite=yes" are not displayed at all.

  2. For example comments can no longer be written to the damaged file later. You can also demonstrate if you copy the exported file again with "ffmpeg -i inputfile -c copy outputfile". Then comments can be re-entered.

Workaround: Delete the ffmpeg parameter "-movflags use_metadata_tags". So the standard tags such as comments are copied and can also be edited later. However, self-created tags like "MyFavorite=yes" etc. will not be copied. But this mistake was already there before.

The error can be on ffmpeg or there must be a new syntax for copying self-created tags.
@mifi
Copy link
Owner

mifi commented Jul 9, 2020

use_metadata_tags was added because of this #99
maybe that was a mistake...

@mifi mifi mentioned this issue Jul 9, 2020
Closed
mifi added a commit that referenced this issue Nov 25, 2020
@mifi
Many improvements
5390085 
- Implement an Export summary/confirmation sheet when pressing export
- Move output option buttons to export sheet
- Add config option to disable use_metadata_tags (default to false) #463 #402 #99
- Add config option for avoid_negative_ts #13
- Escape key to close sheets
- Change from mousetrap to hotkeys.js (better unbind - more compatible with react)
@mifi
Copy link
Owner

mifi commented Nov 25, 2020

In the next version there will be an option to not preserve MOV metadata (disable the -movflags use_metadata_tags flag). This should fix your issue.

@mifi mifi closed this as completed Nov 25, 2020
@mifi mifi mentioned this issue Feb 12, 2022
Open
@sandreas sandreas mentioned this issue Sep 26, 2022
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@mifi @wallobit