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. 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

          Activity

          Stefan Lundmark created issue -
          Stefan Lundmark made changes -
          Field Original Value New Value
          Attachment IcePDF rendering problems.png [ 22433 ]
          Hide
          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
          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
          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
          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
          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
          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
          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
          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: