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

Add image support for OpenAI client #21

Merged
merged 38 commits into from
Mar 25, 2025
Merged

Add image support for OpenAI client #21

merged 38 commits into from
Mar 25, 2025

Conversation

vrslev
Copy link
Contributor

@vrslev vrslev commented Mar 25, 2025 •
edited
Loading

No description provided.

vrslev added 30 commits March 25, 2025 11:01
@vrslev
Rename Message.text to Message.content
d8d09fe
@vrslev
Enhance Message content to support text and image items
d7c25f4
@vrslev
Adding ContentItem, ImageContentItem, and TextContentItem to core and…
c832f6d 
… __all__ list
@vrslev
Update content item types and add validation for content lists
a9f1acf
@vrslev
Enhance message preparation in YandexGPTClient to handle different co…
b1b4cf1 
…ntent types and multiple content items gracefully.
@vrslev
Enhance test to handle cases with list content in messages and raise …
53dd490 
…ValueError
@vrslev
Add helper functions and refactor LLMFuncRequestFactory for better co…
04f46f0 
…verage and readability
@vrslev
Add MessageFactory and update message content functions
d5bd426
@vrslev
Adjust factory class names to match expected data types in conftest.py
d8cdee5
@vrslev
Add MessageWithTextContent class and update LLMFuncRequest to use it
050ba68
@vrslev
Add LLMFuncRequestWithTextContentMessagesFactory for type checking an…
ee8805f 
…d tests
@vrslev
Organize MUTATIONS and ADDITIONAL_OPTIONS within LLMFuncRequestFactor…
d0c244d 
…y class
@vrslev
Adjust function names to start with 'set' for clarity in conftest.py
f1bbd32
@vrslev
Refactor mutation function names and update their usage in LLMFuncReq…
621b2c1 
…uestFactory
@vrslev
Refactor conftest.py by removing unused imports and moving MessageWit…
9307fb3 
…hTextContent and related classes to test_yandexgpt_client.py
@vrslev
Updates method docstrings to include LLMRequestValidationError in exc…
cf03576 
…eption descriptions
@vrslev
Add LLMRequestValidationError and reorganize exceptions in __init__.p…
51da283 
…y and core.py
@vrslev
Adjust error types in YandexGPTClient to use LLMRequestValidationError
fe63e0a
@vrslev
Adjust image content item import and enhance readability in message c…
8ba580c 
…ontent check
@vrslev
Enhance ChatCompletionsMessage to support text and image content items
e83807d
@vrslev
Update ChatCompletion classes to inherit from pydantic.BaseModel and …
93ea7fa 
…set default values
@vrslev
Add ChatCompletionsAnyContentItem and ChatCompletionsContentItems types
fc5da96
@vrslev
Add function to prepare one message for OpenAI client handling mixed …
f26fd79 
…content types
@vrslev
Rename ChatCompletionsMessage to ChatCompletionsInputMessage and upda…
b9f87ea 
…te related functions and tests
@vrslev
Adjust import and type annotations in openai.py
6415d05
@vrslev
Adjust image content item handling and update test case for message s…
862d9ab 
…tructure
@vrslev
Avoid processing empty string messages in OpenAI client
f427208
@vrslev
Add tests for ChatCompletionsInputMessage with multiple TextContentItems
af3b6bc
@vrslev
Adjust variable names from ContentItems to ContentItemList for consis…
b40988b 
…tency and clarity
@vrslev
Adjust formatting and content chunk merging in OpenAI client
1074846
@vrslev vrslev self-assigned this Mar 25, 2025
@vrslev vrslev requested review from xfenix, lesnik512 and insani7y March 25, 2025 09:57
@vrslev vrslev requested a review from insani7y March 25, 2025 11:56
@insani7y insani7y merged commit 2174cd0 into main Mar 25, 2025
14 checks passed
@insani7y insani7y deleted the images branch March 25, 2025 13:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@xfenix xfenix Awaiting requested review from xfenix

@lesnik512 lesnik512 Awaiting requested review from lesnik512

@insani7y insani7y Awaiting requested review from insani7y

Assignees

@vrslev vrslev

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@vrslev @insani7y