Bug 420932 - TIFF images with display resolution stored in pels/cm is displayed as ppi in image/properties panel
Summary: TIFF images with display resolution stored in pels/cm is displayed as ppi in ...
Status: RESOLVED FIXED
Alias: None
Product: krita
Classification: Applications
Component: File formats (other bugs)
Version First Reported In: git master (please specify the git hash!)
Platform: Other Linux
: NOR minor
Target Milestone: ---
Assignee: amyspark
URL:
Keywords: triaged
Depends on:
Blocks:
 
Reported: 2020-05-02 23:01 UTC by Aaron Boxer
Modified: 2022-11-29 15:36 UTC (History)
2 users (show)

See Also:
Latest Commit:
Version Fixed In:
Sentry Crash Report:


Attachments
TIFF image with pixels/cm stored (93.91 KB, image/tiff)
2020-05-02 23:01 UTC, Aaron Boxer
Details
Image with resolution stored in pixels/cm (84.70 KB, image/tiff)
2020-05-04 12:40 UTC, Aaron Boxer
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Aaron Boxer 2020-05-02 23:01:26 UTC
Created attachment 128104 [details]
TIFF image with pixels/cm stored

1. run tiffinfo on attached image - resolution stored as 118 pixels / cm

2. Open attached image, and then open  image / properties

resolution is displayed as 118 pixels / inch
Comment 1 Halla Rempt 2020-05-04 12:27:40 UTC
That is strange, because both ImageMagick and Krita show me a resolution of 300x300 dpi when I open the attached image.
Comment 2 Aaron Boxer 2020-05-04 12:30:42 UTC
118 ppcm is 300 ppi.  I think the problem is just that properties displays the 118
Comment 3 Aaron Boxer 2020-05-04 12:32:06 UTC
oops, I think I upload the wrong image.
Comment 4 Aaron Boxer 2020-05-04 12:40:46 UTC
Created attachment 128134 [details]
Image with resolution stored in pixels/cm

Sorry, the last image had resolution stored in ppi, this one is in
ppcm. But Krita still displays ppi.
Comment 5 Halla Rempt 2020-05-04 12:55:41 UTC
Ah, that explains.
Comment 6 Bug Janitor Service 2022-11-29 01:46:29 UTC
A possibly relevant merge request was started @ https://invent.kde.org/graphics/krita/-/merge_requests/1662
Comment 7 amyspark 2022-11-29 15:36:02 UTC
Git commit 08e3e2356a055b1f3e9da583a395226bdf1249f0 by L. E. Segovia.
Committed on 29/11/2022 at 15:35.
Pushed by lsegovia into branch 'krita/5.1'.

tiff: Implement detection of resolution unit
Part-of: <https://invent.kde.org/graphics/krita/-/merge_requests/1662>
(cherry picked from commit c8d869554b31175b2fe368383955b3a97964e85b)

M  +4    -0    plugins/impex/tiff/kis_tiff_export.cc
M  +20   -5    plugins/impex/tiff/kis_tiff_import.cc

https://invent.kde.org/graphics/krita/commit/08e3e2356a055b1f3e9da583a395226bdf1249f0