Summary: | Displayed dates on Metadata panels come from different dates found in a video file from Ricoh GR II (inconsistency ?) | ||
---|---|---|---|
Product: | [Applications] digikam | Reporter: | mahikeulbody <51mfxqck> |
Component: | Metadata-Date | Assignee: | Digikam Developers <digikam-bugs-null> |
Status: | RESOLVED FIXED | ||
Severity: | minor | CC: | metzpinguin |
Priority: | NOR | ||
Version First Reported In: | 8.4.0 | ||
Target Milestone: | --- | ||
Platform: | Manjaro | ||
OS: | Linux | ||
Latest Commit: | https://invent.kde.org/graphics/digikam/-/commit/22614f113fb61a5e668368bb6312caacba032ccd | Version Fixed In: | 8.4.0 |
Sentry Crash Report: | |||
Attachments: | output of exiftool |
Description
mahikeulbody
2024-03-30 14:19:05 UTC
Created attachment 167947 [details]
output of exiftool
Since the video is larger than 4000KB, I attached the output of exiftool -a -G1 <video>.
Well, we assign the dates to those to whom they are "related". So Exif to Xmp.exif etc. There are always different times in videos, some take the start of the video, others the end. Then there is a video track date etc. Maik > So Exif to Xmp.exif
No, it is not the case (please read my NB).
There is another problem may be linked to that : if you adjust time from Exif:Created (here 14:34:10) to all date fields, including Exif:Created, Exif:Original and Exif:Digitized, none Exif metadata are updated into the file (ie they exist in the file but are not updated). I've also seen that Exif isn't updated, but that could be due to ExiffTool. Test whether it works on the command line. Sorry, I need a sample video, send it by email. For video we use not only ExifTool but also an FFmpeg metadata decoder and mix the results. Maik You can't equate video with images. There are manufacturers that store the Exif information in the embedded image in the video. So here the image would have to be changed and added back to the video. Read a little about the topic in the ExifTool forum. Maik > I need a sample video, send it by email.
I just send it by email.
I don't see any real problems here, the Exif metadata block with "2024:03:30 14:34:10" comes directly from ExifTool. We get the Xmp metadata from the "creation_time" from the FFmpeg API, which is probably the QuickTime date. Maik We could reasonably expect the dates into the XMP-Exif-specific Properties sub-panel to be the same as in Exif Panel. Having said that, I don't think this minor "bug" deserves much more attention. This report was more of a warning in case this behavior would not be what was intended. If the camera firmware results in different times from the start of a video to the time it is saved, we must also reflect them in the metadata. We cannot equate all times information as the ExifTool output also contains different information. Maik There are three different aspects in this report : 1) The one highlighted in the title. As shown by the "?" I was not very sure of me and I wanted above all to draw attention about that. You convinced me that Digikam is doing the right thing. 2) As a normal user, we could expect the dates into the XMP-Exif-specific Properties sub-panel to be the same as shown in Exif Panel (which, as a normal user, we presume to be the ones from Exif metadata in the file). Probably a cosmetic point but it can be disturbing. 3) If you adjust time from Exif:Created (here 14:34:10) to all date fields, including Exif:Created, Exif:Original and Exif:Digitized, none Exif metadata are updated into the file (ie they exist in the file but are not updated). Git commit 22614f113fb61a5e668368bb6312caacba032ccd by Maik Qualmann. Committed on 01/04/2024 at 12:43. Pushed by mqualmann into branch 'master'. add option to change all timestamps nativ with ExifTool FIXED-IN: 8.4.0 M +40 -38 core/dplugins/generic/metadata/timeadjust/timeadjustdialog.cpp M +5 -0 core/dplugins/generic/metadata/timeadjust/timeadjustlist.cpp M +2 -1 core/dplugins/generic/metadata/timeadjust/timeadjustlist.h M +29 -2 core/dplugins/generic/metadata/timeadjust/timeadjusttask.cpp M +7 -0 core/libs/metadataengine/exiftool/exiftoolparser.h M +24 -0 core/libs/metadataengine/exiftool/exiftoolparser_command.cpp M +5 -0 core/libs/metadataengine/exiftool/exiftoolparser_p.cpp M +1 -0 core/libs/metadataengine/exiftool/exiftoolprocess.h M +1 -0 core/libs/timeadjust/timeadjustcontainer.cpp M +1 -0 core/libs/timeadjust/timeadjustcontainer.h M +36 -9 core/libs/timeadjust/timeadjustsettings.cpp M +1 -0 core/libs/timeadjust/timeadjustsettings.h https://invent.kde.org/graphics/digikam/-/commit/22614f113fb61a5e668368bb6312caacba032ccd Git commit 72590299e811eb9e289e65b6112da1ccf71c4a13 by Maik Qualmann. Committed on 01/04/2024 at 13:04. Pushed by mqualmann into branch 'master'. update timestamps with ExifTool not in BQM M +2 -0 core/libs/timeadjust/timeadjustsettings.cpp https://invent.kde.org/graphics/digikam/-/commit/72590299e811eb9e289e65b6112da1ccf71c4a13 |