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.

Sign up for GitHub

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

Jump to bottom

Feat/add multimodal loader #5

Merged
merged 11 commits into from
Apr 3, 2024
Merged

Feat/add multimodal loader #5

merged 11 commits into from
Apr 3, 2024

Conversation

lone17
Copy link
Contributor

@lone17 lone17 commented Apr 1, 2024 •
edited
Loading

Prerequisites:

  • Create a free account and get the API token from the Adobe service
    • Add the client ID to PDF_SERVICES_CLIENT_ID in .env
    • Add the client secret to PDF_SERVICES_CLIENT_SECRET in .env
  • Install the pdfservices-python-sdk following 1 of these methods:
    • Remove the install_dir/env and rerun installer to download fresh dependencies
    • Or install it separately via this repo

@lone17 lone17 requested a review from cin-albert April 1, 2024 15:30
@lone17 lone17 force-pushed the feat/add-multimodal-loader branch from 7e51556 to 3224054 Compare April 1, 2024 18:36
@cin-albert cin-albert self-requested a review April 3, 2024 06:00
@lone17 lone17 merged commit e67a25c into main Apr 3, 2024
3 checks passed
@lone17 lone17 deleted the feat/add-multimodal-loader branch April 3, 2024 07:52
phv2312 pushed a commit to phv2312/kotaemon that referenced this pull request Sep 11, 2024
@lone17 @cin-albert @phv2312
Feat/add multimodal loader (Cinnamon#5)
6e3be69 
* Add Adobe reader as the multimodal loader

* Allow FullQAPipeline to reasoning on figures

* fix: move the adobe import to avoid ImportError, notify users whenever they run the AdobeReader

---------

Co-authored-by: cin-albert <albert@cinnamon.is>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@cin-albert cin-albert cin-albert approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@lone17 @cin-albert