-
Notifications
You must be signed in to change notification settings - Fork 7.8k
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
chore: massive update of the Gemini models based on latest documentation #8822
chore: massive update of the Gemini models based on latest documentation #8822
Conversation
Hello can you take a look at this feature as well? #8805 |
Hi, I could try, but I am not familiar with Dify codebase, so please don't bet on me. |
@crazywoola About the request of #8805, are you referring to #8721 which simply just remove the harm category setting, but this time you want me to do it for Gemini endpoints? If that so, I believe that is simply just removing: Correct? If so, then I think I can do that |
Checklist:
Important
Please review the checklist below before submitting your pull request.
dev/reformat
(backend) andcd web && npx lint-staged
(frontend) to appease the lint gods (no code changes)Description
Fixes #8821.
Also, based on the documentation, "Latest" and "Latest Stable" models are two different versions. Hence, I also separated the
gemini-1.5-pro
away fromgemini-1.5-pro-latest
. Likewise for flash models.Type of Change
Testing Instructions
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration