pypdf icon indicating copy to clipboard operation
pypdf copied to clipboard

NotImplementedError: unsupported filter /JBIG2Decode

Open MartinThoma opened this issue 1 year ago • 18 comments

Explanation

I found an example for the /JBIG2Decode filter :-)

Code Example

PDF: https://github.com/py-pdf/pypdf/files/12090692/New.Jersey.Coinbase.staking.securities.charges.2023-0606_Coinbase-Penalty-and-C-D.pdf

from pypdf import PdfReader, __version__

print(f"pypdf=={__version__}")

reader = PdfReader("New.Jersey.Coinbase.staking.securities.charges.2023-0606_Coinbase-Penalty-and-C-D.pdf")

page = reader.pages[0]
for img in page.images:
    print(img.name)

gives

pypdf==3.12.2
Traceback (most recent call last):
  File "/home/moose/Downloads/pyissue/main.py", line 8, in <module>
    for img in page.images:
  File "/home/moose/Github/py-pdf/pypdf/pypdf/_page.py", line 2604, in __iter__
    yield self[i]
          ~~~~^^^
  File "/home/moose/Github/py-pdf/pypdf/pypdf/_page.py", line 2600, in __getitem__
    return self.get_function(lst[index])
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/moose/Github/py-pdf/pypdf/pypdf/_page.py", line 522, in _get_image
    imgd = _xobj_to_image(cast(DictionaryObject, xobjs[id]))
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/moose/Github/py-pdf/pypdf/pypdf/filters.py", line 844, in _xobj_to_image
    data = x_object_obj.get_data()  # type: ignore
           ^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/moose/Github/py-pdf/pypdf/pypdf/generic/_data_structures.py", line 919, in get_data
    decoded._data = decode_stream_data(self)
                    ^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/moose/Github/py-pdf/pypdf/pypdf/filters.py", line 634, in decode_stream_data
    raise NotImplementedError(f"unsupported filter {filter_

MartinThoma avatar Jul 20 '23 16:07 MartinThoma

PDF found in https://github.com/py-pdf/pypdf/issues/1983

MartinThoma avatar Jul 20 '23 16:07 MartinThoma

from #951

Here is pdfminer implementation: https://github.com/pdfminer/pdfminer.six/blob/master/pdfminer/jbig2.py

ItDoesntWorkScan.pdf

pubpub-zz avatar Jul 20 '23 18:07 pubpub-zz

https://github.com/py-pdf/pypdf/issues/2502#issuecomment-1980190505 - here we have another example

MartinThoma avatar Mar 06 '24 21:03 MartinThoma

This might need a general design decision if I am not mistaken: Pillow does not seem to support JBIG2, while our implementation currently assumes that all images can be loaded as PIL.Image.Image (pdfminer.six does not use Pillow for saving images).

AFAIK there only is jbig2dec which would have to be used in a subprocess to get a "good" image format from the JBIG2 image embedded inside the PDF file (after adding the missing bytes, specifically "the JBIG2 file header, end-of-page segments, and end-of-file segment" which are not part of the XObject according to section 7.4.7 of the PDF 2.0 spec), although this might cause issues with masks etc. (jbig2dec itself is subject to APGL-3.0-or-later and with its strong copyleft effect (including SaaS) rather unlikely to become part of Pillow.) The alternative would be to parse the essential aspects like the pixel data from the JBIG2 image ourselves.

stefan6419846 avatar Apr 14 '24 19:04 stefan6419846

Many platforms support a standalone 'jbig2dec' functionality (e.g. on Mac can brew install jbig2dec) - can you farm that functionality out to that routine? I was going to try, but can't seem to get the raw unfiltered image bytes from the page object. Prob my ignorance... (will keep digging)

mdecaro avatar Jun 01 '24 20:06 mdecaro

Many platforms support a standalone 'jbig2dec' functionality (e.g. on Mac can brew install jbig2dec) - can you farm that functionality out to that routine? I was going to try, but can't seem to get the raw unfiltered image bytes from the page object. Prob my ignorance... (will keep digging)

the XObject is a ContentStream. You should be able to access the data with .get_data()

pubpub-zz avatar Jun 01 '24 20:06 pubpub-zz

Given the data, you should still have a look at the PDF specification on the filter (for PDF 2.0/ISO 32000-2:2020, this is section 7.4.7), especially as jbig2dec probably expects the header and footer to be present, which are omitted within PDF files.

stefan6419846 avatar Jun 02 '24 19:06 stefan6419846

I have it working for the test file I have been using. Cludgey bt working!

I only had to modify one file - filters.py.

mdecaro avatar Jun 02 '24 22:06 mdecaro

Changed file type

mdecaro avatar Jun 02 '24 22:06 mdecaro

If you intended to append a file here, then this did not work due to you answering by e-mail. Uploading files usually requires using the GitHub UI itself.

stefan6419846 avatar Jun 03 '24 06:06 stefan6419846

OK.

On Jun 3, 2024, at 2:05 AM, Stefan @.***> wrote:

If you intended to append a file here, then this did not work due to you answering by e-mail. Uploading files usually requires using the GitHub UI itself.

— Reply to this email directly, view it on GitHub https://github.com/py-pdf/pypdf/issues/1989#issuecomment-2144346762, or unsubscribe https://github.com/notifications/unsubscribe-auth/AENJQIHZOC3Q6WGHX5R5XFLZFQBSDAVCNFSM6AAAAAA2RVC3T6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNBUGM2DMNZWGI. You are receiving this because you commented.

mdecaro avatar Jun 03 '24 10:06 mdecaro

This is a quick McGiver fix for the /JBIG2 error. You need to load the 'jbig2dec' utility. On the Mac this is on home-brew. The fix assumes you have apple silicon - the path to the utility is /opt/homebrew/bin/jbig2ec. If you are Intel Mac, it probably is /usr/local/bin/jbig2dec????
filters.py.zip

mdecaro avatar Jun 03 '24 11:06 mdecaro

Relevant class from the code:

class JBIG2Decode:
    @staticmethod
    def decode(
        data: bytes,
        decode_parms: Optional[DictionaryObject] = None,
        **kwargs: Any,
    ) -> bytes:
        # decode_parms is unused here
        pathin = '/var/tmp/tempin.jbig2'
        pathout = '/var/tmp/tempout.jbig2'
        with open(pathin,"wb") as fl:
            fl.write(data)
        process = subprocess.run(['/opt/homebrew/bin/jbig2dec', '-e', '-o', pathout, pathin])
        with open(pathout,'rb') as fl:
            data = fl.read()
        os.unlink(pathin)
        os.unlink(pathout)
        return data

stefan6419846 avatar Jun 03 '24 13:06 stefan6419846

this solution is not valid for windows / linux... Can you try to rehost the code into python natively ?

pubpub-zz avatar Jun 04 '24 05:06 pubpub-zz

This is untested and may cause issues due to opening the same file twice on Windows, but should in general be OS-independent:

class JBIG2Decode:
    @staticmethod
    def decode(
        data: bytes,
        decode_parms: Optional[DictionaryObject] = None,
        **kwargs: Any,
    ) -> bytes:
        # decode_parms is unused here
        with NamedTemporaryFile(suffix=".jbig2") as infile:
            infile.write(data)
            infile.seek(0)
            result = subprocess.run(
                [shutil.which("jbig2dec"), "--embedded", "--output", "-", infile],
                stdout=subprocess.PIPE
            )
        return result.stdout

stefan6419846 avatar Jun 04 '24 08:06 stefan6419846

The utility jbig2dec is available for other platforms.

I am looking into porting the decompressor to python. There are about 20 separate source files to convert. I’ll work on it but it will take time!!

On Jun 4, 2024, at 1:26 AM, pubpub-zz @.***> wrote:

this solution is not valid for windows / linux... Can you try to rehost the code into python natively ?

— Reply to this email directly, view it on GitHub https://github.com/py-pdf/pypdf/issues/1989#issuecomment-2146638742, or unsubscribe https://github.com/notifications/unsubscribe-auth/AENJQIFALDEG4OZZ4PBVJMLZFVFZRAVCNFSM6AAAAAA2RVC3T6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNBWGYZTQNZUGI. You are receiving this because you commented.

mdecaro avatar Jun 04 '24 11:06 mdecaro

I experimented with this to some extent previously. The version of Jbig2 that I used does not pipe output to standard out, need a file.Sent from my iPadOn Jun 4, 2024, at 7:13 AM, Matthew DeCaro @.> wrote:The utility jbig2dec is available for other platforms.  I am looking into porting the decompressor to python.  There are about 20 separate source files to convert.  I’ll work on it but it will take time!!On Jun 4, 2024, at 1:26 AM, pubpub-zz @.> wrote:this solution is not valid for windows / linux... Can you try to rehost the code into python natively ?—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you commented.Message ID: @.***>

mdecaro avatar Jun 05 '24 00:06 mdecaro

Liked your suggestions!!! I'll still try porting to python (at my pace unfortunately).

Here is the updated file. filters.py.zip

mdecaro avatar Jun 05 '24 00:06 mdecaro