Skip to content

feat: expose compact_tables option for markdown export#108

Open
kristianQh wants to merge 1 commit intodocling-project:mainfrom
kristianQh:feature/compact-tables-option
Open

feat: expose compact_tables option for markdown export#108
kristianQh wants to merge 1 commit intodocling-project:mainfrom
kristianQh:feature/compact-tables-option

Conversation

@kristianQh
Copy link
Copy Markdown

Description:
docling-core's export_to_markdown and save_as_markdown already support
a compact_tables parameter that removes column padding from markdown tables,
but there is currently no way to pass it through docling-jobkit's pipeline.

This PR adds a compact_tables field to ConvertDocumentsOptions and threads
it through all export code paths:

  • _export_document_as_content (InBodyTarget / single-doc path)
  • _export_documents_as_files (multi-doc / zip path)
  • ResultsProcessor.process_documents (S3/target processor path)
  • CLI entry points (local.py, multiproc.py)
  • KFP pipeline (docling_s3in_s3out.py)

Defaults to False, no behavior change for existing users.

@github-actions
Copy link
Copy Markdown
Contributor

DCO Check Failed

Hi @kristianQh, your pull request has failed the Developer Certificate of Origin (DCO) check.

This repository supports remediation commits, so you can fix this without rewriting history — but you must follow the required message format.


🛠 Quick Fix: Add a remediation commit

Run this command:

git commit --allow-empty -s -m "DCO Remediation Commit for kristianQh <kralleyh@gmail.com>

I, kristianQh <kralleyh@gmail.com>, hereby add my Signed-off-by to this commit: 54929a763367b0cae75420f6c334b42934a3962b"
git push

🔧 Advanced: Sign off each commit directly

For the latest commit:

git commit --amend --signoff
git push --force-with-lease

For multiple commits:

git rebase --signoff origin/main
git push --force-with-lease

More info: DCO check report

@mergify
Copy link
Copy Markdown

mergify bot commented Mar 10, 2026

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)(?:\(.+\))?(!)?:

@kristianQh kristianQh changed the title expost compact_tables option for markdown export feat: expose compact_tables option for markdown export Mar 10, 2026
),
] = ""

compact_tables: Annotated[
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Let's name the argument consistently as md_compact_tables

@dolfim-ibm
Copy link
Copy Markdown
Member

@kristianQh thanks for the contribution. Please see the feedback above about renaming the argument.

Then please also sign off your commits as described in CONTRIBUTING.md. You can also apply a remediation commit as posted in the comment above.

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