Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add handling of jpx image decoding #645

Closed
wants to merge 7 commits into from

Conversation

fgregg
Copy link

@fgregg fgregg commented Jul 19, 2021

Pull request

This PR adds handling for the JPXDecode filter, as suggested in #163 (comment). This allows extraction of images from pdfs that use this filter

Will fix #163

How Has This Been Tested?

Tested locally. Images extracted correctly. Unfortunately, I do not have non-proprietary pdfs to share.

Checklist

  • I have added tests that prove my fix is effective or that my feature
    works
  • I have added docstrings to newly created methods and classes
  • I have optimized the code at least one time after creating the initial
    version
  • I have updated the README.md or I am verified that this
    is not necessary
  • I have updated the readthedocs documentation or I
    verified that this is not necessary
  • I have added a consice human-readable description of the change to
    CHANGELOG.md

@jangxyz
Copy link

jangxyz commented Sep 5, 2021

Any updates on this? Reached to the exact same fix as this PR. Is there any specific reason why this PR is on hold?

@pietermarsman pietermarsman changed the title handle jpx decoding Add handling of jpx image decoding Jan 23, 2022
@pietermarsman
Copy link
Member

PIL is used without it being in the requirements. Created #701 for this.

@pietermarsman
Copy link
Member

Had to merge with command line due to conflict: 708dd20

@pietermarsman
Copy link
Member

@fgregg thanks for the work. @jangxyz Thanks for the review.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

JPXDecode error
3 participants