8000 Fix annotations by VikParuchuri · Pull Request #705 · datalab-to/marker · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Fix annotations #705

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 26 commits into from
Jun 2, 2025
Merged

Fix annotations #705

merged 26 commits into from
Jun 2, 2025

Conversation

VikParuchuri
Copy link
Member

No description provided.

VikParuchuri and others added 21 commits May 22, 2025 16:48
When merging provider and detection lines, some boxes may be missing,
but the layout check fails. This catches and merges in these boxes too.
Avoid slight cutting off of the layout boxes
In format lines mode, we include lines from surya which were not present
in the provider lines. However, we do not have ordering of these
relative to the provider lines.

This commit identifies blocks which contain lines from both sources, and
sort with a different method within those blocks (Unchanged for all
other blocks)
When merging multiple detected lines into a single provider line, skip
detected lines which have already been asigned to a different provider
line

Was causing repeated text otherwise
Ensure expansion doesn't cut into other layout blocks, still upper
bounded by the max fraction
Add "no corrections needed" check to llm_equation, standardize in other files
Copy link
Contributor
github-actions bot commented Jun 2, 2025

CLA Assistant Lite bot All contributors have signed the CLA ✍️ ✅

@VikParuchuri
Copy link
Member Author

recheck

@tarun-menta tarun-menta merged commit 49614cb into master Jun 2, 2025
4 checks passed
@github-actions github-actions bot locked and limited conversation to collaborators Jun 2, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants
0