-
Notifications
You must be signed in to change notification settings - Fork 35
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
Support image input in the chat completion request #55
base: main
Are you sure you want to change the base?
Conversation
> Tested with a single image
Thanks for your work, will happily review this once you think it's ready (and passing the pre-commit check). If you have a working example for VLM / image processing to share, that would be a nice addition to the existing ones. |
Don't use grpcio and grpcio-tools 1.65.0 version (remised version) I don't know how to modify it in the poetry requirements |
I just modified the rules regarding the versions of grpcio and grpcio-tools in the toml, and I regenerated the poetry.lock Since this is my first time doing this, I would like to request special attention on this. |
I will provide an example of using my feature in a second step (in another PR I think) |
@lhenault I think you can review this PR (and change the version accordingly) :) |
Hey @Youho99 ! I tried your changes the other day and encountered a few issues, but probably because of me. Thanks again for your PR and sorry for the delay, it's very much appreciated. 😌 Let me have another look soon (and if you have a working example for image inputs that might speed up things). |
In the next few days I'll get back to it, and provide an example. Let me know if you have any problems. |
This pull request responds to issue #54
It allows you to take into account the architecture of the OpenAI API request with an image
Example on the OpenAI documentation:
The code has not been prettyfied, so we need to review that