feat: Docling SaaS ML backend example#889
Open
micaelakaplan wants to merge 10 commits intomasterfrom
Open
Conversation
- Add docling example: IBM Docling SaaS via DoclingServiceClient, DoclingDocument to ReactCode regions, label-studio-sdk get_local_path for uploads, strip trailing /v1 from DOCLING_SERVICE_URL so routes are not doubled. - ModelResponse.update_predictions_version: handle dict predictions and fix PredictionValue/list branching. - api: return 400 when POST /predict body is not JSON; log task counts. - README: catalog row for docling example. Made-with: Cursor
Remove unused config.json and label_mappings.json; ignore whole data/ volume tree. Docker Compose still bind-mounts ./data/... and creates paths at runtime. Made-with: Cursor
requirements-test.txt pins pytest/pytest-cov so Docker TEST_ENV installs stay compatible (_pytest.scope). README explains for CI/local image builds. Made-with: Cursor
…syocr - tests/requirements-test.txt: bump pytest/pytest-cov so anyio pytest plugin (_pytest.scope) loads on GitHub Actions. - docling: unpinned pytest/pytest-cov + responses like easyocr; drop extra README note. Made-with: Cursor
makseq
requested changes
Apr 29, 2026
Document meaningful sections in model.py and list supported Docling, Label Studio, server, and output environment variables in docker-compose.yml. Made-with: Cursor
Wrap Docling ReactCode predictions in PredictionValue so ModelResponse applies model_version normally. Revert earlier core api/response changes so the PR stays focused on the Docling example. Made-with: Cursor
Restore the global test requirements to the repo default and pin pytest/pytest-cov only in the Docling example test requirements. Made-with: Cursor
Rewrite the smoke test without pytest fixtures and disable the unused anyio plugin locally so the example does not require pytest>=8. Made-with: Cursor
Keep the Docling example on the repo's pytest 6 stack and pin anyio<4 so its pytest plugin does not require _pytest.scope. Remove the local pytest.ini workaround. Made-with: Cursor
Prevent pytest 6.x CI startup from auto-loading newer anyio's pytest plugin, which imports _pytest.scope from pytest 8+. Made-with: Cursor
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Made-with: Cursor