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

Add provider config and update spice with new models #562

Merged
merged 2 commits into from
Apr 10, 2024
Merged

Conversation

PCSwingle
Copy link
Member

Fixes #560.

mentat/config.py Show resolved Hide resolved
requirements.txt Show resolved Hide resolved
tests/conftest.py Show resolved Hide resolved
Copy link
Contributor

mentatbot bot commented Apr 10, 2024

MENTAT CODE REVIEW IN ACTIVE DEVELOPMENT. Only in use on mentat and internal repos.
Please Reply with feedback.

This pull request introduces several important changes aimed at enhancing the tool's flexibility and compatibility with multiple LLM providers. By adding a provider parameter across various components and updating dependencies, it ensures that the tool remains versatile and up to date. These changes are well-implemented and consistent across the codebase, demonstrating a thoughtful approach to maintaining and improving the tool's functionality.

@PCSwingle PCSwingle merged commit 264849c into main Apr 10, 2024
16 checks passed
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.

"Provider is required when unknown models are used" but no way to set provider for gpt-4-turbo.
1 participant