Skip to content

Videos: Extract local time from DateTimeOriginal if possible #2640

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

Closed
2 tasks done
lastzero opened this issue Aug 24, 2022 · 3 comments
Closed
2 tasks done

Videos: Extract local time from DateTimeOriginal if possible #2640

lastzero opened this issue Aug 24, 2022 · 3 comments
Assignees
Labels
bug Something isn't working metadata Related to Exif, XMP, IPTC & Co. released Available in the stable release

Comments

@lastzero
Copy link
Member

lastzero commented Aug 24, 2022

As a Canon PowerShot owner, I would like the local time for my videos to be extracted from the DateTimeOriginal Exif field if possible, as they may not have GPS coordinates to calculate the local time from the UTC CreateDate.

Sample Files:

Acceptance Criteria:

  • The correct local time based on DateTimeOriginal is displayed in the UI after indexing
  • Unit Tests for the meta package have been added
@lastzero lastzero added metadata Related to Exif, XMP, IPTC & Co. bug Something isn't working labels Aug 24, 2022
@lastzero lastzero self-assigned this Aug 24, 2022
lastzero added a commit that referenced this issue Aug 24, 2022
Signed-off-by: Michael Mayer <michael@photoprism.app>
@lastzero lastzero added the please-test Ready for acceptance test label Aug 24, 2022
lastzero added a commit that referenced this issue Aug 24, 2022
Signed-off-by: Michael Mayer <michael@photoprism.app>
@dror3go
Copy link

dror3go commented Aug 24, 2022

Should a complete rescan be performed in order to test this?

lastzero added a commit that referenced this issue Aug 24, 2022
Signed-off-by: Michael Mayer <michael@photoprism.app>
@lastzero
Copy link
Member Author

lastzero commented Aug 24, 2022

Should a complete rescan be performed in order to test this?

You may limit the rescan to folders that contain videos or other files with extended metadata, e.g. complex timestamps from different sources. It is also possible to test with our demo, as it is always based on our latest preview.

@dror3go
Copy link

dror3go commented Sep 1, 2022

@lastzero Looks good to me!

@lastzero lastzero added released Available in the stable release and removed please-test Ready for acceptance test labels Sep 13, 2022
@lastzero lastzero moved this to Released 🌈 in Roadmap 🚀✨ Jun 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working metadata Related to Exif, XMP, IPTC & Co. released Available in the stable release
Projects
Status: Release 🌈
Development

No branches or pull requests

2 participants