-
-
Notifications
You must be signed in to change notification settings - Fork 4.4k
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
[Frontend] Add OpenAI Vision API Support #5237
Conversation
@DarkLight1337 This PR is probably good for first pass - I will mark it as "Ready for review" after I address your first round of review, along with adding more test and documentation. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the hard work! I have left a few comments.
With docs and tests modified - going to make this PR ready for review cc @DarkLight1337. |
Assuming that the tests pass, LGTM. I will double-check the docs after they have been built. |
Thanks for the review @simon-mo! I have addressed the comments and all related tests have passed. Could you take a look at the |
|
I have downloaded the CI log and confirm that this is the case (with the help of |
Co-authored-by: DarkLight1337 <tlleungac@connect.ust.hk>
Co-authored-by: DarkLight1337 <tlleungac@connect.ust.hk>
Co-authored-by: DarkLight1337 <tlleungac@connect.ust.hk>
Co-authored-by: DarkLight1337 <tlleungac@connect.ust.hk>
Co-authored-by: DarkLight1337 <tlleungac@connect.ust.hk>
Co-authored-by: DarkLight1337 <tlleungac@connect.ust.hk>
Co-authored-by: DarkLight1337 <tlleungac@connect.ust.hk>
This PR is the final step of the release milestone of adding support for GPT4V compatible inference.
This PR fixes
and closes #4200