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

🎯 fix: Google AI Client Stability; feat: gemini-exp models #4781

Merged
merged 2 commits into from
Nov 23, 2024

Conversation

danny-avila
Copy link
Owner

Summary

  • Removed the 7-second timeout constraint that was likely causing AbortError in the Google client

  • Increased delay timings from 12 to 15 seconds for generative models to prevent premature timeouts

  • Created a regex constant EXCLUDED_GENAI_MODELS to better identify and handle Gemini models

  • Updated model detection logic to use regex testing instead of string, to use @google/generative-ai explicitly for latest models

  • Removed beta API version specification for generative models to use the stable release

  • Simplified the GenAI client initialization by removing unnecessary configuration

  • Updated @langchain/google-genai package from 0.1.3 to 0.1.4 for latest model support

  • Improved debug logging to accurately reflect the model type being used

  • Bug fix (non-breaking change which fixes an issue)

  • My code adheres to this project's style guidelines

  • I have performed a self-review of my own code

  • My changes do not introduce new warnings

  • Local unit tests pass with my changes

@danny-avila danny-avila merged commit 2a77c98 into main Nov 23, 2024
2 checks passed
@danny-avila danny-avila deleted the fix/google branch November 23, 2024 00:08
@kneelesh48
Copy link
Contributor

learnlm is still not working

owengo pushed a commit to openwengo/LibreChat that referenced this pull request Nov 26, 2024
…vila#4781)

* fix: Google timing out and issuing AbortError, bump package, and use `@google/generative-ai` explicitly for latest models

* feat: gemini-exp-
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.

2 participants