| Summary: | Doesn't handle timezones well | ||
|---|---|---|---|
| Product: | [Applications] kgeotag | Reporter: | Isaac Wismer <kdebugs> |
| Component: | General | Assignee: | Tobias Leupold <tl> |
| Status: | RESOLVED FIXED | ||
| Severity: | normal | CC: | tl |
| Priority: | NOR | ||
| Version First Reported In: | 1.0.0 | ||
| Target Milestone: | --- | ||
| Platform: | Compiled Sources | ||
| OS: | Linux | ||
| Latest Commit: | Version Fixed/Implemented In: | ||
| Sentry Crash Report: | |||
|
Description
Isaac Wismer
2021-01-08 21:10:27 UTC
Hi, thanks for your bug report! Am I right that you geotagged the photos in another timezone compared to the timezone they have been taken in? This might cause this ... Actually, we had an extensive discussion about timezone-aware data handling in KPhotoAlbum recently. IIRC, the photo metadata doesn't normally provide a timezone (only the time itself, thus we can't know it), whereas the times of an GPX track are always stored in UTC. If so, there's no other chance than setting the offset manually. I may be wrong though. Would you be so kind to send me an example photo and an example GPX file so that I can try to reproduce this? And the correct corresponding GPX timestamp? (via email if you want) I didn't see this with my test dataset. If it's possible, the expected behavior for KGeoTag would be to calculate the right offset and preset it. Ah, I didn't realize that the EXIF data doesn't contain the timezone. I guess that would explain why I had a hard time finding that data when making this bug report. I will send you an email with test data. If we can't automatically detect what timezone the photos are in, would it be possible to add a dropdown for selecting the timezone the photo was taken in (or at least an offset), so that I don't need to convert 6 hours into seconds manually? I'll have a look at it. Such an option would be no problem I think. Both manual timezone setting and automatic detection of timezones based on GPX data have been implemented now, so I think we can close this one :-) |