[digikam] [Bug 416462] fuzzy search doesn't find photo which is rotated

2022-01-14 Thread Mario Frank
https://bugs.kde.org/show_bug.cgi?id=416462

--- Comment #10 from Mario Frank  ---
(In reply to Daniel from comment #8)
> Hi all,
> 
> so I think they are 2 different cases here (both could theoretically be
> solved by the same solution, but probably shouldn't), and I don't think the
> solution (at least for similar pictures, as written at the end) needs to be
> complicated (maybe I'm naive).
> 
> 
> A) The image is already correctly oriented before importing, using a set
> exif orientation flag.
> 
> If the image is imported (and autorotate is enabled), this flag gets removed
> and the images is fully rotated (that's my understanding).
> 
> The solution to this would be to read the orientation flag when doing the
> fuzzy search. Before running the hash algorith on it, the pictures need to
> be rotated according the flag. This should be almost trivial.
> 
> B) The image is rotated after it has been imported, different to the
> original orientation.
> 
> The solution to this could be to search the hashes for all 4 (let's ignore
> mirrors for now) orientations of the search query picture. I don't think
> this would be difficult or slow (as to my experience the search algorithm is
> very fast/instant). This could be extended to only search for additional
> orientations, if no result is found. If it is technically not possible,
> maybe someone can explain to me why :)
> 
> 
> Ok, this would have one major drawback: It could only find similar pictures,
> not duplicate entries. But if you assume all images are correctly rotated in
> the digikam collection, finding duplicates wouldn't be a problem anymore…
> 
> 
> Daniel

Hi Daniel,

first of all, the similarity search is only quite fast since the fingerprints
are computed
before searching and excessive caching is used. The computation of the
fingerprints
is quite slow. Especially for users with >100k images which is not much.
We should never forget that.

But I think you have a point. The orientation flag could help. I see two
possible ways:

1) For every image, two fingerprints are generated - one with current
orientation and
one with original orientation. And the user can select for the search whether
he wants
to search by original or current orientation.
This has drawbacks, although in most cases not grave ones. First, in many
cases, both
fingerprints of the image will be identical which is a waste of memory (but
this could be optimised).
And second, the database scheme needs to be adopted (not really a problem).
And not to forget, the fingerprint generation will be slower - it is hard to
estimate how much,
since the degree of rotated images differs from user to user.
But the user can decide depending on the situation, which is good.
This could be a solution.

2) The user decides in the fingerprint generation configuration whether
fingerprints shall
be generated for the current orientation or the original one.
In this case, there is no additional memory consumption and rotated images
should be
recognised as similars. One drawback is that if the user changes the
configuration, the
fingerprints partially need to be regenerated. But this drawback can be
reduced. Here, we
also have a possible solution. If the fingerprint entry in the DB gets an
additional orientation flag,
we can compare the orientation flag with the current configuration when
refreshing fingerprints.
So only the fingerprints of the changed and rotated images would be updated.

But I must admit that I do not know what information the orientation flag
provides.
Does it state whether or not the image was rotated or does it give the angle
(90/19/270°)?

Regards,
Mario

-- 
You are receiving this mail because:
You are watching all bug changes.

[digikam] [Bug 416462] fuzzy search doesn't find photo which is rotated

2022-01-14 Thread Mario Frank
https://bugs.kde.org/show_bug.cgi?id=416462

--- Comment #9 from Mario Frank  ---
(In reply to caulier.gilles from comment #7)
> Hi Mario and happy new year,
> 
> Well, i don't have a best idea, excepted the fact to use a deep learning
> method. Typically, We have a bug entry to detect and recognize monuments,
> places, animals, flower, tree, and more. This can be certainly extended to
> detect similar image based on shape.
> 
> I don't want mean to use DNN for similarity, but to check how rotation
> problem is treated with the DNN method, and to see if the algorithm is
> applicable to the current similarity code. 
> 
> For me the DNN performance is poor, especially if video cards acceleration
> is not used. Similarity algorithm has acceptable performances as well.
> 
> So yes, a GoSC 2022 project can be proposed around this topic.
> 
> Gilles

Hi Gilles, and I wish you a happy year, too.

I already had the same idea concerning DNN some time ago.
And I agree that DNN may not be our friend concerning performance in this case.
There seem to be concepts for detecting rotated objects. But an image may have
a huge amount of objects. And I did not find a description of recognising
whether
the complete image is rotated. In principle, this should be possible.
But again, the performance could be a big problem.
Nevertheless, it could be worth a try.

Regards,
Mario

-- 
You are receiving this mail because:
You are watching all bug changes.

[digikam] [Bug 416462] fuzzy search doesn't find photo which is rotated

2022-01-14 Thread Mario Frank
https://bugs.kde.org/show_bug.cgi?id=416462

Mario Frank  changed:

   What|Removed |Added

 CC||mario.fr...@uni-potsdam.de

--- Comment #6 from Mario Frank  ---
(In reply to Daniel from comment #5)
> (In reply to caulier.gilles from comment #4)
> Hi Gilles, 
> 
> a happy new year too :)
> > Can you check if problem remain with digiKam 7.5.0 pre-release bundle
> 
> I just checked and the problem still persists.
> 
> To be a more verbose:
> 
> If a image A.jpeg is imported which has exif orientation set (for example
> Orientation : Rotate 90 CW), digikam will rotate it and adapt the file (and
> lets say save it to B.jpeg).
> 
> If you search with the original image A.jpeg (not touched by digikam) in the
> fuzzy search, the imported image (B.jpeg) is not found. So I think the main
> issue is that the orientation flag isn't adhered to when searching using
> fuzzy search. (Supported by the fact that the orientation of the image
> preview in the fuzzy search box of A.jpeg is wrong (just horizontal))

Hi Daniel,

I am experiencing this flaw for a long time. Solving this is technically
difficult.
For every image, a HAAR fingerprint is calculated. This is comparable to a 
heatmap concerning brightness. So every brightness point gets a coordinate
which is calculated on the "current" orientation of the image. If the image was
rotated in digiKam, the rotation flag is set. In this case, it could with some
effort
be possible to compute the heatmap "unoriented". But in some cases the
information
about rotation is missing as the image was imported in this orientation but
without
orientation info.

The HAAR concept is not orientation-agnostic. In order to support rotated
images,
every image would need at least 4 heatmaps (for rotation) and even much more
as the image cold have been mirrored. And even then, it would be necessary to
compare every heatmap of the one image with every respective one of the other
image for similarity. It is possible to implement this. But the similarity
search would
have an unacceptable performance - I already tried that.

So I conclude that I do not think that this issue can be solved with the
current similarity
comparison concept without grave effect on the performance - at least not with 
combinatorial comparison. At least, I think this is a bigger project, e.g.
something
for GSoC. There are concepts that may be more appropriate (point distance
comparison
or even machine learning based) and the digiKam database and fuzzy/similarity
search
module in principle support using multiple comparison concepts since after
GSoC2017.

Perhaps Gilles has another idea.

Regards,
Mario

-- 
You are receiving this mail because:
You are watching all bug changes.

[digikam] [Bug 426116] Compact database (unique block instead of loose files)

2020-09-03 Thread Mario Frank
https://bugs.kde.org/show_bug.cgi?id=426116

Mario Frank  changed:

   What|Removed |Added

 CC||mario.fr...@uni-potsdam.de

--- Comment #5 from Mario Frank  ---
Hi Jo,

Depending on the OS, the default location of databases may be different.
So there is no easy catch for all OSs.
Usually, the location is defined in the digikamrc.

The digikamrc contains the following settings:

Database Name=/path/to/core/db/
Database Name Face=/path/to/core/db/
Database Name Similarity=/path/to/core/db/
Database Name Thumbnails=/path/to/core/db/

In your case, you could move the files to 

/home/library/application support/digikam/sqlite_db/

and set the values to 

Database Name=/home/library/application support/digikam/sqlite_db/
Database Name Face=/home/library/application support/digikam/sqlite_db/
Database Name Similarity=/home/library/application support/digikam/sqlite_db/
Database Name Thumbnails=/home/library/application support/digikam/sqlite_db/

When starting digiKam again, this should work without any problems.

Also, there is a command line option for digiKam:

--database-directory  

this one will search for the db files in the specified directory.

Does this solve your problem?

Cheers,
Mario

-- 
You are receiving this mail because:
You are watching all bug changes.

[digikam] [Bug 402659] Find Similar context menu action shows no result if used on an image without a fingerprint.

2019-01-09 Thread Mario Frank
https://bugs.kde.org/show_bug.cgi?id=402659

Mario Frank  changed:

   What|Removed |Added

   Version Fixed In||6.0.0
 Resolution|--- |FIXED
 Status|REPORTED|RESOLVED
  Latest Commit||https://commits.kde.org/dig
   ||ikam/b7908fc9894be318e2353a
   ||7bc1c4fdab28ded4ef

--- Comment #1 from Mario Frank  ---
Git commit b7908fc9894be318e2353a7bc1c4fdab28ded4ef by Mario Frank.
Committed on 09/01/2019 at 10:23.
Pushed by mfrank into branch 'master'.

If fuzzy image search is triggered for an image that currently has
no  or only dirty fingerprint, generate the fingerprint first.
FIXED-IN: 6.0.0

M  +2-1NEWS
M  +51   -0core/libs/database/similaritydb/similaritydb.cpp
M  +19   -0core/libs/database/similaritydb/similaritydb.h
M  +16   -0core/utilities/searchwindow/searchmodificationhelper.cpp

https://commits.kde.org/digikam/b7908fc9894be318e2353a7bc1c4fdab28ded4ef

-- 
You are receiving this mail because:
You are watching all bug changes.

[digikam] [Bug 377523] Automated option to delete duplicated pictures

2019-01-09 Thread Mario Frank
https://bugs.kde.org/show_bug.cgi?id=377523

Mario Frank  changed:

   What|Removed |Added

 Status|REPORTED|RESOLVED
 Resolution|--- |DUPLICATE
 CC||mario.fr...@uni-potsdam.de

--- Comment #2 from Mario Frank  ---


*** This bug has been marked as a duplicate of bug 261831 ***

-- 
You are receiving this mail because:
You are watching all bug changes.

[digikam] [Bug 261831] Auto-Process results from duplicates search

2019-01-09 Thread Mario Frank
https://bugs.kde.org/show_bug.cgi?id=261831

Mario Frank  changed:

   What|Removed |Added

 CC||sven.schicked...@gmail.com

--- Comment #6 from Mario Frank  ---
*** Bug 377523 has been marked as a duplicate of this bug. ***

-- 
You are receiving this mail because:
You are watching all bug changes.

[digikam] [Bug 392025] Wishlist: ability to separate images by similarity (instead of album or format)

2018-08-27 Thread Mario Frank
https://bugs.kde.org/show_bug.cgi?id=392025

Mario Frank  changed:

   What|Removed |Added

 CC||mario.fr...@uni-potsdam.de

--- Comment #2 from Mario Frank  ---
Hi Jens,

currently, only few similarity values are really stored in database. This was
not possible before 6.0.0 beta, as the database structure did not allow this
without really nasty constructions. Especially the similarities used by the
duplicates search are not stored easily extractible. To implement your wish,
some (not minor) work has to be done on the HaarIface. Sadly, I did not have
much time lately. I'll try to get some time free to prepare a better use of the
new similarity database. But this will probably not be part of the final 6.0.0
release.

Regards,
Mario

-- 
You are receiving this mail because:
You are watching all bug changes.

[plasmashell] [Bug 394489] Graphical glitches on panel

2018-06-08 Thread Mario Frank
https://bugs.kde.org/show_bug.cgi?id=394489

Mario Frank  changed:

   What|Removed |Added

 CC||mario.fr...@uni-potsdam.de

-- 
You are receiving this mail because:
You are watching all bug changes.

[plasmashell] [Bug 394489] Graphical glitches on panel

2018-06-08 Thread Mario Frank
https://bugs.kde.org/show_bug.cgi?id=394489

Mario Frank  changed:

   What|Removed |Added

 CC||skala.vacla...@seznam.cz

--- Comment #19 from Mario Frank  ---
*** Bug 395086 has been marked as a duplicate of this bug. ***

-- 
You are receiving this mail because:
You are watching all bug changes.

[plasmashell] [Bug 395086] Icons in UI is black or from firefox

2018-06-08 Thread Mario Frank
https://bugs.kde.org/show_bug.cgi?id=395086

Mario Frank  changed:

   What|Removed |Added

 Status|UNCONFIRMED |RESOLVED
 CC||mario.fr...@uni-potsdam.de
 Resolution|--- |DUPLICATE

--- Comment #6 from Mario Frank  ---


*** This bug has been marked as a duplicate of bug 394489 ***

-- 
You are receiving this mail because:
You are watching all bug changes.

[plasmashell] [Bug 394489] Graphical glitches on panel

2018-06-08 Thread Mario Frank
https://bugs.kde.org/show_bug.cgi?id=394489

Mario Frank  changed:

   What|Removed |Added

 CC||davidlev...@yahoo.com

--- Comment #20 from Mario Frank  ---
*** Bug 395125 has been marked as a duplicate of this bug. ***

-- 
You are receiving this mail because:
You are watching all bug changes.

[plasmashell] [Bug 395125] Icons drawn improperly on the desktop and in the task bar

2018-06-08 Thread Mario Frank
https://bugs.kde.org/show_bug.cgi?id=395125

Mario Frank  changed:

   What|Removed |Added

 Resolution|--- |DUPLICATE
 Status|UNCONFIRMED |RESOLVED
 CC||mario.fr...@uni-potsdam.de

--- Comment #1 from Mario Frank  ---


*** This bug has been marked as a duplicate of bug 394489 ***

-- 
You are receiving this mail because:
You are watching all bug changes.

[digikam] [Bug 390136] digiKam crashes when launching scan and recognize faces

2018-02-13 Thread Mario Frank
https://bugs.kde.org/show_bug.cgi?id=390136

Mario Frank <mario.fr...@uni-potsdam.de> changed:

   What|Removed |Added

 CC||mario.fr...@uni-potsdam.de

--- Comment #5 from Mario Frank <mario.fr...@uni-potsdam.de> ---
(In reply to Žygimantas Bruzgys from comment #4)
> Unfortunately, when it crashed I was not able to get the BT. And it seems
> that it was a transient crash, since after a restart it worked fine. I think
> that in order to reproduce this crash I would need to clear my faces DB,
> which I would not want to do right now.
> 
> As this seems to be dependent on an external dependency, which seems to be
> configured correctly in a release image, I'll mark the bug as resolved.

Hey Žygimantas,

if you have a SQLite DB, you can just copy your current faces DB file to some
safe place so you have a backup. Then you can delete the original face DB file
and digiKam will automatically generate the empty faces DB. Then you could
probably get a stack trace.

-- 
You are receiving this mail because:
You are watching all bug changes.

[digikam] [Bug 388981] Ability to set "Reference Album" for Find Duplicates

2018-01-15 Thread Mario Frank
https://bugs.kde.org/show_bug.cgi?id=388981

Mario Frank <mario.fr...@uni-potsdam.de> changed:

   What|Removed |Added

 CC||mario.fr...@uni-potsdam.de

--- Comment #1 from Mario Frank <mario.fr...@uni-potsdam.de> ---
Hey,
this is indeed currently not supported. And I agree with you that this is a
nice feature. I have now more free time - So I can do some digiKam development
again.

I will try to implement the feature for 5.9. There already are some topics in
the similarity subsystem that need to be covered.

-- 
You are receiving this mail because:
You are watching all bug changes.

[digikam] [Bug 381625] Copying/Moving photograph files between albums does not give Overwrite/Rename/Cancel options if file with same name exists in the destination album.

2017-10-27 Thread Mario Frank
https://bugs.kde.org/show_bug.cgi?id=381625

Mario Frank <mario.fr...@uni-potsdam.de> changed:

   What|Removed |Added

 CC||mario.fr...@uni-potsdam.de

--- Comment #6 from Mario Frank <mario.fr...@uni-potsdam.de> ---
(In reply to Maik Qualmann from comment #5)
> I also do not know whether I would like this dialog window. Personally I had
> never desire to decide by the small thumbnail and the file size, whether it
> is the same file. I would suggest that we rename an existing file with a
> counter and always copy it. Gilles, Simon and Mario how do you see that?
> 
> Maik

Hi Maik,

I think you are right - the thumbnail and file size are not always sufficient 
to decide whether the files are identical. The solution using a counter would
be a valid solution. But in this case, the user should be informed that there
were
clashes so he can review the files.

-- 
You are receiving this mail because:
You are watching all bug changes.

[digikam] [Bug 385368] DigiKam causes my machine to blue screen (crash) during installation on Windows 10

2017-10-05 Thread Mario Frank
https://bugs.kde.org/show_bug.cgi?id=385368

Mario Frank <mario.fr...@uni-potsdam.de> changed:

   What|Removed |Added

   Platform|Other   |MS Windows
 OS|Linux   |MS Windows

-- 
You are receiving this mail because:
You are watching all bug changes.

[digikam] [Bug 383326] Moving a tag does not update the parent tag correctly

2017-08-10 Thread Mario Frank
https://bugs.kde.org/show_bug.cgi?id=383326

--- Comment #3 from Mario Frank <mario.fr...@uni-potsdam.de> ---
Hey meku,

yes, I saw these errors yesterday, too. Getting rid of the errors is easy but
does not solve the main problem that lft and rgt are not updated properly.

If noone does solve the problem before I do, I will try again to solve it
tomorrow.

-- 
You are receiving this mail because:
You are watching all bug changes.

[digikam] [Bug 383326] Moving a tag does not update the parent tag correctly

2017-08-09 Thread Mario Frank
https://bugs.kde.org/show_bug.cgi?id=383326

Mario Frank <mario.fr...@uni-potsdam.de> changed:

   What|Removed |Added

 Status|UNCONFIRMED |CONFIRMED
 Ever confirmed|0   |1

-- 
You are receiving this mail because:
You are watching all bug changes.

[digikam] [Bug 383326] Moving a tag does not update the parent tag correctly

2017-08-09 Thread Mario Frank
https://bugs.kde.org/show_bug.cgi?id=383326

--- Comment #1 from Mario Frank <mario.fr...@uni-potsdam.de> ---
I got a small MySQL database to work right now and was able to reproduce

the corruption. I even got an error message in the trace:

"

digikam.dbengine: Prepare failed!
digikam.dbengine: Failure executing query:
 "LOCK TABLE Tags WRITE;"
Error messages: "QMYSQL3: Der Befehl konnte nicht initialisiert werden" "This
command is not supported in the prepared statement protocol yet" 1295 2

"

So there definitely is a problem and as I see it, this will have not worked for
a longer time. Looks like an incomplete migration from dbconfig to source code.

I will try to fix it.

-- 
You are receiving this mail because:
You are watching all bug changes.

[digikam] [Bug 383326] New: Moving a tag does not update the parent tag correctly

2017-08-09 Thread Mario Frank
https://bugs.kde.org/show_bug.cgi?id=383326

Bug ID: 383326
   Summary: Moving a tag does not update the parent tag correctly
   Product: digikam
   Version: 5.7.0
  Platform: Appimage
OS: Linux
Status: UNCONFIRMED
  Severity: major
  Priority: NOR
 Component: Database-Mysql
  Assignee: digikam-bugs-n...@kde.org
  Reporter: mario.fr...@uni-potsdam.de
  Target Milestone: ---

>From Nabble:
http://digikam.1695700.n4.nabble.com/Browsing-by-tag-sometimes-shows-items-with-alternative-tag-td4703595.html

The user experienced a tag in tag view containing an image that has neither
this tag nor a child tag assigned. The image was only shown if images from the
subtags are configured to be shown, too.

After analysing the database values for the tags that were assigned to the
image, he found out, that the lft and rgt values were in the range of the tag
in which he was seeing the image. He assumes that he has moved the really
assigned tag some time ago.

After looking at the code, I assume there is a bug in the case of MySQL
databases.

In TagDragDropHandler::dropEvent, the call
CoreDbAccess().db()->moveTag(newParentTag) is emitted if a MySQL database is
configured. This is done in order to update the lft and rgt values as this is
not done by triggers, opposed to QSLite.

But in CoreDB::moveTag the parameter is never used, as it is commented out.

I can not test it as I do not have a MySQL database. Also, I am not sure how to
fix it without the possibility for testing.

I rated the severity as major as the database is corrupted.

-- 
You are receiving this mail because:
You are watching all bug changes.

[digikam] [Bug 218312] Make search results to exclude children's tags

2017-07-12 Thread Mario Frank
https://bugs.kde.org/show_bug.cgi?id=218312

Mario Frank <mario.fr...@uni-potsdam.de> changed:

   What|Removed |Added

 Status|UNCONFIRMED |RESOLVED
   Version Fixed In||5.7.0
 Resolution|--- |FIXED
  Latest Commit||https://commits.kde.org/dig
   ||ikam/7306b537f67156bbc77522
   ||861522a7e17531a9b5

--- Comment #5 from Mario Frank <mario.fr...@uni-potsdam.de> ---
Git commit 7306b537f67156bbc77522861522a7e17531a9b5 by Mario Frank.
Committed on 12/07/2017 at 10:17.
Pushed by mfrank into branch 'master'.

Marked bug 218312 - "Make search results to exclude children's tags" as fixed
as this bug was in fact already
fixed in 5.5.0. Confirmed the resolution in current 5.7.0 AppImage. As the new
tag selection supports
selecting single tags, the subtags are searched only if they are marked
themselves.
FIXED-IN: 5.7.0

M  +2-2NEWS

https://commits.kde.org/digikam/7306b537f67156bbc77522861522a7e17531a9b5

-- 
You are receiving this mail because:
You are watching all bug changes.

[digikam] [Bug 377719] Cannot rename file with overwrite [patch]

2017-05-13 Thread Mario Frank
https://bugs.kde.org/show_bug.cgi?id=377719

Mario Frank <mario.fr...@uni-potsdam.de> changed:

   What|Removed |Added

 CC||mario.fr...@uni-potsdam.de

--- Comment #20 from Mario Frank <mario.fr...@uni-potsdam.de> ---
(In reply to caulier.gilles from comment #18)
> Simon, Maik, Mario,
> 
> The next 5.6.0 is planed this Sunday. I propose to delay by 2 weeks this
> release to permit to generate a test AppImage including this patch (and also
> other recent commits from Simon), at least to see if nothing is broken
> through users feedback.
> 
> Gilles

Hey,
I agree. I am currently quite busy with preparations for a publication. And I
would like to test some scenarios and fixe some more bugs. You already pointed
me to some problems.

-- 
You are receiving this mail because:
You are watching all bug changes.

[digikam] [Bug 377197] Customize grouping behaviour [patch]

2017-04-14 Thread Mario Frank
https://bugs.kde.org/show_bug.cgi?id=377197

Mario Frank <mario.fr...@uni-potsdam.de> changed:

   What|Removed |Added

 CC||mario.fr...@uni-potsdam.de

--- Comment #31 from Mario Frank <mario.fr...@uni-potsdam.de> ---
(In reply to Maik Qualmann from comment #30)
> I have not found any problems with the V8 patch. But I think we do not need
> to distinguish between BQM, light table, renaming, etc. An option for all
> cases (yes, no, ask) should be enough. Gilles, Mario what is your point of
> view?
> 
> Maik

I agree. Keep it simple if there is no really good reason. And I currently see
none.

-- 
You are receiving this mail because:
You are watching all bug changes.

[digikam] [Bug 183425] Advanced search by tags : "any tags" option does not work

2017-04-11 Thread Mario Frank
https://bugs.kde.org/show_bug.cgi?id=183425

Mario Frank <mario.fr...@uni-potsdam.de> changed:

   What|Removed |Added

  Latest Commit||https://commits.kde.org/dig
   ||ikam/660782be8137d8448ef10e
   ||5e944a2dd213d781cc
 Resolution|--- |FIXED
 Status|UNCONFIRMED |RESOLVED

--- Comment #21 from Mario Frank <mario.fr...@uni-potsdam.de> ---
Git commit 660782be8137d8448ef10e5e944a2dd213d781cc by Mario Frank.
Committed on 11/04/2017 at 17:27.
Pushed by mfrank into branch 'master'.

Added an overall improvement of fuzzy, duplicates and extended search.
1) Users can now restrict the results of fuzzy image/drop/sketch search to be
in specific albums.
2) Users can now select both tags and albums in duplicates search and restrict
the set of images in which duplicates are searched with logical operations (in
selected albums or/and tags, in selected albums but not tags and vice versa and
only in
selected objects of current tab)
3) In extended search, the tags selection can also be restricted such that
images must be either tagged with at least one of or with all selected tags.
Related: bug 353331, bug 300565

M  +7-1NEWS
M  +31   -3app/utils/searchmodificationhelper.cpp
M  +11   -0app/utils/searchmodificationhelper.h
A  +307  -0libs/album/albumselectioncomponent.cpp [License: UNKNOWN]  *
A  +121  -0libs/album/albumselectioncomponent.h [License: GPL (v2+)]
M  +248  -173  libs/album/albumselectors.cpp
M  +34   -19   libs/album/albumselectors.h
M  +7-0libs/database/coredb/coredbsearchxml.cpp
M  +1-0libs/database/coredb/coredbsearchxml.h
M  +1-0libs/database/dbjobs/dbjob.cpp
M  +11   -0libs/database/dbjobs/dbjobinfo.cpp
M  +4-0libs/database/dbjobs/dbjobinfo.h
M  +119  -30   libs/database/haar/haariface.cpp
M  +30   -8libs/database/haar/haariface.h
M  +32   -10   libs/database/item/imagelister.cpp
M  +44   -0libs/database/item/imagequerybuilder.cpp
M  +6-4libs/settings/applicationsettings.cpp
M  +3-0libs/settings/applicationsettings.h
M  +11   -0libs/settings/applicationsettings_miscs.cpp
M  +1-0libs/settings/applicationsettings_p.cpp
M  +2-0libs/settings/applicationsettings_p.h
M  +1-1utilities/facemanagement/facescandialog.cpp
M  +69   -22   utilities/fuzzysearch/findduplicatesview.cpp
M  +83   -21   utilities/fuzzysearch/fuzzysearchview.cpp
M  +1-0utilities/fuzzysearch/fuzzysearchview.h
M  +6-1utilities/maintenance/duplicatesfinder.cpp
M  +1-1utilities/maintenance/duplicatesfinder.h
M  +4-4utilities/maintenance/maintenancedlg.cpp
M  +2-1utilities/maintenance/maintenancemngr.cpp
M  +72   -27   utilities/searchwindow/searchfields.cpp
M  +18   -4utilities/searchwindow/searchfields.h

The files marked with a * at the end have a non valid license. Please read:
http://techbase.kde.org/Policies/Licensing_Policy and use the headers which are
listed at that page.


https://commits.kde.org/digikam/660782be8137d8448ef10e5e944a2dd213d781cc

-- 
You are receiving this mail because:
You are watching all bug changes.

[digikam] [Bug 353331] can’t limit fuzzy-search to albums [patch]

2017-04-11 Thread Mario Frank
https://bugs.kde.org/show_bug.cgi?id=353331

Mario Frank <mario.fr...@uni-potsdam.de> changed:

   What|Removed |Added

  Latest Commit||https://commits.kde.org/dig
   ||ikam/660782be8137d8448ef10e
   ||5e944a2dd213d781cc
 Resolution|--- |FIXED
 Status|UNCONFIRMED |RESOLVED

--- Comment #27 from Mario Frank <mario.fr...@uni-potsdam.de> ---
Git commit 660782be8137d8448ef10e5e944a2dd213d781cc by Mario Frank.
Committed on 11/04/2017 at 17:27.
Pushed by mfrank into branch 'master'.

Added an overall improvement of fuzzy, duplicates and extended search.
1) Users can now restrict the results of fuzzy image/drop/sketch search to be
in specific albums.
2) Users can now select both tags and albums in duplicates search and restrict
the set of images in which duplicates are searched with logical operations (in
selected albums or/and tags, in selected albums but not tags and vice versa and
only in
selected objects of current tab)
3) In extended search, the tags selection can also be restricted such that
images must be either tagged with at least one of or with all selected tags.
Related: bug 300565, bug 183425

M  +7-1NEWS
M  +31   -3app/utils/searchmodificationhelper.cpp
M  +11   -0app/utils/searchmodificationhelper.h
A  +307  -0libs/album/albumselectioncomponent.cpp [License: UNKNOWN]  *
A  +121  -0libs/album/albumselectioncomponent.h [License: GPL (v2+)]
M  +248  -173  libs/album/albumselectors.cpp
M  +34   -19   libs/album/albumselectors.h
M  +7-0libs/database/coredb/coredbsearchxml.cpp
M  +1-0libs/database/coredb/coredbsearchxml.h
M  +1-0libs/database/dbjobs/dbjob.cpp
M  +11   -0libs/database/dbjobs/dbjobinfo.cpp
M  +4-0libs/database/dbjobs/dbjobinfo.h
M  +119  -30   libs/database/haar/haariface.cpp
M  +30   -8libs/database/haar/haariface.h
M  +32   -10   libs/database/item/imagelister.cpp
M  +44   -0libs/database/item/imagequerybuilder.cpp
M  +6-4libs/settings/applicationsettings.cpp
M  +3-0libs/settings/applicationsettings.h
M  +11   -0libs/settings/applicationsettings_miscs.cpp
M  +1-0libs/settings/applicationsettings_p.cpp
M  +2-0libs/settings/applicationsettings_p.h
M  +1-1utilities/facemanagement/facescandialog.cpp
M  +69   -22   utilities/fuzzysearch/findduplicatesview.cpp
M  +83   -21   utilities/fuzzysearch/fuzzysearchview.cpp
M  +1-0utilities/fuzzysearch/fuzzysearchview.h
M  +6-1utilities/maintenance/duplicatesfinder.cpp
M  +1-1utilities/maintenance/duplicatesfinder.h
M  +4-4utilities/maintenance/maintenancedlg.cpp
M  +2-1utilities/maintenance/maintenancemngr.cpp
M  +72   -27   utilities/searchwindow/searchfields.cpp
M  +18   -4utilities/searchwindow/searchfields.h

The files marked with a * at the end have a non valid license. Please read:
http://techbase.kde.org/Policies/Licensing_Policy and use the headers which are
listed at that page.


https://commits.kde.org/digikam/660782be8137d8448ef10e5e944a2dd213d781cc

-- 
You are receiving this mail because:
You are watching all bug changes.

[digikam] [Bug 300565] When selecting folders, subfolders are not selected

2017-04-11 Thread Mario Frank
https://bugs.kde.org/show_bug.cgi?id=300565

Mario Frank <mario.fr...@uni-potsdam.de> changed:

   What|Removed |Added

 Status|UNCONFIRMED |RESOLVED
  Latest Commit||https://commits.kde.org/dig
   ||ikam/660782be8137d8448ef10e
   ||5e944a2dd213d781cc
 Resolution|--- |FIXED

--- Comment #7 from Mario Frank <mario.fr...@uni-potsdam.de> ---
Git commit 660782be8137d8448ef10e5e944a2dd213d781cc by Mario Frank.
Committed on 11/04/2017 at 17:27.
Pushed by mfrank into branch 'master'.

Added an overall improvement of fuzzy, duplicates and extended search.
1) Users can now restrict the results of fuzzy image/drop/sketch search to be
in specific albums.
2) Users can now select both tags and albums in duplicates search and restrict
the set of images in which duplicates are searched with logical operations (in
selected albums or/and tags, in selected albums but not tags and vice versa and
only in
selected objects of current tab)
3) In extended search, the tags selection can also be restricted such that
images must be either tagged with at least one of or with all selected tags.
Related: bug 353331, bug 183425

M  +7-1NEWS
M  +31   -3app/utils/searchmodificationhelper.cpp
M  +11   -0app/utils/searchmodificationhelper.h
A  +307  -0libs/album/albumselectioncomponent.cpp [License: UNKNOWN]  *
A  +121  -0libs/album/albumselectioncomponent.h [License: GPL (v2+)]
M  +248  -173  libs/album/albumselectors.cpp
M  +34   -19   libs/album/albumselectors.h
M  +7-0libs/database/coredb/coredbsearchxml.cpp
M  +1-0libs/database/coredb/coredbsearchxml.h
M  +1-0libs/database/dbjobs/dbjob.cpp
M  +11   -0libs/database/dbjobs/dbjobinfo.cpp
M  +4-0libs/database/dbjobs/dbjobinfo.h
M  +119  -30   libs/database/haar/haariface.cpp
M  +30   -8libs/database/haar/haariface.h
M  +32   -10   libs/database/item/imagelister.cpp
M  +44   -0libs/database/item/imagequerybuilder.cpp
M  +6-4libs/settings/applicationsettings.cpp
M  +3-0libs/settings/applicationsettings.h
M  +11   -0libs/settings/applicationsettings_miscs.cpp
M  +1-0libs/settings/applicationsettings_p.cpp
M  +2-0libs/settings/applicationsettings_p.h
M  +1-1utilities/facemanagement/facescandialog.cpp
M  +69   -22   utilities/fuzzysearch/findduplicatesview.cpp
M  +83   -21   utilities/fuzzysearch/fuzzysearchview.cpp
M  +1-0utilities/fuzzysearch/fuzzysearchview.h
M  +6-1utilities/maintenance/duplicatesfinder.cpp
M  +1-1utilities/maintenance/duplicatesfinder.h
M  +4-4utilities/maintenance/maintenancedlg.cpp
M  +2-1utilities/maintenance/maintenancemngr.cpp
M  +72   -27   utilities/searchwindow/searchfields.cpp
M  +18   -4utilities/searchwindow/searchfields.h

The files marked with a * at the end have a non valid license. Please read:
http://techbase.kde.org/Policies/Licensing_Policy and use the headers which are
listed at that page.


https://commits.kde.org/digikam/660782be8137d8448ef10e5e944a2dd213d781cc

-- 
You are receiving this mail because:
You are watching all bug changes.

[digikam] [Bug 353331] can’t limit fuzzy-search to albums [patch]

2017-04-11 Thread Mario Frank
https://bugs.kde.org/show_bug.cgi?id=353331

Mario Frank <mario.fr...@uni-potsdam.de> changed:

   What|Removed |Added

 Attachment #104963|0   |1
is obsolete||

--- Comment #24 from Mario Frank <mario.fr...@uni-potsdam.de> ---
Created attachment 104965
  --> https://bugs.kde.org/attachment.cgi?id=104965=edit
Updated patch for improvements

Updated the patch to the current state of the master branch and added some
comments concerning target albums.

-- 
You are receiving this mail because:
You are watching all bug changes.

[digikam] [Bug 353331] can’t limit fuzzy-search to albums [patch]

2017-04-11 Thread Mario Frank
https://bugs.kde.org/show_bug.cgi?id=353331

--- Comment #23 from Mario Frank <mario.fr...@uni-potsdam.de> ---
(In reply to caulier.gilles from comment #22)
> Not better terminology for the moment. Just put some comment to clarify it...
> 
> Gilles

Okay,

I would commit the changes with additional comments.
Is that okay for you?

-- 
You are receiving this mail because:
You are watching all bug changes.

[digikam] [Bug 378625] Very slow start

2017-04-11 Thread Mario Frank
https://bugs.kde.org/show_bug.cgi?id=378625

Mario Frank <mario.fr...@uni-potsdam.de> changed:

   What|Removed |Added

 CC||mario.fr...@uni-potsdam.de

--- Comment #4 from Mario Frank <mario.fr...@uni-potsdam.de> ---
I hope that my adoptions solve the problem. Sadly, I cannot reproduce it -
neither with 5.5, nor with 5.6 or current master branch.

The next AppImage will contain the adoptions.

-- 
You are receiving this mail because:
You are watching all bug changes.

[digikam] [Bug 378625] Very slow start

2017-04-11 Thread Mario Frank
https://bugs.kde.org/show_bug.cgi?id=378625

--- Comment #3 from Mario Frank <mario.fr...@uni-potsdam.de> ---
Git commit 29ee560d14ffb22d37b85a55f9d666a116097652 by Mario Frank.
Committed on 11/04/2017 at 12:16.
Pushed by mfrank into branch 'master'.

Ignore the signal that a search album was added until the find duplicates view
is active.
This is already done in fuzzy search view.
Otherwise, the automated sorting of entries is triggered which is not
necessary.

Hopefully, this solves bug id 378625.

M  +17   -1utilities/fuzzysearch/findduplicatesview.cpp
M  +1-0utilities/fuzzysearch/findduplicatesview.h
M  +1-0utilities/fuzzysearch/fuzzysearchview.cpp

https://commits.kde.org/digikam/29ee560d14ffb22d37b85a55f9d666a116097652

-- 
You are receiving this mail because:
You are watching all bug changes.

[digikam] [Bug 353331] can’t limit fuzzy-search to albums [patch]

2017-04-11 Thread Mario Frank
https://bugs.kde.org/show_bug.cgi?id=353331

--- Comment #21 from Mario Frank <mario.fr...@uni-potsdam.de> ---
(In reply to caulier.gilles from comment #20)
> writer.writeField(QLatin1String("noeffect_targetAlbums"), SearchXml::OneOf);
> writer.writeValue(targetAlbums);
> 
> ... why to call "noeffect_targetAlbums" a property which define the target
> album to process ? It's ambiguous... Or i miss understand something in your
> code ?
> 
> Gilles

Hey Gilles,

The field is only set for HAAR search, i.e. fuzzy image and sketch search.
The search can be restricted to only return images if they are in one of the
target folders.
Do you have a better terminus in mind?

Cheers,
Mario

-- 
You are receiving this mail because:
You are watching all bug changes.

[digikam] [Bug 353331] can’t limit fuzzy-search to albums

2017-04-11 Thread Mario Frank
https://bugs.kde.org/show_bug.cgi?id=353331

Mario Frank <mario.fr...@uni-potsdam.de> changed:

   What|Removed |Added

 Attachment #104892|0   |1
is obsolete||

--- Comment #19 from Mario Frank <mario.fr...@uni-potsdam.de> ---
Created attachment 104963
  --> https://bugs.kde.org/attachment.cgi?id=104963=edit
Updated patch for improvements

The AlbumSelectors class is also used in face scan. Updated calls to this class
in order to prevent regression bugs in face scan.

-- 
You are receiving this mail because:
You are watching all bug changes.

[digikam] [Bug 353331] can’t limit fuzzy-search to albums

2017-04-05 Thread Mario Frank
https://bugs.kde.org/show_bug.cgi?id=353331

Mario Frank <mario.fr...@uni-potsdam.de> changed:

   What|Removed |Added

 Attachment #104874|0   |1
is obsolete||

--- Comment #18 from Mario Frank <mario.fr...@uni-potsdam.de> ---
Created attachment 104892
  --> https://bugs.kde.org/attachment.cgi?id=104892=edit
Updated patch for improvements

The new patch adds patches for 
https://bugs.kde.org/show_bug.cgi?id=183425
and
https://bugs.kde.org/show_bug.cgi?id=300565

-- 
You are receiving this mail because:
You are watching all bug changes.

[digikam] [Bug 300565] When selecting folders, subfolders are not selected

2017-04-05 Thread Mario Frank
https://bugs.kde.org/show_bug.cgi?id=300565

Mario Frank <mario.fr...@uni-potsdam.de> changed:

   What|Removed |Added

 CC||mario.fr...@uni-potsdam.de

--- Comment #6 from Mario Frank <mario.fr...@uni-potsdam.de> ---
In https://bugs.kde.org/show_bug.cgi?id=353331 ,
a patch will be submitted that introduces context menus for album and tag
selection where subtrees can be marked explicitly. With the patch, subtrees are
not used automatically. The user can decide himself.

-- 
You are receiving this mail because:
You are watching all bug changes.

[digikam] [Bug 183425] Advanced search by tags : "any tags" option does not work

2017-04-05 Thread Mario Frank
https://bugs.kde.org/show_bug.cgi?id=183425

Mario Frank <mario.fr...@uni-potsdam.de> changed:

   What|Removed |Added

 CC||mario.fr...@uni-potsdam.de

--- Comment #20 from Mario Frank <mario.fr...@uni-potsdam.de> ---
Created attachment 104891
  --> https://bugs.kde.org/attachment.cgi?id=104891=edit
Draft for applicable operations on chosen tags in extended search

For tags, the user can select if the images to find must be in all the chosen
tags or at least in one.

The patch will be uploaded here: https://bugs.kde.org/show_bug.cgi?id=353331

-- 
You are receiving this mail because:
You are watching all bug changes.

[digikam] [Bug 353331] can’t limit fuzzy-search to albums

2017-04-04 Thread Mario Frank
https://bugs.kde.org/show_bug.cgi?id=353331

--- Comment #15 from Mario Frank <mario.fr...@uni-potsdam.de> ---
Created attachment 104874
  --> https://bugs.kde.org/attachment.cgi?id=104874=edit
Proposed patch for the improvements

This patch adds the following improvements to the fuzzy sidebar:

Duplicates search:
The user can select both tags and albums and apply one of 5 operations to the
selected objects:
- "One of" aka union - Search in the set of images that are either in the
selected tags or albums
- "Both" aka intersection - Search in the set of images that are both in the
selected tags and albums
- "Albums but not tags" aka difference - Search in the set of images that are
in the selected albums but not in the selected tags
- "Tags but not albums" aka difference - Search in the set of images that are
in the selected tags but not in the selected albums
- "Only selected tab" aka obvious - Search in the selected albums, if albums
tab is active and in the selected tags if tags tab is active

Fuzzy image (drop) search:
The user can now restrict the results to be in the specified albums.
Restricting to tags is technically possible but would impose complexity
problems potentially leading to a slower search - depending on the tagging
activities of the user.

Fuzzy sketch search:
The user can now restrict the results to be in the specified albums.
Restricting to tags is technically possible but would impose complexity
problems potentially leading to a slower search - depending on the tagging
activities of the user.

As I work on the current master branch, some texts are not localised.

-- 
You are receiving this mail because:
You are watching all bug changes.

[digikam] [Bug 353331] can’t limit fuzzy-search to albums

2017-04-04 Thread Mario Frank
https://bugs.kde.org/show_bug.cgi?id=353331

--- Comment #14 from Mario Frank <mario.fr...@uni-potsdam.de> ---
Created attachment 104873
  --> https://bugs.kde.org/attachment.cgi?id=104873=edit
New fuzzy sidebar : fuzzy sketch search

-- 
You are receiving this mail because:
You are watching all bug changes.

[digikam] [Bug 353331] can’t limit fuzzy-search to albums

2017-04-04 Thread Mario Frank
https://bugs.kde.org/show_bug.cgi?id=353331

--- Comment #13 from Mario Frank <mario.fr...@uni-potsdam.de> ---
Created attachment 104872
  --> https://bugs.kde.org/attachment.cgi?id=104872=edit
New fuzzy sidebar : fuzzy image search

-- 
You are receiving this mail because:
You are watching all bug changes.

[digikam] [Bug 353331] can’t limit fuzzy-search to albums

2017-04-04 Thread Mario Frank
https://bugs.kde.org/show_bug.cgi?id=353331

--- Comment #12 from Mario Frank <mario.fr...@uni-potsdam.de> ---
Created attachment 104871
  --> https://bugs.kde.org/attachment.cgi?id=104871=edit
New fuzzy sidebar : duplicates search

-- 
You are receiving this mail because:
You are watching all bug changes.

[digikam] [Bug 372972] Find duplicates "search in drop down" only shows 1 item

2017-04-03 Thread Mario Frank
https://bugs.kde.org/show_bug.cgi?id=372972

Mario Frank <mario.fr...@uni-potsdam.de> changed:

   What|Removed |Added

 Status|UNCONFIRMED |RESOLVED
 Resolution|--- |FIXED
  Latest Commit||https://commits.kde.org/dig
   ||ikam/5bb8e75151f1a2ca5ade6a
   ||8ea7b3c1e349996f54

--- Comment #4 from Mario Frank <mario.fr...@uni-potsdam.de> ---
Git commit 5bb8e75151f1a2ca5ade6a8ea7b3c1e349996f54 by Mario Frank.
Committed on 03/04/2017 at 09:16.
Pushed by mfrank into branch 'master'.

Fix some minor glitches in fuzzy search sidebar:
1) In a fresh installation, the root albums in the album selection box in
duplicates search are not automatically expanded.
Though this is no bug (expanding is possible via double-click), we now
automatically expand the root albums to not confuse users.
if nothing was expanded, yet.
2) In duplicates search, the last similarity thresholds are saved. But after
restart, one the range of the max similarity
threshold is 90-100 until the min similarity is changed. This is an ugly
glitch. Fixed that.
3) The computation of the similar images had a glitch when max similarity = min
similarity (e.g. 50%) The results were only
the ones with exactly 50.00 % but not also results with 50.xy as it should be.
Fixed that.
4) The xml structure of the haar search queries for fuzzy and sketch search
were not wellformed (the closing "search" tag was missing)
Related: bug 353331

M  +2-1NEWS
M  +3-0app/utils/searchmodificationhelper.cpp
M  +24   -7libs/album/albumtreeview.cpp
M  +7-1libs/database/haar/haariface.cpp
M  +1-1utilities/fuzzysearch/findduplicatesview.cpp

https://commits.kde.org/digikam/5bb8e75151f1a2ca5ade6a8ea7b3c1e349996f54

-- 
You are receiving this mail because:
You are watching all bug changes.

[digikam] [Bug 353331] can’t limit fuzzy-search to albums

2017-04-03 Thread Mario Frank
https://bugs.kde.org/show_bug.cgi?id=353331

--- Comment #11 from Mario Frank <mario.fr...@uni-potsdam.de> ---
Git commit 5bb8e75151f1a2ca5ade6a8ea7b3c1e349996f54 by Mario Frank.
Committed on 03/04/2017 at 09:16.
Pushed by mfrank into branch 'master'.

Fix some minor glitches in fuzzy search sidebar:
1) In a fresh installation, the root albums in the album selection box in
duplicates search are not automatically expanded.
Though this is no bug (expanding is possible via double-click), we now
automatically expand the root albums to not confuse users.
if nothing was expanded, yet.
2) In duplicates search, the last similarity thresholds are saved. But after
restart, one the range of the max similarity
threshold is 90-100 until the min similarity is changed. This is an ugly
glitch. Fixed that.
3) The computation of the similar images had a glitch when max similarity = min
similarity (e.g. 50%) The results were only
the ones with exactly 50.00 % but not also results with 50.xy as it should be.
Fixed that.
4) The xml structure of the haar search queries for fuzzy and sketch search
were not wellformed (the closing "search" tag was missing)
Related: bug 372972

M  +2-1NEWS
M  +3-0app/utils/searchmodificationhelper.cpp
M  +24   -7libs/album/albumtreeview.cpp
M  +7-1libs/database/haar/haariface.cpp
M  +1-1utilities/fuzzysearch/findduplicatesview.cpp

https://commits.kde.org/digikam/5bb8e75151f1a2ca5ade6a8ea7b3c1e349996f54

-- 
You are receiving this mail because:
You are watching all bug changes.

[digikam] [Bug 378219] Optimize includes and polish [patch]

2017-03-29 Thread Mario Frank
https://bugs.kde.org/show_bug.cgi?id=378219

Mario Frank <mario.fr...@uni-potsdam.de> changed:

   What|Removed |Added

 CC||mario.fr...@uni-potsdam.de

--- Comment #1 from Mario Frank <mario.fr...@uni-potsdam.de> ---
Did not have any problems compiling with your patch.

-- 
You are receiving this mail because:
You are watching all bug changes.

[digikam] [Bug 375651] Update 5.4.0 on Windows : no sound with all my videos

2017-03-10 Thread Mario Frank
https://bugs.kde.org/show_bug.cgi?id=375651

Mario Frank <mario.fr...@uni-potsdam.de> changed:

   What|Removed |Added

 CC||mario.fr...@uni-potsdam.de

--- Comment #32 from Mario Frank <mario.fr...@uni-potsdam.de> ---
(In reply to caulier.gilles from comment #31)
> Restore ? Why you delete all previous customization ?

Gilles, the way I understand Op, the installation of the new version has
overridden his configuration.

-- 
You are receiving this mail because:
You are watching all bug changes.

[digikam] [Bug 208713] Quicker deletion of multiple duplicate images

2017-02-28 Thread Mario Frank
https://bugs.kde.org/show_bug.cgi?id=208713

--- Comment #7 from Mario Frank <mario.fr...@uni-potsdam.de> ---
(In reply to Adam Pigg from comment #6)
> And only 8+ years later ;)

Well, sometimes wishes take a long time to fulfill. And some of the parts of
the wish were available at least in 5.0.0. It was just not identified as fixed
;)

-- 
You are receiving this mail because:
You are watching all bug changes.

[digikam] [Bug 377039] Network Collections cause hang on "Reading Database"

2017-02-28 Thread Mario Frank
https://bugs.kde.org/show_bug.cgi?id=377039

Mario Frank <mario.fr...@uni-potsdam.de> changed:

   What|Removed |Added

 CC||mario.fr...@uni-potsdam.de

--- Comment #1 from Mario Frank <mario.fr...@uni-potsdam.de> ---
Hey Andrew,

I think I'think would help if you provide us the log written during start of
digiKam. If you run digiKam from console, the log is written to stdout. There
are many possible reasons. So we need some log.

Cheers, 
Mario

-- 
You are receiving this mail because:
You are watching all bug changes.

[digikam] [Bug 351658] Prevent to fill whole memory when all CPU cores are used to process Maintenance tools

2017-02-28 Thread Mario Frank
https://bugs.kde.org/show_bug.cgi?id=351658

Mario Frank <mario.fr...@uni-potsdam.de> changed:

   What|Removed |Added

 CC||mario.fr...@uni-potsdam.de

--- Comment #32 from Mario Frank <mario.fr...@uni-potsdam.de> ---
Potentially, your problems could be solved in the current AppImage as I
restricted the count of threads that are opened in maintenance. 

Can you test that?

The AppImages are stored here by Gilles:
https://drive.google.com/drive/folders/0BzeiVr-byqt5Y0tIRWVWelRJenM

-- 
You are receiving this mail because:
You are watching all bug changes.

[digikam] [Bug 376359] Sort by Image Quality (assign quality tags) ignores many images

2017-02-28 Thread Mario Frank
https://bugs.kde.org/show_bug.cgi?id=376359

--- Comment #5 from Mario Frank <mario.fr...@uni-potsdam.de> ---
So, here are the AppImages:

https://drive.google.com/drive/folders/0BzeiVr-byqt5Y0tIRWVWelRJenM

-- 
You are receiving this mail because:
You are watching all bug changes.

[digikam] [Bug 375035] Out of memory during face tagging

2017-02-28 Thread Mario Frank
https://bugs.kde.org/show_bug.cgi?id=375035

Mario Frank <mario.fr...@uni-potsdam.de> changed:

   What|Removed |Added

 CC||mario.fr...@uni-potsdam.de

--- Comment #7 from Mario Frank <mario.fr...@uni-potsdam.de> ---
(In reply to lesf from comment #6)
> After activating swap digikam doesn't crash anymore -- it eats up most of
> the RAM, though.
> I guess it doesn't allocate any more memory because OpenCV is miffed and
> throws exceptions for every face-tagged picture:
> digikam.facesengine: Default exception from OpenCV

Can you test with the current AppImage again? I have done some changes in 5.5.0
which reduce the count of created threads. This may also have lead to out of
memory problems. It is possible that your problem does not occur any more.

The most current AppImages are always stored here by Gilles:
https://drive.google.com/drive/folders/0BzeiVr-byqt5Y0tIRWVWelRJenM

-- 
You are receiving this mail because:
You are watching all bug changes.

[digikam] [Bug 373638] Digikam doesn't show any files.

2017-02-28 Thread Mario Frank
https://bugs.kde.org/show_bug.cgi?id=373638

Mario Frank <mario.fr...@uni-potsdam.de> changed:

   What|Removed |Added

 CC||mario.fr...@uni-potsdam.de

--- Comment #2 from Mario Frank <mario.fr...@uni-potsdam.de> ---
Anything new here?

-- 
You are receiving this mail because:
You are watching all bug changes.

[digikam] [Bug 372680] Cannot delete files from NAS

2017-02-28 Thread Mario Frank
https://bugs.kde.org/show_bug.cgi?id=372680

Mario Frank <mario.fr...@uni-potsdam.de> changed:

   What|Removed |Added

 CC||mario.fr...@uni-potsdam.de

--- Comment #9 from Mario Frank <mario.fr...@uni-potsdam.de> ---
(In reply to dajomu from comment #8)
> Hi,
> 
> I could not figure out why digikam could not get access to trash and I am
> not even quite sure which trash it wants access to. Is it trash on my NAS or
> laptop? If it is trash on my laptop, I am wondering if digikam knows where
> Ubuntus' trash is located in the file system.

digiKam does not access the trash of your OS. It creates its own trash -
according to your log in folder /media/photos/.dtrash .
When you "delete" an image and go to the trash in digiKam, are the "deleted"
images listed there?

-- 
You are receiving this mail because:
You are watching all bug changes.

[digikam] [Bug 365331] digikam not loading because rpath information missing from libQt5Qml, libQt5Script and libQt5Quick

2017-02-28 Thread Mario Frank
https://bugs.kde.org/show_bug.cgi?id=365331

Mario Frank <mario.fr...@uni-potsdam.de> changed:

   What|Removed |Added

 CC||mario.fr...@uni-potsdam.de

--- Comment #6 from Mario Frank <mario.fr...@uni-potsdam.de> ---
(In reply to RJVB from comment #5)
> Edit: it would *seem* that the issue has resolved itself in 5.3.0 .
> 
> I agree something was wrong, but digiKam is the only application where I've
> seen this happen with my build settings, and only for a few Qt libraries.
> I don't know how the AppImage stuff is finagled together, but I'm not using
> that. I'm just using unpatched CMake files and everything KF5 is built with
> the settings required to apply full rpaths to all binaries. Qt5 itself is
> built that way too.
> 
> I'd agree the problem was fully on my end if I saw this happening to all Qt
> libraries (in DK or in other applications too), but that isn't the case.
> 
> The simplest explanation would be that somehow the rpath information is
> dropped for the affected Qt libraries

Is the problem resolved now? Can you test this with current master branch
(version 5.5.0)?

-- 
You are receiving this mail because:
You are watching all bug changes.

[digikam] [Bug 202030] Search duplicate images filtered by file type mime

2017-02-28 Thread Mario Frank
https://bugs.kde.org/show_bug.cgi?id=202030

Mario Frank <mario.fr...@uni-potsdam.de> changed:

   What|Removed |Added

 Resolution|--- |FIXED
 Status|UNCONFIRMED |RESOLVED
   Version Fixed In||5.5.0
  Latest Commit||https://commits.kde.org/dig
   ||ikam/c900075b8d938d81c2cf8d
   ||9e6b2d2ce83439fde5

--- Comment #8 from Mario Frank <mario.fr...@uni-potsdam.de> ---
Git commit c900075b8d938d81c2cf8d9e6b2d2ce83439fde5 by Mario Frank.
Committed on 28/02/2017 at 15:44.
Pushed by mfrank into branch 'master'.

Update the news. 1 bug is fixed by multiple changes in digikam in history.
FIXED-IN: 5.5.0

M  +2-1NEWS

https://commits.kde.org/digikam/c900075b8d938d81c2cf8d9e6b2d2ce83439fde5

-- 
You are receiving this mail because:
You are watching all bug changes.

[digikam] [Bug 202030] Search duplicate images filtered by file type mime

2017-02-28 Thread Mario Frank
https://bugs.kde.org/show_bug.cgi?id=202030

Mario Frank <mario.fr...@uni-potsdam.de> changed:

   What|Removed |Added

 CC||mario.fr...@uni-potsdam.de

--- Comment #6 from Mario Frank <mario.fr...@uni-potsdam.de> ---
Currently (5.4.0), it is possible to filter by album and tags.
In 5.5.0, it will be possible to restrict the result set even more. For
example, if you have an image I in Album A, you can exclude all other images in
Album A as potential duplicates of image I. So, if you have your raw images
together with the jpeg pictures in one album, they are filtered out.

By the way, 5.5.0 will also come with database cleanup. But this was not the
problem for your start-up.

Should this file stay open?

-- 
You are receiving this mail because:
You are watching all bug changes.

[digikam] [Bug 261831] Auto-Process results from duplicates search

2017-02-28 Thread Mario Frank
https://bugs.kde.org/show_bug.cgi?id=261831

--- Comment #4 from Mario Frank <mario.fr...@uni-potsdam.de> ---
>From https://bugs.kde.org/show_bug.cgi?id=372378 :

 piotergmo...@hotmail.com 2016-11-12 09:24:38 UTC

Digikam has powerfull search funtcion, which finds duplicates in albums. Nice
feature to have would be to *do* something after the search with the images.
The most obvious action could be: delete the duplicates, but the list could go
on to different scenarios. 

This is the real example which occured after I have imported from the mobile
camera 1000 photos  which could be imported previously, a years ago. They have
different file names of course, so from filesystem point of view they are
different. I would like to clean my albums, but in automated way and wonder
what is possible.
P.


 Wolfgang Scheffner 2016-11-12 18:14:05 UTC

Seems a bit difficult to me. How can an automated process decide which one of
two identical images to process (delete or whatever)? Of course you could set
the threshold to 100% and then say it doesn't matter, just process one of them.
But 1. your search result gets very small with 100% and 2. the process would
still need a rule to decide and that will most likely not match everybody's
needs.

-- 
You are receiving this mail because:
You are watching all bug changes.

[digikam] [Bug 372378] Action after the search of the duplicated images

2017-02-28 Thread Mario Frank
https://bugs.kde.org/show_bug.cgi?id=372378

Mario Frank <mario.fr...@uni-potsdam.de> changed:

   What|Removed |Added

 Resolution|--- |DUPLICATE
 Status|UNCONFIRMED |RESOLVED
 CC||mario.fr...@uni-potsdam.de

--- Comment #2 from Mario Frank <mario.fr...@uni-potsdam.de> ---


*** This bug has been marked as a duplicate of bug 261831 ***

-- 
You are receiving this mail because:
You are watching all bug changes.

[digikam] [Bug 261831] Auto-Process results from duplicates search

2017-02-28 Thread Mario Frank
https://bugs.kde.org/show_bug.cgi?id=261831

Mario Frank <mario.fr...@uni-potsdam.de> changed:

   What|Removed |Added

Version|1.2.0   |5.4.0
 CC||mario.fr...@uni-potsdam.de
Summary|Delete Duplicate photos by  |Auto-Process results from
   |image size  |duplicates search

--- Comment #2 from Mario Frank <mario.fr...@uni-potsdam.de> ---
I use this bug as container for bugs with equivalent wishlist

-- 
You are receiving this mail because:
You are watching all bug changes.

[digikam] [Bug 261831] Auto-Process results from duplicates search

2017-02-28 Thread Mario Frank
https://bugs.kde.org/show_bug.cgi?id=261831

Mario Frank <mario.fr...@uni-potsdam.de> changed:

   What|Removed |Added

 CC||piotergmo...@hotmail.com

--- Comment #3 from Mario Frank <mario.fr...@uni-potsdam.de> ---
*** Bug 372378 has been marked as a duplicate of this bug. ***

-- 
You are receiving this mail because:
You are watching all bug changes.

[digikam] [Bug 208713] Quicker deletion of multiple duplicate images

2017-02-28 Thread Mario Frank
https://bugs.kde.org/show_bug.cgi?id=208713

Mario Frank <mario.fr...@uni-potsdam.de> changed:

   What|Removed |Added

 CC||mario.fr...@uni-potsdam.de
 Status|CONFIRMED   |RESOLVED
 Resolution|--- |FIXED
   Version Fixed In||5.5.0

--- Comment #5 from Mario Frank <mario.fr...@uni-potsdam.de> ---
Choosing multiple duplicates entries from the left sidebar is available since
digiKam 5.4.0 (see https://bugs.kde.org/show_bug.cgi?id=366567 ). The reference
images are not shown as in the mockup (as mostleft images). But this is not
really easy to do. But in upcoming release 5.5.0, the reference images have the
mark "Reference image" in their image info and are thus easily recognised.
Selection by a selection box is also already available, as is grouping by
albums (not vertically but horizontally).

So, I close this file now.

-- 
You are receiving this mail because:
You are watching all bug changes.

[digikam] [Bug 375421] Same name under "Albums" for different collections with same underlying name

2017-02-28 Thread Mario Frank
https://bugs.kde.org/show_bug.cgi?id=375421

Mario Frank <mario.fr...@uni-potsdam.de> changed:

   What|Removed |Added

   Version Fixed In||5.5.0
  Latest Commit||https://commits.kde.org/dig
   ||ikam/c000a498d1776a52422069
   ||15773c41497786fafe
 Resolution|--- |FIXED
 Status|UNCONFIRMED |RESOLVED
 CC||mario.fr...@uni-potsdam.de

--- Comment #1 from Mario Frank <mario.fr...@uni-potsdam.de> ---
Fixed by https://bugs.kde.org/show_bug.cgi?id=376020#c4 . Next version (5.5.0)
or next pre-release AppImage will contain this fix.

-- 
You are receiving this mail because:
You are watching all bug changes.

[digikam] [Bug 376020] Moving of photos within the fuzzy search corrupts database

2017-02-28 Thread Mario Frank
https://bugs.kde.org/show_bug.cgi?id=376020

Mario Frank <mario.fr...@uni-potsdam.de> changed:

   What|Removed |Added

  Latest Commit|https://commits.kde.org/dig |https://commits.kde.org/dig
   |ikam/f58fc11f651c84cae37a75 |ikam/c000a498d1776a52422069
   |8a40ec18be1229b8b1  |15773c41497786fafe

--- Comment #4 from Mario Frank <mario.fr...@uni-potsdam.de> ---
Git commit c000a498d1776a5242206915773c41497786fafe by Mario Frank.
Committed on 28/02/2017 at 14:44.
Pushed by mfrank into branch 'master'.

If two root albums have the same name in file system, they are not
distinguishable as for both albums the header states Albums/albumnane.
But since all albums have a label, this one is better suited as albumname as
the user specifies the album himself.
FIXED-IN: 5.5.0

M  +2-1NEWS
M  +6-1libs/album/album.cpp
M  +13   -12   libs/album/album.h
M  +13   -0libs/database/collection/collectionmanager.cpp
M  +6-0libs/database/collection/collectionmanager.h

https://commits.kde.org/digikam/c000a498d1776a5242206915773c41497786fafe

-- 
You are receiving this mail because:
You are watching all bug changes.

[digikam] [Bug 372972] Find duplicates "search in drop down" only shows 1 item

2017-02-28 Thread Mario Frank
https://bugs.kde.org/show_bug.cgi?id=372972

Mario Frank <mario.fr...@uni-potsdam.de> changed:

   What|Removed |Added

 CC||mario.fr...@uni-potsdam.de

--- Comment #3 from Mario Frank <mario.fr...@uni-potsdam.de> ---
Here is the link where the current AppImages and Windows versions of the 5.5.0
pre-release are distributed by Gilles:

https://drive.google.com/drive/folders/0BzeiVr-byqt5Y0tIRWVWelRJenM

Can you test the following:
If your drop-down-menu shows only one entry, double-click it. It should expand
into the album tree. I had the same problem and it worked.

Cheers,
Mario

-- 
You are receiving this mail because:
You are watching all bug changes.

[digikam] [Bug 376020] Moving of photos within the fuzzy search corrupts database

2017-02-28 Thread Mario Frank
https://bugs.kde.org/show_bug.cgi?id=376020

Mario Frank <mario.fr...@uni-potsdam.de> changed:

   What|Removed |Added

  Latest Commit||https://commits.kde.org/dig
   ||ikam/f58fc11f651c84cae37a75
   ||8a40ec18be1229b8b1
   Version Fixed In||5.5.0
 Status|UNCONFIRMED |RESOLVED
 Resolution|--- |FIXED

--- Comment #3 from Mario Frank <mario.fr...@uni-potsdam.de> ---
Git commit f58fc11f651c84cae37a758a40ec18be1229b8b1 by Mario Frank.
Committed on 28/02/2017 at 13:02.
Pushed by mfrank into branch 'master'.

Added an option in context menu to group an image with another one and move it
to the album of the target image at the same time.
Though the problem described in 376020 is no bug, I think this use-case can
save users some work.
FIXED-IN: 5.5.0

M  +2-1NEWS
M  +26   -2app/dragdrop/imagedragdrop.cpp

https://commits.kde.org/digikam/f58fc11f651c84cae37a758a40ec18be1229b8b1

-- 
You are receiving this mail because:
You are watching all bug changes.

[digikam] [Bug 376359] Sort by Image Quality (assign quality tags) ignores many images

2017-02-24 Thread Mario Frank
https://bugs.kde.org/show_bug.cgi?id=376359

--- Comment #3 from Mario Frank <mario.fr...@uni-potsdam.de> ---
(In reply to Jens from comment #2)
> great, thank you!

Jens,
I merged the branch into master. But we will have to wait for the AppImage. I
remember Gilles writing somewhere that this will probably not be done before
next week.

So, you will have to wait for the AppImage or, if you can, compile the current
master branch state on your system. Just to warn you, the exiv version used in
most linux distros is 0.25 which has many bugs. 0.26 resolves over 200 bugs but
is sadly not yet released. Some of the bugs in 0.25 lead to crashes. Thats one
reason why we use AppImages. There we have a stable basis.

-- 
You are receiving this mail because:
You are watching all bug changes.

[digikam] [Bug 376661] When importing ~200,000 video files Digikam crashes in about 2-5 seconds of starting.

2017-02-24 Thread Mario Frank
https://bugs.kde.org/show_bug.cgi?id=376661

--- Comment #15 from Mario Frank <mario.fr...@uni-potsdam.de> ---
Hi again,

This will be a quite long text - sorry. But I want to make the problems as
clear as possible.

I thought about the fuzzy search for videos a bit more during my train travel.
In fact, even the first non-plain frame is worthless. If a user really wants to
use digiKam as catalog for videos (which is not the scope of digiKam in first
place IMHO), he will potentially have videos that have the same beginning, i.e.
intro but are different videos. Thus, also the first non-plain frame will
potentially lead to rubbish. I remember that I found some tools to find video
duplicates. The process they applied was to take the first n images of a video
and compare it to all others. A quite bad process IMO as with m videos you
generate n*m images and then have to make a comparison. This is awfully bad
from the view of complexity theory. And in practice, this process is, as can be
expected, awfully slow.

Nevertheless, the process is the probably best way to really recognise
duplicate videos. So, a way could be to generate a fingerprint over the first
or last n images (slows down fingerprint generation extremely). This still is
not robust as many videos may have the same intro (at least the first m
seconds, e.g. about m*25 frames. Usual intros take many seconds. So a *rather*
stable approach would be to take 1000 frames. As you can imagine, this is a big
amount of data to compute fingerprints for. Just imagine your 200,000 videos.
Fingerprinting them would mean to generate 200,000,000 images. Every image must
be generated which is no const-time process but at least linear time. So, even
with 1000 videos, i would expect computation time to be in measure of hours,
not minutes.

Let's take a look from the other side, outros are far more distinct than
intros. So, a lower number n can be taken, e.g. 100. This reduces the time
quite a lot. But is probably still not satisfying.

If no or only short intros/outros are there, only few images should be
sufficient and the process could work quite good.

But we cannot estimate, how the videos are structured. The FPS count may/will
differ from video to video. So, woking on frames explicitly may again lead to
low-quality results. So, the best way would be to take the n first/last seconds
and then the complexity cannot really be estimated here.
Also, I think, users should decide themselves, how many seconds are taken
(configuration) and if beginning or ending should be taken (configuration
again). 

So, *if* this feature should be implemented, I see the following options for
users:
1) Take the first non-plain frame for fingerprinting (fast, probable not
suitable for e.g. cinema movies)
2) Take the n first seconds for fingerprinting (probably awfully slow, may be
suitable for e.g. cinema movies, overkill for self-produced movies)
3) Take the n last seconds for fingerprinting (probably slow, probably suitable
for e.g. cinema movies, less overkill for self-produced movies)

In a more precise algorithmic way, we would need an adoption of the
fingerprints maintenance stage:
Option 1: take the first non-plain frame for video fingerprints
Option 2: take the Option(number n) Option(first,last) seconds for video
fingerprinting.
Changing the current options *must* trigger delete the current fingerprints of
videos as otherwise, different
fingerprintings would coexist which leads to wrong results - except rebuild all
fingerprints is chosen.

Then, the fuzzy search could probably work without adoptions - but I am not
completely sure if it would work out of the box.

Best,
Mario

-- 
You are receiving this mail because:
You are watching all bug changes.

[digikam] [Bug 376661] When importing ~200,000 video files Digikam crashes in about 2-5 seconds of starting.

2017-02-23 Thread Mario Frank
https://bugs.kde.org/show_bug.cgi?id=376661

Mario Frank <mario.fr...@uni-potsdam.de> changed:

   What|Removed |Added

 CC||mario.fr...@uni-potsdam.de

--- Comment #14 from Mario Frank <mario.fr...@uni-potsdam.de> ---
(In reply to Poz from comment #13)
> Are the thumbnails not readily available to do the fuzzy search on? I know
> they are not the biggest but I think they are big enough, or if there is a
> setting to render them a slightly higher resolution... That is how I
> imagined it would work anyways, since the thumbnails would already be
> generated, half the work is already done to fuzzy search videos...

Hey Poz,

Sadly, it is not this easy.
The fuzzy search creates a signature from images. This does not hold for
videos. Videos are quite more complex as the signature creation must be
uniformly done for all videos. But if videos have black frames in the
beginning, the search would lead to results which are, let's say, rubbish. The
most stable way I see is to take the first frame from every video that is not
plain, i.e. single-coloured. But this means we would have to generate images
until we find the first appropriate frame. This would slow down the
fingerprints generation significantly. 
A stable implementation is not trivial here. I will think about a way more
closely over the weekend. 

Best,
Mario

-- 
You are receiving this mail because:
You are watching all bug changes.

[digikam] [Bug 376815] Restore from Trash works incorrectly

2017-02-23 Thread Mario Frank
https://bugs.kde.org/show_bug.cgi?id=376815

Mario Frank <mario.fr...@uni-potsdam.de> changed:

   What|Removed |Added

   Version Fixed In|5.5 |5.5.0

-- 
You are receiving this mail because:
You are watching all bug changes.

[digikam] [Bug 376815] Restore from Trash works incorrectly

2017-02-23 Thread Mario Frank
https://bugs.kde.org/show_bug.cgi?id=376815

Mario Frank <mario.fr...@uni-potsdam.de> changed:

   What|Removed |Added

   Version Fixed In||5.5
 CC||mario.fr...@uni-potsdam.de
  Latest Commit||https://commits.kde.org/dig
   ||ikam/f528f24b54894b271a4bee
   ||44a850b1ac46c10cc9
 Resolution|--- |FIXED
 Status|REOPENED|RESOLVED

-- 
You are receiving this mail because:
You are watching all bug changes.

[digikam] [Bug 362023] Extremely slow metadata writing via maintenance

2017-02-22 Thread Mario Frank
https://bugs.kde.org/show_bug.cgi?id=362023

--- Comment #18 from Mario Frank <mario.fr...@uni-potsdam.de> ---
Git commit 2f8ddd42ef62d7aea9e490cdb05ffcc644810c81 by Mario Frank.
Committed on 22/02/2017 at 15:05.
Pushed by mfrank into branch 'master'.

Merged the current state of the garbage collection branch which improves the
database cleanup stage of the maintenance
and improves the reactiveness of the maintenance overall. We ported the way
items are processed to a queue based method
that can use the CPUs more effectively and does not create thousands of
threads.
Related: bug 283062, bug 216895, bug 374225, bug 351658, bug 329353
FIXED-IN: 5.5.0

M  +17   -12   NEWS

https://commits.kde.org/digikam/2f8ddd42ef62d7aea9e490cdb05ffcc644810c81

-- 
You are receiving this mail because:
You are watching all bug changes.

[digikam] [Bug 351658] Prevent to fill whole memory when all CPU cores are used to process Maintenance tools

2017-02-22 Thread Mario Frank
https://bugs.kde.org/show_bug.cgi?id=351658

--- Comment #31 from Mario Frank <mario.fr...@uni-potsdam.de> ---
Git commit 2f8ddd42ef62d7aea9e490cdb05ffcc644810c81 by Mario Frank.
Committed on 22/02/2017 at 15:05.
Pushed by mfrank into branch 'master'.

Merged the current state of the garbage collection branch which improves the
database cleanup stage of the maintenance
and improves the reactiveness of the maintenance overall. We ported the way
items are processed to a queue based method
that can use the CPUs more effectively and does not create thousands of
threads.
Related: bug 283062, bug 216895, bug 374225, bug 362023, bug 329353
FIXED-IN: 5.5.0

M  +17   -12   NEWS

https://commits.kde.org/digikam/2f8ddd42ef62d7aea9e490cdb05ffcc644810c81

-- 
You are receiving this mail because:
You are watching all bug changes.

[digikam] [Bug 374225] Add possibility to remove face identities by removing tag and remove person tags but preserving the tags themselves [patch]

2017-02-22 Thread Mario Frank
https://bugs.kde.org/show_bug.cgi?id=374225

Mario Frank <mario.fr...@uni-potsdam.de> changed:

   What|Removed |Added

   Version Fixed In||5.5.0
 Resolution|--- |FIXED
  Latest Commit||https://commits.kde.org/dig
   ||ikam/2f8ddd42ef62d7aea9e490
   ||cdb05ffcc644810c81
 Status|UNCONFIRMED |RESOLVED

--- Comment #24 from Mario Frank <mario.fr...@uni-potsdam.de> ---
Git commit 2f8ddd42ef62d7aea9e490cdb05ffcc644810c81 by Mario Frank.
Committed on 22/02/2017 at 15:05.
Pushed by mfrank into branch 'master'.

Merged the current state of the garbage collection branch which improves the
database cleanup stage of the maintenance
and improves the reactiveness of the maintenance overall. We ported the way
items are processed to a queue based method
that can use the CPUs more effectively and does not create thousands of
threads.
Related: bug 283062, bug 216895, bug 351658, bug 362023, bug 329353
FIXED-IN: 5.5.0

M  +17   -12   NEWS

https://commits.kde.org/digikam/2f8ddd42ef62d7aea9e490cdb05ffcc644810c81

-- 
You are receiving this mail because:
You are watching all bug changes.

[digikam] [Bug 216895] SQLITE : improve performance and reduce space with some pragma's

2017-02-22 Thread Mario Frank
https://bugs.kde.org/show_bug.cgi?id=216895

Mario Frank <mario.fr...@uni-potsdam.de> changed:

   What|Removed |Added

   Version Fixed In||5.5.0
  Latest Commit||https://commits.kde.org/dig
   ||ikam/2f8ddd42ef62d7aea9e490
   ||cdb05ffcc644810c81
 Resolution|--- |FIXED
 Status|UNCONFIRMED |RESOLVED

--- Comment #6 from Mario Frank <mario.fr...@uni-potsdam.de> ---
Git commit 2f8ddd42ef62d7aea9e490cdb05ffcc644810c81 by Mario Frank.
Committed on 22/02/2017 at 15:05.
Pushed by mfrank into branch 'master'.

Merged the current state of the garbage collection branch which improves the
database cleanup stage of the maintenance
and improves the reactiveness of the maintenance overall. We ported the way
items are processed to a queue based method
that can use the CPUs more effectively and does not create thousands of
threads.
Related: bug 283062, bug 374225, bug 351658, bug 362023, bug 329353
FIXED-IN: 5.5.0

M  +17   -12   NEWS

https://commits.kde.org/digikam/2f8ddd42ef62d7aea9e490cdb05ffcc644810c81

-- 
You are receiving this mail because:
You are watching all bug changes.

[digikam] [Bug 283062] THUMBDB : limit the size of thumbnail database

2017-02-22 Thread Mario Frank
https://bugs.kde.org/show_bug.cgi?id=283062

Mario Frank <mario.fr...@uni-potsdam.de> changed:

   What|Removed |Added

  Latest Commit||https://commits.kde.org/dig
   ||ikam/2f8ddd42ef62d7aea9e490
   ||cdb05ffcc644810c81
 Status|UNCONFIRMED |RESOLVED
   Version Fixed In||5.5.0
 Resolution|--- |FIXED

--- Comment #9 from Mario Frank <mario.fr...@uni-potsdam.de> ---
Git commit 2f8ddd42ef62d7aea9e490cdb05ffcc644810c81 by Mario Frank.
Committed on 22/02/2017 at 15:05.
Pushed by mfrank into branch 'master'.

Merged the current state of the garbage collection branch which improves the
database cleanup stage of the maintenance
and improves the reactiveness of the maintenance overall. We ported the way
items are processed to a queue based method
that can use the CPUs more effectively and does not create thousands of
threads.
Related: bug 216895, bug 374225, bug 351658, bug 362023, bug 329353
FIXED-IN: 5.5.0

M  +17   -12   NEWS

https://commits.kde.org/digikam/2f8ddd42ef62d7aea9e490cdb05ffcc644810c81

-- 
You are receiving this mail because:
You are watching all bug changes.

[digikam] [Bug 329353] Make slow processing better

2017-02-22 Thread Mario Frank
https://bugs.kde.org/show_bug.cgi?id=329353

--- Comment #3 from Mario Frank <mario.fr...@uni-potsdam.de> ---
Git commit 2f8ddd42ef62d7aea9e490cdb05ffcc644810c81 by Mario Frank.
Committed on 22/02/2017 at 15:05.
Pushed by mfrank into branch 'master'.

Merged the current state of the garbage collection branch which improves the
database cleanup stage of the maintenance
and improves the reactiveness of the maintenance overall. We ported the way
items are processed to a queue based method
that can use the CPUs more effectively and does not create thousands of
threads.
Related: bug 283062, bug 216895, bug 374225, bug 351658, bug 362023
FIXED-IN: 5.5.0

M  +17   -12   NEWS

https://commits.kde.org/digikam/2f8ddd42ef62d7aea9e490cdb05ffcc644810c81

-- 
You are receiving this mail because:
You are watching all bug changes.

[digikam] [Bug 376359] Sort by Image Quality (assign quality tags) ignores many images

2017-02-22 Thread Mario Frank
https://bugs.kde.org/show_bug.cgi?id=376359

Mario Frank <mario.fr...@uni-potsdam.de> changed:

   What|Removed |Added

 CC||mario.fr...@uni-potsdam.de

--- Comment #1 from Mario Frank <mario.fr...@uni-potsdam.de> ---
(In reply to Jens from comment #0)
> When I choose to let my images be tagged according to image quality using
> Digikam's default settings, only very few images are actually tagged.
> 
> I wonder if this is an expected behaviour? I would expect all images of all
> albums that I chose in the respective maintenance dialog to be tagged (with
> the colored flag, meaning "accepted", "pending" or "declined"). But most
> (>50%) images are not tagged at all.
> 
> Thanks!

Hey Jens,

I just tested myself. I have 3454 images in my database and ran the
maintenance.
In fact, all my images are tagged now. I am working on a branch currently.
It is possible that it works for me as a side effect due to changes I
introduced in the branch. I will signal you when I have merged the branch in
our master.
Then, you can test again after we have new AppImage bundles or compile it
yourself.

Cheers,
Mario

-- 
You are receiving this mail because:
You are watching all bug changes.

[digikam] [Bug 353331] can’t limit fuzzy-search to albums

2017-02-20 Thread Mario Frank
https://bugs.kde.org/show_bug.cgi?id=353331

Mario Frank <mario.fr...@uni-potsdam.de> changed:

   What|Removed |Added

 CC||mario.fr...@uni-potsdam.de

--- Comment #10 from Mario Frank <mario.fr...@uni-potsdam.de> ---
(In reply to sylvain.l.sauvage from comment #8)
> (In reply to caulier.gilles from comment #7)
> > Can we considerate this file fixed for last digiKam 5.0.0 ?
> 
> You do whatever want but if you’re asking me if it now works then the answer
> is a big no!
> 
> Two local collections.
> The combobox still presents them (and only them, not their sub-albums) as
> two “albums.”
> If I select one, only the root album is selected. And then the search is
> only made in the root album.
> If I use the context menu, I can add all the children of the selected
> collections but I still can’t chose among my albums.
> If the children are selected (“453 albums selected” shown), I can’t deselect
> them easily: deselecting one collection only deselects the root album of
> that collection (“452 albums selected” instead of the expected “255…”).
> 
> IOW, that UI is still a mess, nothing changed.
> If the only way to search in a selected set of albums is to go through the
> maintenance tool, then simply suppress that useless combobox.
> 
> To be frank, I’m disappointed with 5.0.0.
> The entire UI has not improved.
> Half the menus are not translated.
> Dialogs are always too small, half the elements are hidden (I don’t have
> that problem with my other Qt/KDE apps).
> Hiding the menu bar (Ctrl+M or via the menu) doesn’t work: it’s the status
> bar that disappear!
> 
> I had just compiled digikam 5 to test that bug, I’ll stay with Sid’s 4.14
> for now.

Sylvain,

I had the same problem as you had. But I only had one collection. I first
thought it was a bug in my database. But my database was new.
After some time of useless debugging, I found out, that I first have to
double-click on the collection. After doing that, all sub albums of this
collection were shown. So far, so good. I added another collection. In fuzzy
search, the first collection (and its sub-albums were shown). Additionally, I
now had the second collection (but no sub-albums for it). After double-klicking
on it, the sub-albums showed up.

Can you please test this with the current state of digiKam (Gilles wrote the
location)? I tested this with the current state from git.

Perhaps the missing functionality was just not shown prominently.

Cheers,
Mario

-- 
You are receiving this mail because:
You are watching all bug changes.

[digikam] [Bug 317210] THUMBDB : delete image removes file, but does not remove thumbnails

2017-02-09 Thread Mario Frank
https://bugs.kde.org/show_bug.cgi?id=317210

Mario Frank <mario.fr...@uni-potsdam.de> changed:

   What|Removed |Added

 Resolution|--- |FIXED
 Status|UNCONFIRMED |RESOLVED
   Version Fixed In||5.5.0
  Latest Commit||https://commits.kde.org/dig
   ||ikam/834221c8d1300bbdd4ae78
   ||e345cdc0fce3e92be8

--- Comment #39 from Mario Frank <mario.fr...@uni-potsdam.de> ---
Git commit 834221c8d1300bbdd4ae78e345cdc0fce3e92be8 by Mario Frank.
Committed on 09/02/2017 at 20:51.
Pushed by mfrank into branch 'master'.

Updated the news. The newly introduced garbage collection gets rid of
thumbnails in DB that are not referenced by any image any more.
FIXED-IN: 5.5.0

M  +2-1NEWS

https://commits.kde.org/digikam/834221c8d1300bbdd4ae78e345cdc0fce3e92be8

-- 
You are receiving this mail because:
You are watching all bug changes.

[digikam] [Bug 374591] Deleting image only removes the file and sets the status to hidden but does not delete the image from DB [patch]

2017-02-08 Thread Mario Frank
https://bugs.kde.org/show_bug.cgi?id=374591

Mario Frank <mario.fr...@uni-potsdam.de> changed:

   What|Removed |Added

  Latest Commit||https://commits.kde.org/dig
   ||ikam/a1f67531b3269941df5ff5
   ||31baa3e487cf58f1fc
 Resolution|--- |FIXED
   Version Fixed In||5.5.0
 Status|UNCONFIRMED |RESOLVED

--- Comment #17 from Mario Frank <mario.fr...@uni-potsdam.de> ---
Git commit a1f67531b3269941df5ff531baa3e487cf58f1fc by Mario Frank.
Committed on 08/02/2017 at 14:09.
Pushed by mfrank into branch 'master'.

Merged the garbage collection into master. The garbage collector is a
maintenance stage that runs before the rebuild of thumbnails and must be
triggered explicitely.
It removes stale image entries in core db and if enabled also stale thumbnails
and face identities from thumbnails and recognition DB.
If configured so, the core DB part of the garbage collector removes stale image
entries in core db during the start of digiKam.
Note that cleaning the databases does not necessarily make them smaller as no
auto-vacuum is done on the databases. The vacuuming proces differs highly
between the three
supported database variants (SQLite, internal MySQL and external MySQL). Thus,
currently there is no automatism.
Related: bug 323718
FIXED-IN: 5.5.0

M  +3-1NEWS

https://commits.kde.org/digikam/a1f67531b3269941df5ff531baa3e487cf58f1fc

-- 
You are receiving this mail because:
You are watching all bug changes.

[digikam] [Bug 323718] THUMBDB : rebuild all thumbnails does not get rid of all thumbnails first

2017-02-08 Thread Mario Frank
https://bugs.kde.org/show_bug.cgi?id=323718

Mario Frank <mario.fr...@uni-potsdam.de> changed:

   What|Removed |Added

   Version Fixed In||5.5.0
 Resolution|--- |FIXED
 Status|UNCONFIRMED |RESOLVED
  Latest Commit||https://commits.kde.org/dig
   ||ikam/a1f67531b3269941df5ff5
   ||31baa3e487cf58f1fc

--- Comment #16 from Mario Frank <mario.fr...@uni-potsdam.de> ---
Git commit a1f67531b3269941df5ff531baa3e487cf58f1fc by Mario Frank.
Committed on 08/02/2017 at 14:09.
Pushed by mfrank into branch 'master'.

Merged the garbage collection into master. The garbage collector is a
maintenance stage that runs before the rebuild of thumbnails and must be
triggered explicitely.
It removes stale image entries in core db and if enabled also stale thumbnails
and face identities from thumbnails and recognition DB.
If configured so, the core DB part of the garbage collector removes stale image
entries in core db during the start of digiKam.
Note that cleaning the databases does not necessarily make them smaller as no
auto-vacuum is done on the databases. The vacuuming proces differs highly
between the three
supported database variants (SQLite, internal MySQL and external MySQL). Thus,
currently there is no automatism.
Related: bug 374591
FIXED-IN: 5.5.0

M  +3-1NEWS

https://commits.kde.org/digikam/a1f67531b3269941df5ff531baa3e487cf58f1fc

-- 
You are receiving this mail because:
You are watching all bug changes.

[digikam] [Bug 374591] Deleting image only removes the file and sets the status to hidden but does not delete the image from DB [patch]

2017-02-08 Thread Mario Frank
https://bugs.kde.org/show_bug.cgi?id=374591

--- Comment #16 from Mario Frank <mario.fr...@uni-potsdam.de> ---
Merge is done. I will close this file now

-- 
You are receiving this mail because:
You are watching all bug changes.

[digikam] [Bug 374591] Deleting image only removes the file and sets the status to hidden but does not delete the image from DB [patch]

2017-02-08 Thread Mario Frank
https://bugs.kde.org/show_bug.cgi?id=374591

--- Comment #15 from Mario Frank <mario.fr...@uni-potsdam.de> ---
(In reply to caulier.gilles from comment #14)
> Hi Mario,
> 
> Well, merge back will be done when you decide. If more time is necessary to
> hack, we can postbone release date for 5.5.0.
> 
> In other word, we don't have any restriction here (:=)))...
> 
> Gilles

Hey Gilles,
the stale data (in ImageTags) seems to be the result of some queries I made in
the DB with my DB browser.
I did not find more stale data in core DB that can be cleaned.
I just resolved the last thing I disliked. Before 5.5, items were marked as
Removed (now Trashed) and deleted after a retention period. So users may have
items with that status. But the garbage collection only removes Obsolete items. 
I adopted the CollectionsScanner to marke trashed items as obsolete after a
retention period. This way, users will also get rid of old trashed items.

Also, there are no more tables in thumbs and recognition db that are not
handled by the garbage collection.

To conclude: in functionality, we are finished for now. I am in fact working
with the garbage collection since we have the branch and did not encounter
problems. I would merge the branch into the master today. Then, more people can
test.

No need to postpone 5.5 - except users raid us with bug requests.

-- 
You are receiving this mail because:
You are watching all bug changes.

[digikam] [Bug 374591] Deleting image only removes the file and sets the status to hidden but does not delete the image from DB [patch]

2017-02-08 Thread Mario Frank
https://bugs.kde.org/show_bug.cgi?id=374591

--- Comment #13 from Mario Frank <mario.fr...@uni-potsdam.de> ---
What about the garbage collection branch. When will we merge it to master?
I identified some potential stale data, i.e. table entries that reference a
non-existent image id, in core DB that can also be collected.
I will implement that when I have the time.

-- 
You are receiving this mail because:
You are watching all bug changes.

[digikam] [Bug 375703] Moving grouped images into another album removes groups

2017-02-08 Thread Mario Frank
https://bugs.kde.org/show_bug.cgi?id=375703

--- Comment #15 from Mario Frank <mario.fr...@uni-potsdam.de> ---
Okay,

I thought about the problems for some time.

I try to resume the expected behaviour:
1) Moving a group of items should preserve the grouping relation on the moved
items (Bug description by Jens) - This works in garbage collection branch
without workaround.
2) Moving an item that is grouped should preserve its grouping relation - This
also works in garbage collection branch without the workaround.
3) Copying a group within digiKam should also copy the grouping relation for
the newly generated items - This is not done.
4) Copying an item of a group within digiKam - This is not clear. Should the
new item belong to the group?
5) Importing images into digiKam that are identical to grouped items - Should
the imported images be grouped automatically?
6) What do you mean with
> Also externally copied grouped images with the file manager makes problems.
   I assume that you mean that some user copies grouped images that are
imported in digiKam with a file manager.
   But this then also makes problems when grouped images are moved with a file
manager.

I give here my opinion to 3-6 and I am open for discussion. I will try to make
my arguments as precise and founded as possible.

I would state the following for 3 and 4: Copy means copy and we should give the
user what can be expected by the terms we use. Since we copy all image
properties, the grouping relation should be copied in principle, too. But the
problem here is: What does the user expect? Should the copies have a grouping
relation with the source group leader or should the copied group be a new group
with the copy of the old group leader being the new group leader? I cannot
answer this question. Only the users can. So the default would be for me to not
set group relations for copied items and (potentially) give the user an option
to decide about the relations to established.

I would state the following for 5: If a user imports some images into digiKam,
we cannot know whether he wants to establish a group or add the images to a
group. Also, the grouping relation is existent in digiKam but not in file
managers - at least not that I know. Thus, the imported images should not be
grouped automatically. We could ask the user or give an option as import. But I
would not take the decision from the user.

I would state the following for 6: I like good user experience. But this case
is a clear misuse for me. One cannot expect some complex software to work
properly when tampering with the resources managed by the software. If you just
copy an image tag relation in our database, you will also have unexpected
behaviour if our constraints do prohibit that. And this also holds for other
software projects. If you copy some bundles from a OSGI container, he will
probably yell at you to stop tampering with his internal resources. Probably
with a crash. But we cannot prohibit changes in the file system level. We
cannot even detect them if digiKam is not started. This is - at least for me -
something that should be stated in the documentation.

To conclude: 
cases 3 to 5 are no bugs for me. I think they are usability issues and we could
try to give the user the control about what is done with the group
1) establish a new group with copy of group leader being new group leader
2) link to existent group leader
3) do nothing (default)
case 6 is misuse and should be documented.

Opinions?

All the best
Mario

-- 
You are receiving this mail because:
You are watching all bug changes.

[digikam] [Bug 375703] Moving grouped images into another album removes groups

2017-02-07 Thread Mario Frank
https://bugs.kde.org/show_bug.cgi?id=375703

--- Comment #11 from Mario Frank <mario.fr...@uni-potsdam.de> ---
I have to correct myself. Moving the group back suddenly destroyed the group
relation... That's odd.

-- 
You are receiving this mail because:
You are watching all bug changes.

[digikam] [Bug 375703] Moving grouped images into another album removes groups

2017-02-07 Thread Mario Frank
https://bugs.kde.org/show_bug.cgi?id=375703

--- Comment #10 from Mario Frank <mario.fr...@uni-potsdam.de> ---
(In reply to Maik Qualmann from comment #8)
> I have for a test the workaround code removed.
> 
> https://cgit.kde.org/digikam.git/tree/libs/database/item/imagescanner.
> cpp#n393
> 
> But the problem with the wrongly linked grouped image IDs is also not fixed
> with the garbagecollection branch.
> 
> Maik

Maik,
I just compiled the current state of the garbage collection branch and did the
following:
1) Define 3 images as grouped
2) move the group to a different directory
Result:
The images are now located in the target directory and are still grouped.

My garbage avoidance for moved items works like follows: If items are moved, I
set the album to the target album. Thus, the ImageScanner should not detect the
moved image files as new and thus also should not detect the images as
identical. Moreover, the images entries stay intact and the image relation
grouped should not be touched.

I can not reproduce the problem in the current state of the garbage collection
branch.

Can you both test against this branch?

-- 
You are receiving this mail because:
You are watching all bug changes.

[digikam] [Bug 376020] Moving of photos within the fuzzy search corrupts database

2017-02-07 Thread Mario Frank
https://bugs.kde.org/show_bug.cgi?id=376020

Mario Frank <mario.fr...@uni-potsdam.de> changed:

   What|Removed |Added

 CC||mario.fr...@uni-potsdam.de

--- Comment #2 from Mario Frank <mario.fr...@uni-potsdam.de> ---
Paul,
I second Maik. If you would move the image into the other folder, the image
would really be moved and vanish from source folder. Grouping is just an image
relation that is established.

This seems to me as a feature request in the way: Add option to group an image
to another image and move it to the same folder.

Is this what you expect?

-- 
You are receiving this mail because:
You are watching all bug changes.

[digikam] [Bug 375573] Don't reset/destroy context after deleting one image among a set of duplicates

2017-02-04 Thread Mario Frank
https://bugs.kde.org/show_bug.cgi?id=375573

Mario Frank <mario.fr...@uni-potsdam.de> changed:

   What|Removed |Added

  Latest Commit||https://commits.kde.org/dig
   ||ikam/7ceca1f172828e48b47c50
   ||88b61b2452b7820e52
   Version Fixed In||5.5.0
 Status|UNCONFIRMED |RESOLVED
 Resolution|--- |FIXED

--- Comment #5 from Mario Frank <mario.fr...@uni-potsdam.de> ---
Git commit 7ceca1f172828e48b47c5088b61b2452b7820e52 by Mario Frank.
Committed on 04/02/2017 at 15:47.
Pushed by mfrank into branch 'master'.

We do not rescan for duplicates if an image is deleted any more.
Instead, all duplicates albums in left pane are updated, i.e. the items count,
and average similarity are recalculated. If only one duplicate is left,
the duplicates album is hidden. This solves the problem of losing context
due to the rebuild of the SAlbums. I see no other good technical
possibility of preserving the context since the SAlbums are deleted
automatically.
Also, the similarities to images are not deleted any more. Otherwise the
calculation
of the average similarity would be wrong. We will take care of the similarity
values
in garbage collection branch.
FIXED-IN: 5.5.0

M  +2-1NEWS
M  +9-20   libs/album/albummanager.cpp
M  +1-1libs/album/albummanager.h
M  +17   -0libs/database/item/imageinfo.cpp
M  +5-0libs/database/item/imageinfo.h
M  +18   -0utilities/fuzzysearch/findduplicatesalbum.cpp
M  +4-0utilities/fuzzysearch/findduplicatesalbum.h
M  +65   -16   utilities/fuzzysearch/findduplicatesalbumitem.cpp
M  +10   -0utilities/fuzzysearch/findduplicatesalbumitem.h
M  +4-13   utilities/fuzzysearch/findduplicatesview.cpp
M  +1-1utilities/fuzzysearch/findduplicatesview.h

https://commits.kde.org/digikam/7ceca1f172828e48b47c5088b61b2452b7820e52

-- 
You are receiving this mail because:
You are watching all bug changes.

[digikam] [Bug 374591] Deleting image only removes the file and sets the status to hidden but does not delete the image from DB [patch]

2017-02-04 Thread Mario Frank
https://bugs.kde.org/show_bug.cgi?id=374591

--- Comment #12 from Mario Frank <mario.fr...@uni-potsdam.de> ---
(In reply to Kusi from comment #9)
> do you have by any chance a branch on a git repo to check out from? I use an
> external mysql server and could test it with that configuration
> 
> When you delete a tagged photo and restart digikam, the tags are gone
> forever, right? It wouldn't be possible to restore the photo including tags
> from the trash. What about a retention period (e.g 30 days), or if the
> deletion date is not saved, just clean up once every month instead of every
> launch of digikam(just as an idea)

Kusi,
I never would automatically delete items from trash. I think this is bad
usability. I only delete the items that were either removed directly (not moved
to trash) or removed from trash. So, as long as you do not delete the items
from trash, I will not do this either.

-- 
You are receiving this mail because:
You are watching all bug changes.

[digikam] [Bug 375945] Face detection not scanning my images

2017-02-03 Thread Mario Frank
https://bugs.kde.org/show_bug.cgi?id=375945

Mario Frank <mario.fr...@uni-potsdam.de> changed:

   What|Removed |Added

 CC||mario.fr...@uni-potsdam.de

--- Comment #3 from Mario Frank <mario.fr...@uni-potsdam.de> ---
(In reply to Struan Lownie from comment #2)
> Created attachment 103793 [details]
> Debug as requested.
> 
> Debug log starts when DigiKam had finished loading, and saved once the
> Facedetection reported itself to be finished (5 seconds after start)
> 
> I was running Facedetection against a folder containing 8.48GB of NEF files,
> 609 images

Acording to your log, it seems that you chose detect and recognise as option.
I'm concluding this from:

"...
001013.10399723 [9788] digikam.facesengine: Face database ready
for use 
001113.12266350 [9788] digikam.general: Face PipeLine: add
preview thread   
>0012   13.12278080 [9788] digikam.general: Face PipeLine: add 
>single thread detector   
>0013   13.12292194 [9788] digikam.general: Face PipeLine: add 
>recognition worker   
001413.12304497 [9788] digikam.general: Face PipeLine: add
database writer  
..."

This problem was already solved in this bug:
https://bugs.kde.org/show_bug.cgi?id=375375

So, this is a known bug in 5.4 which is fixed for the upcoming release 5.5, I
woud say.

Seems to me like a duplicate, Gilles.

Struan,
can you test with the current 5.5 pre-release for Windows which is located
here:

https://drive.google.com/drive/folders/0BzeiVr-byqt5Y0tIRWVWelRJenM

Should be fixed.

-- 
You are receiving this mail because:
You are watching all bug changes.

[digikam] [Bug 375700] Digikam freezes for a long time (I/O wait) if similarity range is set too low

2017-02-03 Thread Mario Frank
https://bugs.kde.org/show_bug.cgi?id=375700

Mario Frank <mario.fr...@uni-potsdam.de> changed:

   What|Removed |Added

  Latest Commit||https://commits.kde.org/dig
   ||ikam/574a76623105b08dc4f1f6
   ||2ccb8cc6bd822bae0d
   Version Fixed In||5.5.0
 Status|UNCONFIRMED |RESOLVED
 Resolution|--- |FIXED

--- Comment #4 from Mario Frank <mario.fr...@uni-potsdam.de> ---
Git commit 574a76623105b08dc4f1f62ccb8cc6bd822bae0d by Mario Frank.
Committed on 03/02/2017 at 13:44.
Pushed by mfrank into branch 'master'.

Introduced a configurable lower bound for the minimum similarity in fuzzy and
duplicates search.
This reduces the probability of unresponsiveness due too many images being
scanned for similarity.
The bound can be configured in setup->misc and takes effect immediately in
fuzzy search view and
duplicates view. Thus, no restart of digiKam is necessary.
FIXED-IN: 5.5.0

M  +3-1NEWS
M  +2-0libs/settings/applicationsettings.cpp
M  +3-0libs/settings/applicationsettings.h
M  +10   -0libs/settings/applicationsettings_miscs.cpp
M  +3-0libs/settings/applicationsettings_p.cpp
M  +2-0libs/settings/applicationsettings_p.h
M  +21   -1utilities/fuzzysearch/findduplicatesview.cpp
M  +1-0utilities/fuzzysearch/findduplicatesview.h
M  +22   -2utilities/fuzzysearch/fuzzysearchview.cpp
M  +2-0utilities/fuzzysearch/fuzzysearchview.h
M  +11   -1utilities/maintenance/maintenancedlg.cpp
M  +24   -1utilities/setup/setupmisc.cpp

https://commits.kde.org/digikam/574a76623105b08dc4f1f62ccb8cc6bd822bae0d

-- 
You are receiving this mail because:
You are watching all bug changes.

[digikam] [Bug 375700] Digikam freezes for a long time (I/O wait) if similarity range is set too low

2017-02-03 Thread Mario Frank
https://bugs.kde.org/show_bug.cgi?id=375700

Mario Frank <mario.fr...@uni-potsdam.de> changed:

   What|Removed |Added

 CC||mario.fr...@uni-potsdam.de

--- Comment #3 from Mario Frank <mario.fr...@uni-potsdam.de> ---
Hi Jens,

This is a known problem. If the similarity range is too big or the minimum
similarity too low, the process takes much time. And the application becomes
unresponsive since the image signatures are located in the core DB currently.
Thus, everything that wants to access the core DB is slowed down extremely.

We will have to migrate everything concerning fuzzy search and duplicates
search into a dedicated database. This would give us the opportunity to make
searches in background easier.

The search result you see is just a virtual album which is loaded every time
you visit it. Thus, we cannot do it in background currently. 

Limiting the search by time is formally no good approach, I think. If we limit
the time, we can only scan a limited amount of images. The result would be that
images that would be scanned later and have a higher similarity would not be
shown. Just because of timing constraints.

I would now re-introduce a lower similarity bound for fuzzy and similarity
searches. This bound was 40 %. And frankly said, I would not use fuzzy search
with similarities lower than 40 % myself. The probability that images have such
a low similarity is quite high.

-- 
You are receiving this mail because:
You are watching all bug changes.

[digikam] [Bug 374591] Deleting image only removes the file and sets the status to hidden but does not delete the image from DB [patch]

2017-02-02 Thread Mario Frank
https://bugs.kde.org/show_bug.cgi?id=374591

--- Comment #8 from Mario Frank <mario.fr...@uni-potsdam.de> ---
Created attachment 103773
  --> https://bugs.kde.org/attachment.cgi?id=103773=edit
Patch including new files.

-- 
You are receiving this mail because:
You are watching all bug changes.

[digikam] [Bug 317210] THUMBDB : delete image removes file, but does not remove thumbnails

2017-02-02 Thread Mario Frank
https://bugs.kde.org/show_bug.cgi?id=317210

Mario Frank <mario.fr...@uni-potsdam.de> changed:

   What|Removed |Added

 CC||mario.fr...@uni-potsdam.de

--- Comment #37 from Mario Frank <mario.fr...@uni-potsdam.de> ---
Hi, I just uploaded a patch in https://bugs.kde.org/show_bug.cgi?id=374591 that
introduces garbage collection as maintenance stage. This cleans the core,
thumbs and recognition db from stale entries. But be advised: The patch is
still in testing phase. I did some tests on my own databases and had no
problems. But just to be sure, backup your databases before you test.

-- 
You are receiving this mail because:
You are watching all bug changes.

[digikam] [Bug 375703] Moving grouped images into another album removes groups

2017-02-02 Thread Mario Frank
https://bugs.kde.org/show_bug.cgi?id=375703

Mario Frank <mario.fr...@uni-potsdam.de> changed:

   What|Removed |Added

 CC||mario.fr...@uni-potsdam.de

--- Comment #3 from Mario Frank <mario.fr...@uni-potsdam.de> ---
Hi Jens, I just uploaded a patch in https://bugs.kde.org/show_bug.cgi?id=374591
that, apart from garbage collection also leads to generating less junk.
Meaning, if files are moved/renamed, the old images table entries are reused.
This way, the image id of the image is preserved. This could solve your
problem.
This patch does not work for images that were already moved before applying the
patch. You can test it if you compile from source and apply the patch.
But be advised: The patch is in testing phase. If you do not run the garbage
collector in maintenance, nothing bad should happen to your databases. But just
to be sure, backup your databases before you test.

-- 
You are receiving this mail because:
You are watching all bug changes.

[digikam] [Bug 323718] THUMBDB : rebuild all thumbnails does not get rid of all thumbnails first

2017-02-02 Thread Mario Frank
https://bugs.kde.org/show_bug.cgi?id=323718

Mario Frank <mario.fr...@uni-potsdam.de> changed:

   What|Removed |Added

 CC||mario.fr...@uni-potsdam.de

--- Comment #14 from Mario Frank <mario.fr...@uni-potsdam.de> ---
There is a patch that introduces garbage collection as maintenance stage before
thumbnail rebuild here: https://bugs.kde.org/show_bug.cgi?id=374591 .
This could solve your problem. But be advised: The patch is still in testing
phase. So, backup your databases before you test.

-- 
You are receiving this mail because:
You are watching all bug changes.

[digikam] [Bug 374591] Deleting image only removes the file and sets the status to hidden but does not delete the image from DB

2017-02-02 Thread Mario Frank
https://bugs.kde.org/show_bug.cgi?id=374591

--- Comment #7 from Mario Frank <mario.fr...@uni-potsdam.de> ---
To all who test this patch: Please backup your databases before you test.
Though I made tests without any problems, your databases differ from mine. And
I do not want you to lose data. The core database is only slightly touched,
i.e. really removed image entries are deleted. The most work is done in
thumbnails db.

-- 
You are receiving this mail because:
You are watching all bug changes.

[digikam] [Bug 374591] Deleting image only removes the file and sets the status to hidden but does not delete the image from DB

2017-02-02 Thread Mario Frank
https://bugs.kde.org/show_bug.cgi?id=374591

--- Comment #6 from Mario Frank <mario.fr...@uni-potsdam.de> ---
Created attachment 103766
  --> https://bugs.kde.org/attachment.cgi?id=103766=edit
This patch introduces garbage collection as maintenance stage and reduces the
amount of generated garbage.

This patch introduces garbage collection as maintenance stage and reduces the
amount of generated garbage.
The stage in which the garbage collector is run is before the rebuilding of
thumbnails.

Description of problems and approach.

The garbage that bloated my databases was quite annoying. I want to draw
a sketch about where the garbage comes from:

1) Move image to trash. Everytime I delete some image, the Images table
entry is set to status Removed (3) and
the original album id is removed from the entry.
If I restore the image from trash, a new Images entry is generated.

2) Deleting images/albums directly or from trash. The image files are
deleted from hard drive but the Images entries
are not removed (and also ImageTagProperties and so on.

3) Moving/renaming images creates a new Images entry. The old one is set
to status Removed.

4) Deleting images does not remove the thumbnails. (by path/ uniqueHash)

5) Deleting face regions does not remove the region thumbnails (custom
identifier)

6) Removing tag does not remove identities from recognition db (every
identity should have the same faceEngineUUID as a tag)

Here is a description about what the patch does:
1) Creating less junk:
I introduced a new item status "Obsolete" and renamed the status
"Removed" to "Trashed".
Items are set to status Trashed if they are moved to trash. If items are
deleted directly/permanently,
they get the status "Obsolete".

If an image is restored from trash, i search for an item entry that has
status Trashed and has the same properties
as the new one. If i find such one, I use this entry and set the new/old
album and the status to visible.

If an image is renamed/moved, I use the moveItem method of the core DB
to set the new album/name of the image.
This way, the ImageScanner does not think that this is a new image. The
old entry is reused.
This could solve the grouping problem.

I cannot solve points 4 to 6 in the same easy way, explicitely not the
thumbnails problem since thumbnails can
be referenced by image path, image uniqueHash/file size and image
path/face region (custom identifier)

Thus, I made some clean-up routine for our databases.

2) Collecting junk:
I implemented the DbCleaner Maintenance module. It runs at every start
of digikam (if configured so in setup->misc) and removes
all stale Images entries (detectable by status Obsolete). This does not
take much time. But the DbCleaner can do more.
In Maintenance dialog, I added a stage Database Cleanup, which can be
triggered. The stage can also clean the thumbs db
and recognition db. But this must be explicitely selected in the menu as
this can take more time. Also, the thumbs and
recognition db are never cleaned at the start of digikam. I do not want
our users to wait minutes until they can work.

Now to what the DbCleaner does. As already said, it removes the stale
images. But let's take a step back. Getting the
stale images is just one call to core db. Getting the stale thumbnails
is more complicated. Getting the stale face identities
is less complicated. In first phase of the DbCleaner I analyse the
databases (thumbs and recognition only if enabled).
Identities are stale if there is no tag in core db that has the same
faceEngineUUID as the identity.
Thumbnails are stale if the following holds:
1) There is no image in core DB whose file path leads to that
thumbnail (FilePaths table)
2) There is no image in core DB whose uniqueHash and file size leads
to that thumbnail (UniqueHashes table)
3) There is no face region of an image whose custom identifier
(image file path + region) leads to that thumbnail (CustomIdentifiers table)

So I first get all thumbnail ids from thumbs db into a list A and all
image ids into another list B.
Then I get the thumbnail ids for every image by their file path,
uniqueHash/file size and also
the thumbnails for the face regions and remove those thumbnail ids from
my list A.
The remainder in the list is thus neither connected to an image/video
nor to face regions.
I know that this is no really efficient way. But if, let's say a face
region is deleted, I cannot delete the thumbnail since it could
still be used from some other image by file path for example.

When I am done with that, I first clean the core db (stale images) After
that, I clean the thumbs db and after that the
recognition db. So far for the main process. The progress is shown to
the user and I show, what currently is done (analyse,
clean core DB, clean thumbs DB, clean recognition DB).

Then I tested my implementation with my database. I have got about 4
images and my thumbnail db contains
96000 File

[digikam] [Bug 374591] Deleting image only removes the file and sets the status to hidden but does not delete the image from DB

2017-02-02 Thread Mario Frank
https://bugs.kde.org/show_bug.cgi?id=374591

Mario Frank <mario.fr...@uni-potsdam.de> changed:

   What|Removed |Added

Version|5.4.0   |5.5.0

-- 
You are receiving this mail because:
You are watching all bug changes.

[digikam] [Bug 375573] Don't reset/destroy context after deleting one image among a set of duplicates

2017-01-26 Thread Mario Frank
https://bugs.kde.org/show_bug.cgi?id=375573

--- Comment #3 from Mario Frank <mario.fr...@uni-potsdam.de> ---
Hey Dan,

I will answer inline since there are some things that came me in mind.

(In reply to Dan Dascalescu from comment #2)
> Hey Mario,
> 
> Thank you for the explanation. I understand the tradeoff - accuracy in
> reporting the number of dupes, vs. speedy processing. The solution I propose
> revolved around lazy calculation - does the user care more about a precise
> number shown next to the album *when they get to see it*, or to be able to
> move on to examine the other duplicates in the cluster?

I would expect the latter to be more important than the accuracy. Thus,
delaying is an option for me.

> 
> I mentioned "when they get to see it" because after the user deletes one of
> the duplicates, the list of duplicate clusters in the left pane always
> scrolls to the top (IMO this could be improved to try to keep the scroll
> position, but digiKam probably just re-sorts the list), so if they were
> working on a duplicate cluster below the fold (i.e. if they have scrolled
> down at all), the number of duplicates in that album won't be visible
> anyway. In fact, when you deal with many clusters of duplicates, only those
> items at the top, according to the sort order (Ref. images filename, # of
> items, or Avg. similarity) will be visible.

Okay, let's switch to your terminus. With duplicates albums, we refer to
what you call duplicates clusters (internally called search albums), i.e.
the entries in the left table - one duplicates album is one entry here.
Scrolling to the top is really annoying. This could be resolved. 
But I will come to that later.

> 
> Not sure what you meant by "one duplicates album" (needs to be adjusted) -
> did you mean a cluster (in DUFF terminology, http://duff.dreda.org/) of
> duplicates (which may be spread across different albums), or an album that
> contains duplicates, so the count of items in the album needs to be
> adjusted? In the latter case, that count is even farther from the user's
> attention, because the user is in the Fuzzy tab, vs. in the Albums tab.
> Could the recalculation of counts be done only once, when the user leaves
> the Fuzzy tab?
> 
> Also, there are two different scenarios I see when it comes to deleting
> duplicates:
> 
> 1) Deleting images in duplicate clusters one by one, while the user looks at
> the picture in Preview Mode, to examine it in as large of a size as
> possible. In this case, only one image is deleted at a time. Would counts be
> easier to decrement in this case?

Yes, this was my first approach when I tried to fix the referenced bug.
But the fact that the image should also vanish from other duplicates clusters
would have forced me to decrement there, too. But the count of images is
defined
in the internal search albums in the way that the count is the count of image
ids.
And the cluster list does not know how many of the images are existent.
Nevertheless, it is technically possible to get the cluster list to know which
images
still exist and which do not. But then again, the average similarity is not
correct
anymore as it is calculated on the complete set of images.
This could be also solved by the fact that I introduced the similarities
between images
in database shortly before release of 5.4.

> 
> 2) Staying in Thumbnails or Table, selecting multiple images, and deleting
> them at once.
> 
> Finally, question about "the deleted image may be member of other duplicates
> albums" (this relates to the cluster vs. album distinction) - is the
> duplicate relationship transitive? I mean, if images A and B are dupes
> within the similarity range, and B is part of another cluster of duplicates,
> A should be part of that cluster too, which means only two counts need to be
> updates: the number of dupes in that cluster, and the number of items in the
> album the image belongs to.

Theoretically, you are right. If image A is a duplicate of reference images
B and C, the images B and C have *some* similarity, too. But as in audio
streams -
if stream a is part of stream b and c, the latter streams have *some*
similarity
in *some* position. Perhaps the similar parts are only 2 %. Depending on the
given
similarity range, this similarity is ignored. We cannot use transitive closures
here.

So, to roll up.
If we have duplicates cluster A and we delete some image that is also part of
duplicates
cluster B, we need to update both clusters - in some way:
rescanning/decrementing counts.
If we delete the reference image of cluster A itself, the cluster would
currently vanish.
As consequence, the internal search album is removed and you lose context. This
is a problem
which was not addressed in the referenced bug. And this is a real disturbance
in the workflow.

I 

[digikam] [Bug 375573] Don't reset/destroy context after deleting one image among a set of duplicates

2017-01-26 Thread Mario Frank
https://bugs.kde.org/show_bug.cgi?id=375573

Mario Frank <mario.fr...@uni-potsdam.de> changed:

   What|Removed |Added

 CC||mario.fr...@uni-potsdam.de

--- Comment #1 from Mario Frank <mario.fr...@uni-potsdam.de> ---
Hey Dan,

there was a bug before 5.4 with a quite long discussion (
https://bugs.kde.org/show_bug.cgi?id=261417 ). To make it short:
When some image from a duplicates album is deleted, the count of duplicates for
this album has to be adjusted. Otherwise, we provide wrong information. Also,
the deleted image may be member of other duplicates albums. Thus, they have to
be adjusted, too. Some of the albums may even vanish if this was the only
duplicate to the reference image.

Following this, I took the most performant approach: all duplicates albums that
contained the image are rescanned for duplicates and followingly refreshed.
This may take some time depending on images involved. During this time, the
image view loses the connection to the duplicates album since it is not present
during rescan but only afterwards.

So, what you experience is the lost connection.

I agree that the workflow is interrupted in this case. If only one duplicates
album needs to be adjusted, trying to just decrement the image count would be
feasible. But as soon as another duplicates album becomes dirty by the
deletion,
a rescan should be definitely done, I think.
Delaying the rescan would technically be possible. the problem here is that we
cannot estimate the usual time a user should have until a rescan is done.
If a duplicates album has 100 items and you delete one image per second, the
delay is okay. But 10 seconds delay, for example may again interrupt the
workflow of users.

Any comments/opinions to this?

-- 
You are receiving this mail because:
You are watching all bug changes.

  1   2   3   >