Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 6.3
    • Component/s: Colour, Imaging
    • Labels:
      None
    • Environment:
      any

      Description

      A client sent us a file that was a bit slow to render. After closer inspection there were two main hot spots.

      The first being the application of DeviceCMYK using ICC colour profiles. In this particular case the CMYK converter code had excellent colour match and was a lot faster then using the ICC colour profile.

      The second issue was the use of fairly large image bands that used the calRGB colour space. This particular colour space is decoded by the RawDecoder and is quite slow.

        Activity

        Patrick Corless created issue -
        Patrick Corless made changes -
        Field Original Value New Value
        Fix Version/s 6.3 [ 13093 ]
        Hide
        Patrick Corless added a comment -

        I've added raster building code to the RasterDecode class. This significantly speeds up the decode of the PDF. When both approaches are used we are seeing a 250% reduction in render time.

        Show
        Patrick Corless added a comment - I've added raster building code to the RasterDecode class. This significantly speeds up the decode of the PDF. When both approaches are used we are seeing a 250% reduction in render time.
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #52104 Thu Nov 23 10:40:24 MST 2017 patrick.corless PDF-1219 addition of catch for calRGB colour spaces to RasterDecode.
        Files Changed
        Commit graph MODIFY /icepdf/trunk/icepdf/core/core-awt/src/main/java/org/icepdf/core/pobjects/graphics/images/ImageUtility.java
        Hide
        Patrick Corless added a comment -

        Appears to have passed QA, marking as fixed.

        Show
        Patrick Corless added a comment - Appears to have passed QA, 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:
            Patrick Corless
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: