Skip to content

Faster version of Chipper #252

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 6 commits into from
Oct 13, 2023
Merged

Faster version of Chipper #252

merged 6 commits into from
Oct 13, 2023

Conversation

@ajjimeno
Copy link
Contributor

@ajjimeno ajjimeno commented Oct 13, 2023

No description provided.

@ajjimeno ajjimeno requested a review from LaverdeS October 13, 2023 01:15
cragwolfe
cragwolfe reviewed Oct 13, 2023
View reviewed changes
LaverdeS
LaverdeS approved these changes Oct 13, 2023
View reviewed changes
Copy link
Contributor

@LaverdeS LaverdeS left a comment •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGMT! It is considerably faster now (X3) and it also fixed some UncategorizedText remnants/extras during generation.

@cragwolfe
Copy link
Contributor

cragwolfe commented Oct 13, 2023

Test instructions, how verified?

@LaverdeS
Copy link
Contributor

LaverdeS commented Oct 13, 2023 •
edited
Loading

Test instructions, how verified?

%%time

doc = DocumentLayout.from_file("layout-parser-paper-fast.pdf", detection_model=model)
for i in range(len(docb.pages)):
    print(f"--> Page {i}")
    for element in doc.pages[i].elements:
        display(element.type)
        display(element.text)

@ajjimeno ajjimeno merged commit f11559b into main Oct 13, 2023
@ajjimeno ajjimeno deleted the feat/fast-chipper-v2 branch October 13, 2023 21:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@cragwolfe cragwolfe cragwolfe left review comments

+1 more reviewer

@LaverdeS LaverdeS LaverdeS approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@ajjimeno @cragwolfe @LaverdeS