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

Less lossy processing #24

Open
wants to merge 54 commits into
base: master
Choose a base branch
from
Open

Conversation

inakaboy
Copy link

@inakaboy inakaboy commented Oct 10, 2021

Various changes to reduce the chance that a user would lose data if they exported media files, processed them with this tool and then deleted the takeout files after making use of the processed files.

The main changes are:

  • Files with an unsupported extension or for which a JSON sidecar could not be found are now moved to the errors directory so the user can manually inspect them.
  • More detailed log messages (could be a bit cleaner but I tried to preserve the original logging as well).
  • Added some additional patterns that were causing my motion photo MP4s' JSON sidecars to not be found.
  • Added .webp, .dng, and .m4v to the supported files list because I had some files of these formats in my google photos export.

(sorry for all the random commits; I was editing directly in Github....)

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

Successfully merging this pull request may close these issues.

None yet

4 participants