Skip to content
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

Copilot chat: Document import supports pdf #700

Merged

Conversation

TaoChenOSU
Copy link
Contributor

Motivation and Context

Currently the document import feature only supports .txt files. We would like to support PDF files as well.

Description

  1. Add PdfPig 0.1.8 to the project as the lib to read PDFs.
  2. Update READMEs.

Contribution Checklist

@TaoChenOSU TaoChenOSU added the PR: ready for review All feedback addressed, ready for reviews label Apr 27, 2023
@gitri-ms gitri-ms added PR: ready to merge PR has been approved by all reviewers, and is ready to merge. and removed PR: ready for review All feedback addressed, ready for reviews labels Apr 27, 2023
@shawncal shawncal force-pushed the users/taochen/copilot_chat_pdf_support branch from 7935116 to 0d1eb44 Compare April 28, 2023 04:15
@github-actions github-actions bot added the kernel Issues or pull requests impacting the core kernel label Apr 28, 2023
@shawncal shawncal merged commit 16024e7 into microsoft:main Apr 28, 2023
dluc pushed a commit that referenced this pull request Apr 29, 2023
Currently the document import feature only supports .txt files. We would
like to support PDF files as well.

### Description
1. Add PdfPig 0.1.8 to the project as the lib to read PDFs.
2. Update READMEs.
dehoward pushed a commit to lemillermicrosoft/semantic-kernel that referenced this pull request Jun 1, 2023
Currently the document import feature only supports .txt files. We would
like to support PDF files as well.

### Description
1. Add PdfPig 0.1.8 to the project as the lib to read PDFs.
2. Update READMEs.
golden-aries pushed a commit to golden-aries/semantic-kernel that referenced this pull request Oct 10, 2023
Currently the document import feature only supports .txt files. We would
like to support PDF files as well.

### Description
1. Add PdfPig 0.1.8 to the project as the lib to read PDFs.
2. Update READMEs.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kernel Issues or pull requests impacting the core kernel PR: ready to merge PR has been approved by all reviewers, and is ready to merge.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants