Bug 307223

Created attachment 74095 [details]
error0.png

Hi Marek

Can you please send me two screenshots? One showing the group option you select and other showing the problem in the image area? It doesnt need to be in English. We will figure that out.

Thanks.

Smit

> Can you please send me two screenshots? One showing the group option you select
> and other showing the problem in the image area? It doesnt need to be in
> English. We will figure that out.

OK, they are two screenshots
error0.png: polish option "Pogrupuj wybrane wg. czasu" (translate
error - properly "wg." is without dot at end)
error1.png: photo DSC05096.(ARW and JPG) aren't groupped. when I
change sort order, photo with error is another.

Created attachment 74096 [details]
error1.png

Hi Marek

I am not able to reproduce it here. However it might be possible that the time might be different by "seconds". to check that
(a) click on captions/tags on the right pane
(b) check the date and time

i have attached a screenshot to guide you how to reproduce the above steps.

if both are same for both the "raw" and "jpeg" image, then send those files to me here.

Smit

Created attachment 74130 [details]
description for checking the date

I found regularity of this bug. I click right button of mouse for display context menu over one of photos, and pair with this file is not created.

Created attachment 74965 [details]
group1.jpg

Hi Marek

I am not sure what you mean by your last comment. Can you give some screenshot of what you are referring to?

Smit

2012/11/3 Smit Mehta <smit.meh@gmail.com>:
> https://bugs.kde.org/show_bug.cgi?id=307223
>
> --- Comment #7 from Smit Mehta <smit.meh@gmail.com> ---
> Hi Marek
>
> I am not sure what you mean by your last comment. Can you give some screenshot
> of what you are referring to?

OK. Step by step:

1. I select all photos in album, and open context menu over any photo
in my example - IMG_0230.JPG (screen1.jpg)
2. After select group by time (pogrupuj wybrane wg czasu), all
selected files are grouped, exclude pair with this photo (screen2.jpg)
BUT
Bug doesn't appear when context menu is opened over any photo from
first pair in album (screen1a.jpg and screen2a.jpg)

Created attachment 74966 [details]
group2.jpg

Created attachment 74967 [details]
group1a.jpg

Created attachment 74968 [details]
group2a.jpg

Marek, thanks for your precise analysis.
Smit, look at DigikamImageView::createGroupByTimeFromSelection(), first line:
    QList<ImageInfo> selectedInfos = selectedImageInfosCurrentFirst();

"CurrentFirst" -> current info is prepended in the list, thus out of order. Can be replaced by the more simple selectedImageInfos().
Why is that a problem at all? The algorithm is simple, but seems to imply that the infos are sorted by date, which is often the case, but in no way guaranteed. Easiest way to fix is to qSort() the list with a custom lessThan method which sorts by date/time.
When it comes to nit-picking optimization, an algorithm which only iterates (by incrementing index or iterator) without editing the list (takeFirst) will be slightly more efficient.

Git commit 729399dcd5bf63046097ed3e479e2d2906ed82cc by Marcel Wiesweg.
Committed on 12/01/2013 at 18:47.
Pushed by mwiesweg into branch 'master'.

Rewrite algorithm for grouping by time:
- sort list by date, dont take this state for granted
- iterator-based on read-only list, less expensive than list editing
Works for me, testing appreciated.

M  +27   -8    digikam/items/digikamimageview.cpp

http://commits.kde.org/digikam/729399dcd5bf63046097ed3e479e2d2906ed82cc