ICEpdf
  1. ICEpdf
  2. PDF-1115

Image rendering in scanned documents

          Details

          • Type: Bug Bug
          • Status: Closed
          • Priority: Major Major
          • Resolution: Fixed
          • Affects Version/s: 6.1.3
          • Fix Version/s: 6.2.4
          • Component/s: Core/Rendering
          • Labels:
            None
          • Environment:
            Windows 7 and 10

            Description

            We are currently in the process of migrating from IcePDF 4.3.2 to 6.1.3 and are experiencing major image scaling quality problems.

            We have attempted to set org.icepdf.core.imageReference=smoothScaled. We have also double checked that this is indeed the scale-type being used by the ImageReferenceFactory-class.

            The rendered image doesn't really look very 'smooth scaled' so I'm not sure what's going on.

            Is there some setting I'm missing?
            1. PDF File
              Journalanteckningar - Edited.pdf
              439 kB
              Stefan Lundmark
            1. 100_percent.png
              31 kB
            2. 200_percent.png
              65 kB
            3. IcePDF rendering problems.png
              41 kB

              Issue Links

              is duplicated by

              Bug - A problem which impairs or prevents the functions of the product. PDF-1143 PDF rendered sometimes with very bad quality despite 'smoothScaled'

              • Major - Major loss of function.
              • Closed

                Activity

                Ascending order - Click to sort in descending order
                Stefan Lundmark created issue -
                Stefan Lundmark made changes -
                Field Original Value New Value
                Attachment IcePDF rendering problems.png [ 22433 ]
                Hide
                Permalink
                Patrick Corless added a comment -

                Could you post a sample file or submit one via support?

                Show
                Patrick Corless added a comment - Could you post a sample file or submit one via support?
                Stefan Lundmark made changes -
                Attachment Journalanteckningar - Edited.pdf [ 22434 ]
                Stefan Lundmark made changes -
                Attachment Journalanteckningar - Edited.pdf [ 22435 ]
                Stefan Lundmark made changes -
                Attachment Journalanteckningar - Edited.pdf [ 22435 ]
                Hide
                Permalink
                Stefan Lundmark added a comment -

                PDF has been uploaded.

                Show
                Stefan Lundmark added a comment - PDF has been uploaded.
                Patrick Corless made changes -
                Fix Version/s 6.2.2 [ 13097 ]
                Hide
                Permalink
                Patrick Corless added a comment -

                Thanks for providing the file, I dug a little deeper and it appears the issue is related not to scaling for say as we more or less do that the same in 6.x but in our old ImageCache class. Files once decode were saved to disk as PNG tmp files. We moved away from this approach as it was fairly costly at the time.

                We'll give this a hard look again as we are in the process of updating our image handling code.

                Show
                Patrick Corless added a comment - Thanks for providing the file, I dug a little deeper and it appears the issue is related not to scaling for say as we more or less do that the same in 6.x but in our old ImageCache class. Files once decode were saved to disk as PNG tmp files. We moved away from this approach as it was fairly costly at the time. We'll give this a hard look again as we are in the process of updating our image handling code.
                Patrick Corless made changes -
                Fix Version/s 6.3 [ 13093 ]
                Fix Version/s 6.2.2 [ 13097 ]
                Hide
                Permalink
                Stefan Lundmark added a comment - - edited

                Are there any news regarding this problem? We are seeing more and more PDF-documents that the old version we are using, is unable to handle. We need this fix as soon as possible.

                Show
                Stefan Lundmark added a comment - - edited Are there any news regarding this problem? We are seeing more and more PDF-documents that the old version we are using, is unable to handle. We need this fix as soon as possible.
                Hide
                Permalink
                Patrick Corless added a comment -

                I've been looking at a blurring filter that takes advantage of the convolution kernel. Java has a build in ConvolveOp which makes for some pretty solid performance. I'll attached the results at 100% and 200% zoom.

                Show
                Patrick Corless added a comment - I've been looking at a blurring filter that takes advantage of the convolution kernel. Java has a build in ConvolveOp which makes for some pretty solid performance. I'll attached the results at 100% and 200% zoom.
                Patrick Corless made changes -
                Attachment 100_percent.png [ 22503 ]
                Attachment 200_percent.png [ 22504 ]
                Patrick Corless made changes -
                Fix Version/s 6.2.4 [ 13099 ]
                Fix Version/s 6.3 [ 13093 ]
                Hide
                Permalink
                Stefan Lundmark added a comment -

                We've received feedback from our users and the results that you've attached are more than acceptable, we would appreciate if you could implement this in an official release as soon as possible.

                Show
                Stefan Lundmark added a comment - We've received feedback from our users and the results that you've attached are more than acceptable, we would appreciate if you could implement this in an official release as soon as possible.
                Patrick Corless made changes -
                Link This issue is duplicated by PDF-1143 [ PDF-1143 ]
                Repository Revision Date User Message
                ICEsoft Public SVN Repository #51509 Wed May 10 15:08:40 MDT 2017 patrick.corless PDF-1115 addition of new image reference type for blurring images which
                actually makes some tiff images more legible at low zoom values.
                Files Changed
                Commit graph MODIFY /icepdf/branches/icepdf-6.2.0/icepdf/core/src/org/icepdf/core/pobjects/graphics/MipMappedImageReference.java
                Commit graph MODIFY /icepdf/branches/icepdf-6.2.0/icepdf/core/src/org/icepdf/core/pobjects/graphics/ImageReferenceFactory.java
                Commit graph ADD /icepdf/branches/icepdf-6.2.0/icepdf/core/src/org/icepdf/core/pobjects/graphics/BlurredImageReference.java
                Repository Revision Date User Message
                ICEsoft Public SVN Repository #51510 Wed May 10 15:09:00 MDT 2017 patrick.corless PDF-1115 addition of new image reference type for blurring images which
                actually makes some tiff images more legible at low zoom values.
                Files Changed
                Commit graph ADD /icepdf/trunk/icepdf/core/core-awt/src/main/java/org/icepdf/core/pobjects/graphics/BlurredImageReference.java
                Commit graph MODIFY /icepdf/trunk/icepdf/core/core-awt/src/main/java/org/icepdf/core/pobjects/graphics/MipMappedImageReference.java
                Commit graph MODIFY /icepdf/trunk/icepdf/core/core-awt/src/main/java/org/icepdf/core/pobjects/graphics/ImageReferenceFactory.java
                Hide
                Permalink
                Patrick Corless added a comment -

                The new system property is -Dorg.icepdf.core.imageReference=blurred and will be part of 6.2.4 out later this week.

                Show
                Patrick Corless added a comment - The new system property is -Dorg.icepdf.core.imageReference=blurred and will be part of 6.2.4 out later this week.
                Hide
                Permalink
                Patrick Corless added a comment -

                Marking as fixed.

                Show
                Patrick Corless added a comment - Marking as fixed.
                Patrick Corless made changes -
                Status Open [ 1 ] Resolved [ 5 ]
                Resolution Fixed [ 1 ]
                Patrick Corless made changes -
                Status Resolved [ 5 ] Closed [ 6 ]

                  People

                  • Assignee:
                    Patrick Corless
                    Reporter:
                    Stefan Lundmark
                  • Votes:
                    0 Vote for this issue
                    Watchers:
                    2 Start watching this issue

                    Dates

                    • Created:
                      Updated:
                      Resolved: