Skip to content

feat: Improvements in user experience: Performance, error handling, logging #189

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
Open
nikos-livathinos wants to merge 8 commits into
base: main
Choose a base branch
from
Open

feat: Improvements in user experience: Performance, error handling, logging #189

nikos-livathinos wants to merge 8 commits into from
+348 −240

Conversation

@nikos-livathinos
Copy link
Member

@nikos-livathinos nikos-livathinos commented Dec 11, 2025

After receiving feedback from docling-eval users this PR addresses the following issues:

  • Improve the runtime performance.
  • Improve error handling in CLI.
  • Logging: Ensure to have one progress bar per modality. Remove excessive logs.

@nikos-livathinos
feat: Extend evaluate_dpbench_on_external_predictions.sh to include v…
4e2597e 
…isualisations of the evaluations

Signed-off-by: Nikos Livathinos <[email protected]>
@nikos-livathinos
fix: Improve error checking in main.py:visualize()
684808f 
Signed-off-by: Nikos Livathinos <[email protected]>
@nikos-livathinos
fix: Improve logging
f3b2f5f 
Signed-off-by: Nikos Livathinos <[email protected]>
@nikos-livathinos nikos-livathinos marked this pull request as draft December 11, 2025 10:33
@mergify
Copy link

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

@github-actions
Copy link
Contributor

github-actions bot commented Dec 11, 2025 •
edited
Loading

✅ DCO Check Passed

Thanks @nikos-livathinos, all your commits are properly signed off. 🎉

@nikos-livathinos nikos-livathinos marked this pull request as ready for review December 11, 2025 22:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@cau-git cau-git cau-git approved these changes

@Matteo-Omenetti Matteo-Omenetti Awaiting requested review from Matteo-Omenetti

@PeterStaar-IBM PeterStaar-IBM Awaiting requested review from PeterStaar-IBM

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@nikos-livathinos @cau-git