nw9663644-eng commented on issue #345: URL: https://github.com/apache/hugegraph-ai/issues/345#issuecomment-4591838525
Hi maintainers, I would like to work on this issue. I understand that the current upload path supports TXT and DOCX, while PDF currently raises a TODO error in read_documents(). My plan is: 1. Add text-based PDF extraction support in read_documents(). 2. Use a lightweight maintained PDF parser, such as pypdf, unless you prefer another dependency. 3. Extract PDF text page by page and keep the page order stable. 4. Return clear user-facing errors for encrypted, unreadable, or scanned-image-only PDFs. 5. Keep existing TXT and DOCX behavior unchanged. 6. Update the UI copy to mention TXT, DOCX, and PDF. 7. Add tests for PDF reading and regression tests for TXT/DOCX paths. Please let me know if this approach sounds good. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
