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

Thread Message content object is not compliant with openai API #201

Closed
Pizzajongen opened this issue Dec 6, 2023 · 2 comments · Fixed by #202 or RageAgainstThePixel/com.openai.unity#139
Closed

Thread Message content object is not compliant with openai API #201

Pizzajongen opened this issue Dec 6, 2023 · 2 comments · Fixed by #202 or RageAgainstThePixel/com.openai.unity#139
Assignees
@StephenHodgson
Labels
bug Something isn't working

Comments

@Pizzajongen
Copy link
Contributor

Threads.Content does not implement the openAI API correctly. It looks like it is assumed that the image content would be the same as in a regular Chat.Content object. but instead of an Common.ImageUrl the Threads.Message object should contain an ImageFile object, containing only a file_id field (api docs).
@StephenHodgson StephenHodgson added the bug Something isn't working label Dec 6, 2023
@StephenHodgson StephenHodgson self-assigned this Dec 6, 2023
@chadcts
Copy link

chadcts commented Dec 7, 2023

0001-Added-missing-ContentType-image_file-and-associated-.patch

Here's a patch.

@StephenHodgson
Copy link
Member

So I've just been busy today haven't been able to knock this out

This was linked to pull requests Dec 7, 2023
OpenAI-DotNet 7.4.2 #202
Merged
com.openai.unity 7.0.2 RageAgainstThePixel/com.openai.unity#139
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@StephenHodgson StephenHodgson

Labels
bug Something isn't working
Milestone
No milestone
3 participants
@Pizzajongen @StephenHodgson @chadcts