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

Fix LLM parameter propagation and refactor BrowserUseTool #1033

Merged
merged 26 commits into from
Feb 20, 2025

Conversation

rjambrecic
Copy link
Collaborator

Why are these changes needed?

Related issue number

Closes #985

Checks

@rjambrecic rjambrecic changed the title Fix browser-use and crawl4ai tools when using Ollama with client_host parameter Fix LLM parameter propagation and refactor BrowserUseTool Feb 20, 2025
@davorrunje davorrunje enabled auto-merge February 20, 2025 10:48
@davorrunje davorrunje added this pull request to the merge queue Feb 20, 2025
Merged via the queue into main with commit 812acf7 Feb 20, 2025
463 of 484 checks passed
@davorrunje davorrunje deleted the fix-websurfer-tools-for-ollama branch February 20, 2025 11:05
Copy link

codecov bot commented Feb 20, 2025

Codecov Report

Attention: Patch coverage is 90.47619% with 14 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
autogen/import_utils.py 75.00% 3 Missing and 4 partials ⚠️
...ools/experimental/browser_use/langchain_factory.py 95.61% 2 Missing and 3 partials ⚠️
...ogen/tools/experimental/browser_use/browser_use.py 50.00% 1 Missing ⚠️
autogen/tools/experimental/crawl4ai/crawl4ai.py 66.66% 1 Missing ⚠️

❗ There is a different number of reports uploaded between BASE (5d2f165) and HEAD (5dfc518). Click for more details.

HEAD has 67 uploads less than BASE
Flag BASE (5d2f165) HEAD (5dfc518)
3.9 75 73
ubuntu-latest 143 119
3.10 95 93
interop 13 9
crawl4ai 13 9
browser-use 7 3
3.11 67 51
retrievechat 15 14
teachable 4 3
lmm 4 3
gemini 15 14
falkordb 2 0
autobuild 1 0
deepseek 1 0
neo4j 2 0
openai 1 0

📢 Thoughts on this report? Let us know!

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.

[Bug]: crawl4ai tool ollama api fails in WebSurferAgent
2 participants