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

"date-unknown" folder logic #197

Closed
wicked-head opened this issue Apr 11, 2023 · 3 comments
Closed

"date-unknown" folder logic #197

wicked-head opened this issue Apr 11, 2023 · 3 comments

Comments

@wicked-head
Copy link

Hi,

I have run the program for an extract that had multiple folders, and actually each folder already represented the year of the photo at least. But during that run, the program was unable to calculate the "real" date to put it into the folder structure.

The problem is that the program created a single folder named "date-unknown" and put everything into it, and now I dont even know the year of the photo.

Can I make the suggestion for the program to create the same folder structure (below the "Google Photos" folder) under the "date-unknown" folder and place the files under the correct folder structure? that would make it way easier to troubleshoot.

Thanks and sorry for the storm of tickets i've been creating

@TheLastGimbus
Copy link
Owner

hi!

storm of tickets

it's great to have bug reports!

um, no, i will not consider pure year to be good enought not to put it in "date unknown"

instead, let's try to solve the root problem!

how many photos do you have there? what are their names? do they have corresponding jsons? give me details about everything please, and we will fix this!

@wicked-head
Copy link
Author

Hi,

Thats is the true spirit :)

I discovered the root cause of it, the videos are from an Iphone when you take the picture and then it records as small video with the same name as the photo, so I think that there is not any JSON for the video, just the image...

bug.zip

I've attached an example and there is no json for the .MOV file... :(

@TheLastGimbus
Copy link
Owner

ohh, so it's the same case as #180 - let's merge everything there

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

No branches or pull requests

2 participants