dupeguru icon indicating copy to clipboard operation
dupeguru copied to clipboard

match all orientations

Open brccabral opened this issue 1 year ago • 2 comments

Hi, I was able to solve the problem with rotated/mirrored images. Had to add cache blocks for all 8 orientation in the sqlite, but it is working fine. Let me know if you need any edit. Cheers, Bruno

brccabral avatar May 14 '23 01:05 brccabral

I think the general idea here is good. I do think this should be an option instead of always matching all orientations as some users may not want to match images with different orientations.

arsenetar avatar Jun 08 '23 23:06 arsenetar

I think the general idea here is good. I do think this should be an option instead of always matching all orientations as some users may not want to match images with different orientations.

Hi @arsenetar , added option for rotated images

brccabral avatar Jul 04 '23 00:07 brccabral

This change massively slows down the scan, even when the "match all orientations" option is not enabled. The calculation of the blocks for all orientations is always done, predictably slowing down the scan by a factor of 8? or so.

A close to 10x slowdown is too much. I reverted this change on my local copy.

ne1188 avatar May 01 '24 04:05 ne1188

@ne1188 thanks for reporting, please open an issue ticket referencing this PR

glubsy avatar May 04 '24 16:05 glubsy