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

Features/mistralai intergrate #143

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

Features/mistralai intergrate #143

wants to merge 3 commits into from

Conversation

pad918
Copy link

@pad918 pad918 commented Jan 24, 2025

Added API for mistralAI

@CLAassistant
Copy link

CLAassistant commented Jan 24, 2025

CLA assistant check
All committers have signed the CLA.

@warmshao
Copy link
Collaborator

Cool! I donot have a mistral api key. Could it run browser-use normally?

@pad918
Copy link
Author

pad918 commented Jan 27, 2025

Cool! I donot have a mistral api key. Could it run browser-use normally?

Yes, it performs as well as the other models. However, models that don't support images will fail when used with 'Use Vision' (just like openai gpt-3.5). If you'd like, I can give you with a temporary API key for testing, but I won't share it publicly.

@warmshao
Copy link
Collaborator

OK, please solve the conflicts. I will let it pass

@pad918
Copy link
Author

pad918 commented Jan 27, 2025

OK, please solve the conflicts. I will let it pass

It seems like commit "6ceaf8d" by vvincent1234 (6ceaf8d) removed the requirements for: [langchain-ollama==0.2.2
langchain-openai==0.2.14 and langchain-google-genai==2.0.8], but the code is still using these libraries. I think that was a mistake, should I add back these requirements or let them go? If I don't include at least the langchain-mistral requirement, my code will not work for people without that library.

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.

3 participants