ICEpdf
  1. ICEpdf
  2. PDF-306

YCCK2BGRA colour conversion is too dark.

    Details

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

      Description

      This PDF came in from a client. The file in question has several jpeg images that show up too dark. At first look it appears the fudge we use to converst a YCCK colour space to RGB is a little off and produces an image that is too dark. I have to do a little more investigation but we should be able to fix this.

        Activity

        Patrick Corless created issue -
        Arran Mccullough made changes -
        Field Original Value New Value
        Salesforce Case [5007000000Gu96k]
        Repository Revision Date User Message
        ICEsoft Public SVN Repository #24886 Thu Jun 23 12:24:41 MDT 2011 patrick.corless PDF-306 revisited YCCK2BGRA algorthm to try and lighten up resulting image.
        Files Changed
        Commit graph MODIFY /icepdf/trunk/icepdf/core/src/org/icepdf/core/pobjects/Stream.java
        Hide
        Patrick Corless added a comment -

        Altered the YCCK to BGRA conversion to more closely match the adobe conversion for CCIR Rec. 601-1 standard. I removed the our Y tweak in favour of just leaving Y alone. Still kept to custom mods which make the resulting images a bit blue.

        The image in question is still not as dark as adobe but the correct contrast and images shapes can be seen.

        Show
        Patrick Corless added a comment - Altered the YCCK to BGRA conversion to more closely match the adobe conversion for CCIR Rec. 601-1 standard. I removed the our Y tweak in favour of just leaving Y alone. Still kept to custom mods which make the resulting images a bit blue. The image in question is still not as dark as adobe but the correct contrast and images shapes can be seen.
        Patrick Corless made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Resolution Fixed [ 1 ]
        Ken Fyten 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: