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: add Groq Model Constants #4689

Open
wants to merge 9 commits into
base: main
Choose a base branch
from

Conversation

edwinjosechittilappilly
Copy link
Collaborator

This solves the issue temporarily for the GROQ Models not visible in Agent Component.

This pull request includes updates to the GroqModel class and its associated constants to improve the handling of model names and simplify the codebase. The most significant changes involve the introduction of a new GROQ_MODELS constant and the removal of unnecessary methods for fetching model names dynamically.

Updates to model constants and usage:

Simplification of GroqModel class:

These changes streamline the code, making it easier to maintain and reducing the need for dynamic fetching of model names.

groq models updated
@dosubot dosubot bot added the size:M This PR changes 30-99 lines, ignoring generated files. label Nov 18, 2024
Copy link

codspeed-hq bot commented Nov 18, 2024

CodSpeed Performance Report

Merging #4689 will not alter performance

Comparing fix-groq-in-agents (685c44b) with main (6133fed)

Summary

✅ 15 untouched benchmarks

updated Groq Model list to be also a backup list if the API call fails, also at start.
format issue solved
@dosubot dosubot bot added size:S This PR changes 10-29 lines, ignoring generated files. and removed size:M This PR changes 30-99 lines, ignoring generated files. labels Nov 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size:S This PR changes 10-29 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants