Skip to content

feat: map OCR text-layer skip option#149

Open
drukpa1455 wants to merge 1 commit into
docling-project:mainfrom
drukpa1455:codex/map-ocr-skip-text-layer
Open

feat: map OCR text-layer skip option#149
drukpa1455 wants to merge 1 commit into
docling-project:mainfrom
drukpa1455:codex/map-ocr-skip-text-layer

Conversation

@drukpa1455

Copy link
Copy Markdown
Contributor

Summary

This maps Docling's new service request option into OCR engine options:

ConvertDocumentsOptions(ocr_skip_text_layer_pages=True)

When present, jobkit passes skip_text_layer_pages=True into the selected OCR options. The mapper reads the field with getattr(..., False) so this remains compatible with the current released Docling package until docling-project/docling#3465 is released.

Related to docling-project/docling#3464 and docling-project/docling#3465.

Validation

  • uv run ruff check docling_jobkit/convert/manager.py tests/test_pipeline_options_translation.py
  • uv run pytest tests/test_pipeline_options_translation.py
  • git diff --check

Signed-off-by: drk <drukpa1455@gmail.com>
@github-actions

Copy link
Copy Markdown
Contributor

DCO Check Passed

Thanks @drukpa1455, all your commits are properly signed off. 🎉

@mergify

mergify Bot commented May 17, 2026

Copy link
Copy Markdown
Contributor

Merge Protections

🟢 Merge protection satisfied — ready to merge.

Show 1 satisfied protection

🟢 Enforce conventional commit

Make sure that we follow https://www.conventionalcommits.org/en/v1.0.0/

  • title ~= ^(fix|feat|docs|style|refactor|perf|test|build|ci|chore|revert)(?:\(.+\))?(!)?:

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