Bug 313190

Summary:	Thumbnails of faces not rotated correctly according to the configuration settings
Product:	[Applications] digikam	Reporter:	Jean-Martial NDOUTOUME NFENGONE <jean-martial>
Component:	Faces-Engine	Assignee:	Digikam Developers <digikam-bugs-null>
Status:	RESOLVED FIXED
Severity:	normal	CC:	trans9006, veaceslav.munteanu90
Priority:	NOR
Version First Reported In:	2.9.0
Target Milestone:	---
Platform:	Microsoft Windows
OS:	Microsoft Windows
Latest Commit:	http://commits.kde.org/digikam/57c3b7249e9b9b1e20c4da13d3a013dfda93ddbb	Version Fixed/Implemented In:	3.2.0
Sentry Crash Report:
Attachments:	The original file view The original file viewed in digiKam The image with orientation tag applied The image with orientation tag applied and face area definition The bug: the wrong oriented face thumbnail

Description Jean-Martial NDOUTOUME NFENGONE 2013-01-13 15:57:23 UTC

The setting "Configure DigiKam -> Metadata -> Rotation -> Show image/thumbnails rotated according to the orientation flag." don't apply to the thumbnails of faces' people for images only with an orientation flag and pixel data not changed.

Reproducible: Always

Steps to Reproduce:
For example:

1. Take an image with only an orientation flag: due to the "Configure DigiKam -> Metadata -> Rotation -> Show image/thumbnails rotated according to the orientation flag." setting, the image and thumbnails are correctly rotated (even if the pixel data of the image file are oriented differently);

2. Tag a face.
Actual Results:  
The face thumbnail is oriented as the pixel data file and not as the orientation flag.

Expected Results:  
As the image and thumbnail, the face's thumbnail must be rotated according to the orientation flag.

Comment 1 Marcel Wiesweg 2013-01-13 19:27:23 UTC

I dont understand the problem. Do you change the mentioned setting? Which thumbnails are shown incorrectly where and when?

Comment 2 Jean-Martial NDOUTOUME NFENGONE 2013-01-13 22:15:56 UTC

Created attachment 76448 [details]
The original file view

Comment 3 Jean-Martial NDOUTOUME NFENGONE 2013-01-13 22:17:50 UTC

Created attachment 76449 [details]
The original file viewed in digiKam

Comment 4 Jean-Martial NDOUTOUME NFENGONE 2013-01-13 22:18:51 UTC

Created attachment 76450 [details]
The image with orientation tag applied

Comment 5 Jean-Martial NDOUTOUME NFENGONE 2013-01-13 22:20:10 UTC

Created attachment 76451 [details]
The image with orientation tag applied and face area definition

Comment 6 Jean-Martial NDOUTOUME NFENGONE 2013-01-13 22:21:11 UTC

Created attachment 76452 [details]
The bug: the wrong oriented face thumbnail

Comment 7 Jean-Martial NDOUTOUME NFENGONE 2013-01-13 22:29:33 UTC

Marcel,

(In reply to comment #1)
> I dont understand the problem. Do you change the mentioned setting?

Yes, the setting is set.

> Which thumbnails are shown incorrectly where and when?

Screenshots:
    1. The original file view (attachment #76448 [details]);
    2. The original file viewed in digiKam (attachment #76449 [details]);
    3. The image with orientation tag applied  (attachment #76450 [details]);
    4. The image with orientation tag applied and face area definition (attachment #76451 [details]);
    5. The bug: the wrong oriented face thumbnail  (attachment #76452 [details]).

Jean-Martial

Comment 8 Jean-Martial NDOUTOUME NFENGONE 2013-03-13 15:57:00 UTC

Steel  UNCONFIRMED?

Jean-Martial

Comment 9 Veaceslav Munteanu 2013-03-13 22:08:25 UTC

Hello Jean,

Yes I can confirm this bung. In fact digikam doesn't rotate face tags from database.

I was working on implementing FaceTag write for digikam and came across to the issue that it doesn't update facetags. Now I have implemented rotation for images rotated pixel-by-pixel(not by exif flag).
 
My implementation is still experimental and it's not even in digikam's git.

I'm aware of this one and I will fix this one after stabilizing code that update tags.

Comment 10 Jean-Martial NDOUTOUME NFENGONE 2013-03-14 09:30:35 UTC

Hello Veaceslav,

Thanks for your comment.

So, work in progress…

Jean-Martial

Comment 11 Marcel Wiesweg 2013-03-21 20:20:41 UTC

I can confirm this, it is easily reproducable.
Veaceslav: This is probably not related to your work in the branch, there is no "new" rotation involved, any image with the rotation flag set by the camera exhibits this behavior. I suspect an interaction when rotation-by-database was introduced.

Comment 12 Marcel Wiesweg 2013-04-18 20:09:49 UTC

Git commit 57c3b7249e9b9b1e20c4da13d3a013dfda93ddbb by Marcel Wiesweg.
Committed on 18/04/2013 at 22:07.
Pushed by mwiesweg into branch 'master'.

Do not exif-rotate face thumbnails twice.
Clean up an unused parameter in ThumbnailCreator.
FIXED-IN: 3.2.0

M  +2    -1    NEWS
M  +69   -68   libs/threadimageio/thumbnailcreator.cpp
M  +4    -4    libs/threadimageio/thumbnailcreator.h
M  +2    -1    libs/threadimageio/thumbnailloadthread.cpp

http://commits.kde.org/digikam/57c3b7249e9b9b1e20c4da13d3a013dfda93ddbb