Skip to content

Conversation

@pad918
Copy link
Contributor

@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
Contributor 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
Contributor 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.

@warmshao
Copy link
Collaborator

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.

langchain-ollama==0.2.2 langchain-openai==0.2.14 and langchain-google-genai==2.0.8 will be installed with browser-use

@pad918
Copy link
Contributor Author

pad918 commented Jan 31, 2025

Should be ready for merging now

@warmshao
Copy link
Collaborator

warmshao commented Feb 1, 2025

Ok, please help me check whether langchain_mistralai will be installed with browser-use, otherwise please add it to requirements.txt

@pad918
Copy link
Contributor Author

pad918 commented Feb 1, 2025

Ok, please help me check whether langchain_mistralai will be installed with browser-use, otherwise please add it to requirements.txt

Yes I incorrectly assumed that it would also be installed with browser-use, but turns out it was not. I am now able to run it in a new uv environment after installing the requirements in requirements.txt

@warmshao warmshao merged commit fa7e168 into browser-use:main Feb 1, 2025
1 check passed
bobsov pushed a commit to docsmore/svg-browgene that referenced this pull request Jun 8, 2025
Features/mistralai intergrate
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