Skip to content

Features/mistralai intergrate - #143

Merged
warmshao merged 7 commits into
browser-use:mainfrom
pad918:main
Feb 1, 2025
Merged

warmshao merged 7 commits into
browser-use:mainfrom
pad918:main

Conversation

@pad918

@pad918 pad918 commented Jan 24, 2025

Copy link
Copy Markdown
Contributor

Added API for mistralAI

@CLAassistant

CLAassistant commented Jan 24, 2025 •

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

@warmshao

Copy link
Copy Markdown
Contributor

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

@pad918

pad918 commented Jan 27, 2025

Copy link
Copy Markdown
Contributor Author

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
Copy Markdown
Contributor

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

@pad918

pad918 commented Jan 27, 2025

Copy link
Copy Markdown
Contributor Author

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
Copy Markdown
Contributor

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

pad918 commented Jan 31, 2025

Copy link
Copy Markdown
Contributor Author

Should be ready for merging now

@warmshao

warmshao commented Feb 1, 2025

Copy link
Copy Markdown
Contributor

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

@pad918

pad918 commented Feb 1, 2025

Copy link
Copy Markdown
Contributor Author

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
bobsov pushed a commit to Solvrays/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