-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
[vertex] Add PDF/plein texts support #1520
[vertex] Add PDF/plein texts support #1520
Conversation
src/lib/server/endpoints/document.ts
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This document processor might be enriched when specific needs will arise in the future
], | ||
preferredMimeType: "image/webp", | ||
maxSizeInMB: Infinity, | ||
maxSizeInMB: 20, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
vertex supports 20mb of inline files
@@ -213,7 +213,7 @@ | |||
...(!$page.data?.assistant && currentModel.tools | |||
? activeTools.flatMap((tool: ToolFront) => tool.mimeTypes ?? []) | |||
: []), | |||
...(currentModel.multimodal ? ["image/*"] : []), | |||
...(currentModel.multimodal ? currentModel.multimodalAcceptedMimetypes ?? ["image/*"] : []), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I prefered to allow passing specific mimetypes rather than adding a quite specific option enableDocumentParsing
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is great !
Thanks a lot, looks like a great PR! going to take a look 😄 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks good ! thanks for making multimodalAcceptedMimetypes
backwards compatible 😄
Following this issue #1505, adding PDF and plain text support to Vertex.
The basis has also been setup for other endpoints that would support inline documents.
DEMO
Enregistrement.de.l.ecran.2024-10-14.a.18.51.24.mov