Summary: | Incorrect rotation of DNG portrait images in preview | ||
---|---|---|---|
Product: | [Applications] digikam | Reporter: | Sander van der Heijden <smvdheijden> |
Component: | Preview-RAW | Assignee: | Digikam Developers <digikam-bugs-null> |
Status: | RESOLVED FIXED | ||
Severity: | normal | CC: | caulier.gilles, kunst.uber.alles, michal |
Priority: | NOR | ||
Version: | 2.6.0 | ||
Target Milestone: | --- | ||
Platform: | unspecified | ||
OS: | Linux | ||
Latest Commit: | http://commits.kde.org/digikam/1ed6ff7259d16992797f30c966c1bd795e6d553d | Version Fixed In: | 2.8.0 |
Attachments: | Screenshot of incorrect orientation |
Description
Sander van der Heijden
2012-06-19 20:43:42 UTC
Created attachment 71966 [details]
Screenshot of incorrect orientation
Can you share CR2 and DNG file to test here ? Gilles Caulier (In reply to comment #2) > Can you share CR2 and DNG file to test here ? > > Gilles Caulier Hi Gilles, Files to test are shared via following URLs. CR2: http://db.tt/MhKk4Ako DNG: http://db.tt/PqtNbAVA Sander van der Heijden I have same experience with out of the camera DNG from Ricoh GR Digital IV. Thumbnail is displayed in portrait correctly while RAW is not. Pentax PEF raw files are displayed correctly both thumbnail and preview. Canon CR2 raw files are displayed correctly both thumbnail and preview, DNG not, after convertion. CR2 works all right here as well. DNG thumb is ok (based on embedded preview), DNG full import is ok. The DNG's preview is using half-size conversion because the preview is only 36% per dimension (13% of pixels), too small. The half-size converted image is improperly rotated. I need to investigate a bit more. Usually, IIRC, libraw fully handles exif rotation, so I suspect a libraw problem. Git commit 1ed6ff7259d16992797f30c966c1bd795e6d553d by Marcel Wiesweg. Committed on 22/07/2012 at 20:04. Pushed by mwiesweg into branch 'master'. Add metadata to DImg created from QImage-based loading methods to fix finding the correct exif orientation. Since we make use of the database field, we need to compare the database value with the metadata value. Images created with the preview loading methods had no metadata, so a wrong value was calculated. FIXED-IN: 2.8.0 M +2 -1 NEWS M +2 -0 libs/threadimageio/previewtask.cpp http://commits.kde.org/digikam/1ed6ff7259d16992797f30c966c1bd795e6d553d |