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

Update xpdfreader package installation #2491

Merged
merged 1 commit into from
May 3, 2022

Conversation

AI-Ahmed
Copy link
Contributor

@AI-Ahmed AI-Ahmed commented May 3, 2022

This Update will fix this exception Exception: pdftotext is not installed. It is part of xpdf or poppler-utils software suite. Now, converting PDFs wouldn't have any issues.

closes #2455

This Update will fix this exception `Exception: pdftotext is not installed. It is part of xpdf or poppler-utils software suite. ` Now, converting PDFs wouldn't have any issues.
@review-notebook-app
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

Copy link
Member

@julian-risch julian-risch left a comment

Choose a reason for hiding this comment

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

Looks good to me! 👍 Thank you @AI-Ahmed for your contribution to Haystack.

@AI-Ahmed
Copy link
Contributor Author

AI-Ahmed commented May 3, 2022

I'm happy to help at any time!

@AI-Ahmed AI-Ahmed closed this May 3, 2022
@julian-risch
Copy link
Member

Hi @AI-Ahmed please do not close the issues or pull requests. The pull request closes automatically when we merge the changes and the issue will also be closed automatically then.

@julian-risch julian-risch reopened this May 3, 2022
@julian-risch julian-risch merged commit 9cdd719 into deepset-ai:master May 3, 2022
@AI-Ahmed
Copy link
Contributor Author

AI-Ahmed commented May 3, 2022

Sorry, maybe I did it by mistake!

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.

Updating Tutorials16 package installation issue. [Fixes included]
2 participants