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

              Descending order - Click to sort in ascending order
              Ken Fyten made changes -
              Status Resolved [ 5 ] Closed [ 6 ]
              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
              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
              Patrick Corless made changes -
              Status Open [ 1 ] Resolved [ 5 ]
              Resolution Fixed [ 1 ]
              Hide
              Permalink
              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 -
              Assignee Mark Collette [ mark.collette ] Patrick Corless [ patrick.corless ]
              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 Patrick Corless [ patrick.corless ] Mark Collette [ mark.collette ]
              Patrick Corless made changes -
              Salesforce Case []
              Fix Version/s 4.1 [ 10227 ]
              Patrick Corless made changes -
              Field Original Value New Value
              Attachment Example Ring 0001.pdf [ 12200 ]
              Patrick Corless created issue -

                People

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

                  Dates

                  • Created:
                    Updated:
                    Resolved: