Skip to content

Remove --output-type pdf from PDF OCR processing - #4

Merged
pklaschka merged 1 commit into
mainfrom
fix-ocr-infinite-loop
Sep 1, 2026
Merged

pklaschka merged 1 commit into
mainfrom
fix-ocr-infinite-loop

Conversation

@pklaschka

@pklaschka pklaschka commented Sep 1, 2026

Copy link
Copy Markdown
Member

This ran into a seeming bug in ocrmypdf where for some PowerPoint PDF exports, the OCR didn't succeed and instead ran into a seemingly infinite output of

Recursion depth exceeded in _find_image_xrefs_page                                                     optimize.py:259

This seems to be tracked in ocrmypdf/OCRmyPDF#1321, but for now has to be fixed by a workaround.

This previously resulted in a RAM usage overload of the DMS, presumably because it tried to collect and log the (infinite) output logs of ocrmypdf. For internal discussions on this, cf. messages from "SysAdmin Int" from August 31st and September 1st 2026.

  • main
    • Remove --output-type pdf from PDF OCR processing 👈

This ran into a seeming bug in `ocrmypdf` where for some PowerPoint PDF exports, the OCR didn't succeed and instead ran into a seemingly infinite output of

```txt
Recursion depth exceeded in _find_image_xrefs_page                                                     optimize.py:259
```

This seems to be tracked in ocrmypdf/OCRmyPDF#1321, but for now has to be fixed by a workaround.

This previously resulted in a RAM usage overload of the DMS, presumably because it tried to collect and log the (infinite) output logs of `ocrmypdf`. For internal discussions on this, cf. messages from "SysAdmin Int" from August 31st and September 1st 2026.
@pklaschka
pklaschka enabled auto-merge September 1, 2026 09:45
@pklaschka pklaschka self-assigned this Sep 1, 2026
@pklaschka
pklaschka disabled auto-merge September 1, 2026 09:55
@pklaschka
pklaschka merged commit c5df4f1 into main Sep 1, 2026
2 checks passed
@pklaschka
pklaschka deleted the fix-ocr-infinite-loop branch September 1, 2026 09:55
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.

1 participant