Skip to content
This repository has been archived by the owner on Jan 7, 2025. It is now read-only.

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 subscribe to this conversation on GitHub. Already have an account? Sign in.
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