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

feat: Improve the LayoutPostprocessor #948

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

nikos-livathinos
Copy link
Collaborator

LayoutPostprocessor improvements:

  • Do not throw away Formulas which have no assigned cells (neither programmatic nor OCR)
  • Add some post-processing heuristics that look at the text to descide if a cluster label must be corrected
    • (e.g. flip text to caption label when the text starts with "fig" or "table")
  • Go through issues labeld with "layout", pick provided inputs, and decide if to make CVAT GT for it
  • Disable orphan creation code and check if it improves mAP
  • ...

Checklist:

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

Signed-off-by: Nikos Livathinos <nli@zurich.ibm.com>
Copy link

mergify bot commented Feb 12, 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)(?:\(.+\))?(!)?:

Signed-off-by: Nikos Livathinos <nli@zurich.ibm.com>
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