-
Notifications
You must be signed in to change notification settings - Fork 3.1k
Add page-level text extraction for PDF/PPTX/DOCX documents #1263
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
base: main
Are you sure you want to change the base?
Conversation
- Add PageInfo class to store page number and content - Enhance DocumentConverterResult with optional pages attribute - Extend PdfConverter with extract_pages parameter for page-by-page processing - Add CLI support with --extract-pages and --pages-json flags - Implement robust error handling with fallback to full document extraction - Maintain 100% backward compatibility with existing API - Add comprehensive test suite with 8 test cases covering all scenarios
@microsoft-github-policy-service agree |
- Add slide-level extraction for PPTX files with extract_pages parameter - Each slide is treated as a PageInfo object with sequential numbering - Add extract_pages parameter to DOCX for API consistency (returns None due to dynamic pagination) - Import PageInfo class in both converters to support the new functionality - Add comprehensive test suites for both formats ensuring backward compatibility - Maintain 100% backward compatibility with existing API
I like this idea. It meshes well with the pptx slide output as well. I need to do a little testing before merging -- I'll try to do that this weekend. |
- Format all Python files with Black (v23.7.0) - Fix line length and formatting issues in page extraction feature files - Ensure consistent code style across the codebase
Hi team - any ETA on the release of this PR? This would greatly help our project. |
@jeonsworld It seems that some statuses are on standby, and we need them for our project, so please move forward. |
@afourney Hi, the workflows for this PR are currently pending approval. Could you please review and approve them so the checks can run? Thank you. |
Summary
Adds optional page extraction to PDF, PPTX, and DOCX converters with extract_pages parameter, returning structured page data while maintaining full backward compatibility.
Motivation
Users need to process PDF/PPTX/DOCX pages separately and know which content comes from which page for page-aware applications. Additionally, local development settings should not be tracked in version control.
Changes
Usage
Python API
CLI
Resolved #210 #122