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

fix: Fix the modalities for DPBench, OmniDocBench, DLNv1. Switch to new settings in SmolDocling API. Improve the documentation. #37

Merged
merged 11 commits into from
Feb 26, 2025

Conversation

nikos-livathinos
Copy link
Contributor

This PR addresses the improvements in Readme from #35.

Additionally:

  • Changes the way to initialize the VlmPipelineOptions() to support flash_attn.
  • It fixes the modalities column for DPBench, OmniDocBench, DLNv1.

…broken links in the documentation.

Signed-off-by: Nikos Livathinos <[email protected]>
Signed-off-by: Nikos Livathinos <[email protected]>
Copy link
Contributor

@PeterStaar-IBM PeterStaar-IBM left a comment

Choose a reason for hiding this comment

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

very good!

@nikos-livathinos nikos-livathinos merged commit ddae1ec into main Feb 26, 2025
5 of 6 checks passed
kmyusk pushed a commit that referenced this pull request Feb 26, 2025
…ew settings in SmolDocling API. Improve the documentation. (#37)

* chore: Change the pinning of docling

Signed-off-by: Nikos Livathinos <[email protected]>

* fix: Fix the modalities supported for DPBench, OmniDocBench, DLNv1. Clean up code.

Signed-off-by: Nikos Livathinos <[email protected]>

* docs: Update documentation to have all benchmarks in separate md files and place links in Readme.

Signed-off-by: Nikos Livathinos <[email protected]>

* fix: Change the initialization of the create_smol_docling_converter() to allow flash-attn

Signed-off-by: Nikos Livathinos <[email protected]>

* docs: List benchmarks in the main readme with short description. Fix broken links in the documentation.

Signed-off-by: Nikos Livathinos <[email protected]>

* docs: Fix broken link in Readme.

Signed-off-by: Nikos Livathinos <[email protected]>

* chore: Update lock file

Signed-off-by: Nikos Livathinos <[email protected]>

* fix: Add debug code to dump the predicted text in create_dlnv1_e2e_dataset()

Signed-off-by: Nikos Livathinos <[email protected]>

* chore: Update toml to pin docling with branch and extras

Signed-off-by: Nikos Livathinos <[email protected]>

* fix: Disable the generation of VLM text debugging files for DLNv1 benchmark

Signed-off-by: Nikos Livathinos <[email protected]>

* chore: Update toml to docling v2.25.0 with vln extra

Signed-off-by: Nikos Livathinos <[email protected]>

---------

Signed-off-by: Nikos Livathinos <[email protected]>
kmyusk pushed a commit that referenced this pull request Feb 26, 2025
…ew settings in SmolDocling API. Improve the documentation. (#37)

* chore: Change the pinning of docling

Signed-off-by: Nikos Livathinos <[email protected]>

* fix: Fix the modalities supported for DPBench, OmniDocBench, DLNv1. Clean up code.

Signed-off-by: Nikos Livathinos <[email protected]>

* docs: Update documentation to have all benchmarks in separate md files and place links in Readme.

Signed-off-by: Nikos Livathinos <[email protected]>

* fix: Change the initialization of the create_smol_docling_converter() to allow flash-attn

Signed-off-by: Nikos Livathinos <[email protected]>

* docs: List benchmarks in the main readme with short description. Fix broken links in the documentation.

Signed-off-by: Nikos Livathinos <[email protected]>

* docs: Fix broken link in Readme.

Signed-off-by: Nikos Livathinos <[email protected]>

* chore: Update lock file

Signed-off-by: Nikos Livathinos <[email protected]>

* fix: Add debug code to dump the predicted text in create_dlnv1_e2e_dataset()

Signed-off-by: Nikos Livathinos <[email protected]>

* chore: Update toml to pin docling with branch and extras

Signed-off-by: Nikos Livathinos <[email protected]>

* fix: Disable the generation of VLM text debugging files for DLNv1 benchmark

Signed-off-by: Nikos Livathinos <[email protected]>

* chore: Update toml to docling v2.25.0 with vln extra

Signed-off-by: Nikos Livathinos <[email protected]>

---------

Signed-off-by: Nikos Livathinos <[email protected]>
Signed-off-by: Yusik Kim <[email protected]>
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.

2 participants