Skip to content

fix: Fixes tables when using OCR #1261

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

Merged
merged 1 commit into from
Mar 29, 2025
Merged

fix: Fixes tables when using OCR #1261

merged 1 commit into from
Mar 29, 2025

Conversation

maxmnemonic
Copy link
Contributor

@maxmnemonic maxmnemonic commented Mar 29, 2025

Afer latest update, when using backend that supports word-level cells, but at the same time using OCR - tableformer received empty text for cells as @KapyGenius noticed. This PR fixes the issue.

Checklist:

  • Documentation has been updated, if necessary.
  • Examples have been added, if necessary.
  • Tests have been added, if necessary.

Signed-off-by: Maksym Lysak <mly@zurich.ibm.com>
Copy link

mergify bot commented Mar 29, 2025

Merge Protections

Your pull request matches the following merge protections and will not be merged until they are valid.

🟢 Enforce conventional commit

Wonderful, this rule succeeded.

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)(?:\(.+\))?(!)?:

@maxmnemonic maxmnemonic marked this pull request as ready for review March 29, 2025 08:38
@maxmnemonic maxmnemonic merged commit 7afad7e into main Mar 29, 2025
10 checks passed
@maxmnemonic maxmnemonic deleted the dev/ocr-tables-fix branch March 29, 2025 09:06
@KapyGenius
Copy link

I tested the changes it works well now 👍 thanks @maxmnemonic !

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

Successfully merging this pull request may close these issues.

3 participants