Skip to content

fix(chunker): included content layers#593

Open
Vdaleke wants to merge 1 commit into
docling-project:mainfrom
Vdaleke:fix/hybrid-chunker-included-content-layers
Open

fix(chunker): included content layers#593
Vdaleke wants to merge 1 commit into
docling-project:mainfrom
Vdaleke:fix/hybrid-chunker-included-content-layers

Conversation

@Vdaleke

@Vdaleke Vdaleke commented Apr 21, 2026

Copy link
Copy Markdown
Contributor

Fix the use of content layers in HierarchicalChunker and HybridChunker.

Add new tests with passing the additional Furniture layer to MarkdownParams.

Resolves #592

@mergify

mergify Bot commented Apr 21, 2026

Copy link
Copy Markdown
Contributor

Merge Protections

🔴 1 of 2 protections blocking · waiting on 👀 reviews

Protection Waiting on
🔴 Require two reviewer for test updates 👀 reviews
🟢 Enforce conventional commit

🔴 Require two reviewer for test updates

Waiting for

  • #approved-reviews-by >= 2
This rule is failing.

When test data is updated, we require two reviewers

  • #approved-reviews-by >= 2

Show 1 satisfied protection

🟢 Enforce conventional commit

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

github-actions Bot commented Apr 21, 2026

Copy link
Copy Markdown
Contributor

DCO Check Passed

Thanks @Vdaleke, all your commits are properly signed off. 🎉

@codecov

codecov Bot commented Apr 22, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@Vdaleke

Vdaleke commented Apr 27, 2026

Copy link
Copy Markdown
Contributor Author

Hello @ceberam, this PR is similar to yours #518, could you review it?

@Vdaleke

Vdaleke commented May 5, 2026

Copy link
Copy Markdown
Contributor Author

@dolfim-ibm, @ceberam really need this feature, could you assign someone to this PR and issue?

Signed-off-by: Vdaleke <vdalekesmirnov@gmail.com>
@Vdaleke Vdaleke force-pushed the fix/hybrid-chunker-included-content-layers branch from c5556ba to 7e5b00d Compare May 29, 2026 07:20
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.

HybridChunker ignores layers in serialization params

1 participant