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 error message on invalid model name and groq access to open inter… #1377

Merged

Conversation

ighoshsubho
Copy link
Contributor

Describe the changes you have made:

I modified the error handling code to include a check for invalid model names and provide appropriate feedback to the user -

  1. When user enters incorrect model name.
  2. When user enters model from groq but not accessible.

Reference any relevant issues (e.g. "Fixes #000"):

Fixes #1368

Pre-Submission Checklist (optional but appreciated):

  • I have included relevant documentation updates (stored in /docs)
  • I have read docs/CONTRIBUTING.md
  • I have read docs/ROADMAP.md

OS Tests (optional but appreciated):

  • Tested on Windows
  • Tested on MacOS
  • Tested on Linux

@MikeBirdTech
Copy link
Collaborator

Nice work @ighoshsubho! Thanks for picking up this issue and resolving it!

@KillianLucas take a look and let Subho know what you think

@KillianLucas
Copy link
Collaborator

Much better. There's probably even more throughout the repo that's OpenAI centric / not model agnostic, this is a huge one that happens very frequently. Great fix @ighoshsubho, merging now!

@KillianLucas KillianLucas changed the base branch from main to development August 1, 2024 17:48
@KillianLucas KillianLucas merged commit 3b2ca3d into OpenInterpreter:development Aug 1, 2024
@ighoshsubho
Copy link
Contributor Author

Much better. There's probably even more throughout the repo that's OpenAI centric / not model agnostic, this is a huge one that happens very frequently. Great fix @ighoshsubho, merging now!

Thanks @KillianLucas will be fixing more since Open Interpreter became my necessity!

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.

Misleading error messages when users enter invalid Groq model
3 participants