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.

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

Deprecate gpt-4-vision-preview model #630

Closed
wants to merge 1 commit into from

Conversation

tianshanghong
Copy link

  • Format the file bot/openai_helper.py
  • Use gpt-4o as default vision model

- Format the file `bot/openai_helper.py`
- Use `gpt-4o` as default vision model
@@ -26,8 +26,9 @@
GPT_3_16K_MODELS = ("gpt-3.5-turbo-16k", "gpt-3.5-turbo-16k-0613", "gpt-3.5-turbo-1106", "gpt-3.5-turbo-0125")
GPT_4_MODELS = ("gpt-4", "gpt-4-0314", "gpt-4-0613", "gpt-4-turbo-preview")
GPT_4_32K_MODELS = ("gpt-4-32k", "gpt-4-32k-0314", "gpt-4-32k-0613")
GPT_4_VISION_MODELS = ("gpt-4-vision-preview",)
GPT_4_128K_MODELS = ("gpt-4-1106-preview","gpt-4-0125-preview","gpt-4-turbo-preview", "gpt-4-turbo", "gpt-4-turbo-2024-04-09")
GPT_4_VISION_MODELS = ("gpt-4o",)

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

gpt-4o-mini is also recommended here

@didyouexpectthat
Copy link

I implemented these changes and it worked for me. Thank you.

@p3452
Copy link

p3452 commented Nov 28, 2024

Thanks! Now everything is clearly working!

@moritzfl
Copy link

moritzfl commented Dec 26, 2024

I can confirm this is working. I deployed the bot from tianshanghong:support-gpt-4o as the vision capability is clearly broken in n3d1117:main.

It would be great if this PR was merged!

@n3d1117
Copy link
Owner

n3d1117 commented Dec 28, 2024

Thanks @tianshanghong, done in #652

@n3d1117 n3d1117 closed this Dec 28, 2024
@tianshanghong tianshanghong deleted the support-gpt-4o branch December 28, 2024 23:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants