Bug 400777 - Group images by timelapse / burst [patch]
Summary: Group images by timelapse / burst [patch]
Status: RESOLVED FIXED
Alias: None
Product: digikam
Classification: Applications
Component: Albums-ItemGroup (show other bugs)
Version: 6.0.0
Platform: unspecified All
: NOR wishlist
Target Milestone: ---
Assignee: Digikam Developers
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2018-11-07 02:48 UTC by Krzysztof Kapusta
Modified: 2018-11-09 19:09 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed In: 6.0.0


Attachments
Patch implementing this functionality (7.08 KB, patch)
2018-11-07 02:48 UTC, Krzysztof Kapusta
Details
Updated patch to newest master (7.27 KB, patch)
2018-11-07 19:32 UTC, Krzysztof Kapusta
Details
Removed dependency on QStringView from patch (9.42 KB, patch)
2018-11-09 02:26 UTC, Krzysztof Kapusta
Details
Removed dependency on QStringView from patch (7.33 KB, patch)
2018-11-09 02:31 UTC, Krzysztof Kapusta
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Krzysztof Kapusta 2018-11-07 02:48:54 UTC
Created attachment 116139 [details]
Patch implementing this functionality

SUMMARY

Provide functionality to group images together when they were part of the same timelapse or burst. This will be especially usefull when timelapse photos are mixed with single-shots, or when importing from a sports camera like Gopro.

Photos from the same timelapse are assumed to have an increasing number in their filename (IMG0001.jpg, IMG00002.jpg and so on). They will be also taken with the same time interval between each other (ex. every 30 seconds or so).

STEPS TO REPRODUCE
1. Compile with the patch provided
2. Import some files that belong to a timelapse.
3. Select imported files (and more possibly) and use the context menu to select Group->Group By Timelapse / Burst

EXPECTED RESULT
Images belonging to the same timelapse should be grouped together. The selection process should be smart enough to distinguish several different timelapses from each other as well as ignore single-shot photos in between.

SOFTWARE VERSIONS
All

ADDITIONAL INFORMATION
All suggestions are welcome :) I'm wondering whether the minimum acceptable number of photos in a series should be configurable in the GUI. The patch assumes 3.
Comment 1 Maik Qualmann 2018-11-07 07:38:57 UTC
Can you please update the patch to compile with the digiKam git/master version? We've made some recent changes, for example, ImageInfo() is now ItemInfo().

Maik
Comment 2 Krzysztof Kapusta 2018-11-07 19:32:34 UTC
Created attachment 116159 [details]
Updated patch to newest master

Updated patch to newest master. Thanks Maik for pointing out my mistake :)
Comment 3 Maik Qualmann 2018-11-08 21:10:12 UTC
Your patch uses the QStringView class that exists only since Qt-5.10. DigiKam minimum Qt version is currently set to Qt-5.6.0. I think it will take some time before we can set the minimum QT version 5.10. It would be best if the patch would do without QStringView. Otherwise, the patch works fine.

Maik
Comment 4 Krzysztof Kapusta 2018-11-09 02:26:31 UTC
Created attachment 116190 [details]
Removed dependency on QStringView from patch

Removed dependency on QStringView from patch. It actually looks even better now.
Comment 5 Krzysztof Kapusta 2018-11-09 02:31:55 UTC
Created attachment 116191 [details]
Removed dependency on QStringView from patch
Comment 6 Maik Qualmann 2018-11-09 19:09:15 UTC
Git commit 333f2ff6a6e5c411f67547b5ef1905567f2594bc by Maik Qualmann.
Committed on 09/11/2018 at 19:08.
Pushed by mqualmann into branch 'master'.

apply patch #116191 from Krzysztof Kapusta to group images by timelapse / burst
FIXED-IN: 6.0.0

M  +3    -1    NEWS
M  +105  -0    core/app/items/imageviewutilities.cpp
M  +1    -0    core/app/items/imageviewutilities.h
M  +4    -0    core/app/utils/contextmenuhelper.cpp
M  +1    -0    core/app/utils/contextmenuhelper.h
M  +11   -0    core/app/views/digikamview.cpp
M  +1    -0    core/app/views/digikamview.h

https://commits.kde.org/digikam/333f2ff6a6e5c411f67547b5ef1905567f2594bc