Extend Chat API with Vision feature

betalgo / openai

OpenAI .NET sdk - Azure OpenAI, ChatGPT, Whisper, and DALL-E

https://betalgo.github.io/openai/

MIT License

2.89k stars 521 forks source link

Extend Chat API with Vision feature #434

Closed belaszalontai closed 10 months ago

belaszalontai commented 10 months ago

Introduce Vision feature extending the Chat Completion API.

Chat Completions and Vision is Tested.
Readme is updated
it contains a possible breaking change: In ChatMessage.cs in the ChatMessage model the constructor have been removed because of a calculated json property. (System.Text.Json has limitations)

kayhantolga commented 10 months ago

hi, could you update the target branch as dev branch, please?

belaszalontai commented 10 months ago

done

ErikDombi commented 10 months ago

Seems like there's a merge conflict on this at the moment.

Excited to see this one merged!

belaszalontai commented 10 months ago

Conflict has been resolved manually..

kayhantolga commented 10 months ago

The last merge to the dev branch caused some merge conflicts. I synchronized the dev branch and resolved the merge conflicts. I also renamed the "VisionContent" class to "MessageContent". I believe that OpenAI will allow us to pass different types of contents, so I think the new name is future-proof. Please let me know if you have any notes on my changes. I will be merging this PR but I am happy to discuss any changes that you don't like.

belaszalontai commented 10 months ago

@kayhantolga thnx for findings and resolution. I don't understand the merge conflicts but it does not matter anymore :)