pulsejet / memories

Fast, modern and advanced photo management suite. Runs as a Nextcloud app.
https://memories.gallery
GNU Affero General Public License v3.0
3.06k stars 82 forks source link

Canon Poor Exif Handling, Images Incorrectly Sorted #985

Open Civlac opened 8 months ago

Civlac commented 8 months ago

Describe the bug

To Reproduce

Platform:

Additional context

Feature Request

Civlac commented 8 months ago

Additionally, since not all cameras appear to have sub-second time data in the exif, has it been considered to make use of the filename for these instances?

pulsejet commented 8 months ago

Using the filename sounds like a good solution for now (though it wouldn't close this issue)

Civlac commented 8 months ago

Sounds good, and whether using Canon's Sub Sec metadata handling gets added in the future or not, other cameras, like an older Sony I looked at, doesn't have any Sub Second time data anywhere. Using the filename as a fallback when multiple files have an equal date and time sounds like a good fix for all images, beyond the scope of Canon.

pulsejet commented 8 months ago

Basename is now used as a tie breaker for the date. Leave this issue open since it's not really resolved.

Civlac commented 8 months ago

Thank you! As for the second bug, do you wish to address it here or should I move it to a separate Issue?

pulsejet commented 8 months ago

Yup, it's just that the "ideal" solution would also consider subsec metadata, then fall back to basename, then fall back to fileid (as earlier) etc.

Civlac commented 8 months ago

Sorry, I'm talking about the second bug I mentioned in the original post: The shutter speed is showing the "Shutter Speed Value", which can result in unfamiliar values. Should use "Exposure Time". I supplied a link under the additional context if you wish to have some more info on it

pulsejet commented 8 months ago

Ah yes, that's a separate bug. I've zero knowledge about these things.

Civlac commented 8 months ago

No worries, in hindsight I should've made a seperate post about it 😁

Hopefully that one should be easy to swap over. Maybe worth looking into what the "metadata" nextcloud app uses for shutter speed, just to have some confidence behind the change 👍