ICEpdf
  1. ICEpdf
  2. PDF-156

Image colour model not correctly loaded

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 4.0
    • Fix Version/s: 4.1
    • Component/s: Core/Parsing
    • Labels:
      None
    • Environment:
      any

      Description

      The file in question does not load correctly. The JAI jars seem to have not effect. The colour model does not seem to be properly applied to the image in question.

        Activity

        Patrick Corless created issue -
        Patrick Corless made changes -
        Field Original Value New Value
        Attachment Example Ring 0001.pdf [ 12200 ]
        Patrick Corless made changes -
        Salesforce Case []
        Fix Version/s 4.1 [ 10227 ]
        Patrick Corless made changes -
        Assignee Patrick Corless [ patrick.corless ] Mark Collette [ mark.collette ]
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #21951 Mon Jul 19 12:34:52 MDT 2010 patrick.corless PDF-156 - fixed additive colour logic parsing error.
        Files Changed
        Commit graph MODIFY /icepdf/trunk/icepdf/core/src/org/icepdf/core/pobjects/graphics/Separation.java
        Patrick Corless made changes -
        Assignee Mark Collette [ mark.collette ] Patrick Corless [ patrick.corless ]
        Hide
        Patrick Corless added a comment -

        After quite a bit of debugging it turns out that this is a colour space issue and not an image issue. I was chasing the PNG predictor value as a possible issue but then got side tracked and took a look at the ColourSpace.

        The image had a seperation colour space and after reviewing the specification and code we where correctly handling things. For screen rendering we shouldn't be trying to resolve the named colour value and instead rely on the alternative and tint transform. Screen rendering is considered an additive colour model where printing can be subtractive assuming your printing api has control over this sort of thing which we don't in Java.

        By Changing the colour space logic the PDF in question renders correctly. I also have several other examples where the colour is now being calculated rather then being derived by a named and the resulting colour match is greatly improved.

        Show
        Patrick Corless added a comment - After quite a bit of debugging it turns out that this is a colour space issue and not an image issue. I was chasing the PNG predictor value as a possible issue but then got side tracked and took a look at the ColourSpace. The image had a seperation colour space and after reviewing the specification and code we where correctly handling things. For screen rendering we shouldn't be trying to resolve the named colour value and instead rely on the alternative and tint transform. Screen rendering is considered an additive colour model where printing can be subtractive assuming your printing api has control over this sort of thing which we don't in Java. By Changing the colour space logic the PDF in question renders correctly. I also have several other examples where the colour is now being calculated rather then being derived by a named and the resulting colour match is greatly improved.
        Patrick Corless made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #22042 Tue Jul 27 15:01:37 MDT 2010 patrick.corless PDF-156 - tweaked logic to fall back on named colour if alternative was not an instance of deviceCMYK
        Files Changed
        Commit graph MODIFY /icepdf/trunk/icepdf/core/src/org/icepdf/core/pobjects/graphics/Separation.java
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #22048 Wed Jul 28 03:05:17 MDT 2010 patrick.corless PDF-156 - tweaked logic to look for the black<->white colour switch condition, not a great solution be takes care of the corner case.
        Files Changed
        Commit graph MODIFY /icepdf/trunk/icepdf/core/src/org/icepdf/core/pobjects/graphics/Separation.java
        Ken Fyten made changes -
        Status Resolved [ 5 ] Closed [ 6 ]

          People

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

            Dates

            • Created:
              Updated:
              Resolved: