feat: support pdf files in prompt #591
Conversation
|
Trivy scanning results. Report Summary ┌──────────────────────┬──────┬─────────────────┬─────────┐
For OSS Maintainers: VEX NoticeIf you're an OSS maintainer and Trivy has detected vulnerabilities in your project that you believe are not actually exploitable, consider issuing a VEX (Vulnerability Exploitability eXchange) statement. To disable this notice, set the TRIVY_DISABLE_VEX_NOTICE environment variable. uv.lock (uv)Total: 14 (MEDIUM: 8, HIGH: 4, CRITICAL: 2) ┌──────────────┬────────────────┬──────────┬────────┬───────────────────┬───────────────┬──────────────────────────────────────────────────────────────┐ |
| if self.api_version is not None: | ||
| completion_kwargs["api_version"] = self.api_version | ||
|
|
||
| response = await entrypoint.acompletion(**completion_kwargs) |
There was a problem hiding this comment.
This edit was caused by change in litellm acompletion signature?
| for item in content: | ||
| if item.get("type") == "image_url": | ||
| has_images = True | ||
| elif item.get("type") == "file": |
There was a problem hiding this comment.
Does type "file" always mean that it is a pdf file or it can also be some other type of file?
| attachments = [] | ||
| for message in self.chat: | ||
| content = message["content"] | ||
| if not isinstance(content, list): |
There was a problem hiding this comment.
If we have only one attachment then it a list with one item?
| @@ -4,7 +4,7 @@ | |||
| import pytest | |||
There was a problem hiding this comment.
Do we also need unit tests for prompts with pdf files?
No description provided.