Skip to content

Fix exception in CaptainAgent when using LLMConfig #1447

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

Merged
merged 4 commits into from
Mar 24, 2025
Merged

Conversation

kumaranvpl
Copy link
Collaborator

Why are these changes needed?

Related issue number

Closes #1446

Checks

@davorrunje davorrunje added this pull request to the merge queue Mar 24, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Mar 24, 2025
@kumaranvpl kumaranvpl enabled auto-merge March 24, 2025 20:20
@kumaranvpl kumaranvpl added this pull request to the merge queue Mar 24, 2025
Merged via the queue into main with commit 55d6aa4 Mar 24, 2025
16 checks passed
@kumaranvpl kumaranvpl deleted the fix-captain-agent branch March 24, 2025 20:34
Copy link

codecov bot commented Mar 24, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

❗ There is a different number of reports uploaded between BASE (338f89b) and HEAD (4acb2d8). Click for more details.

HEAD has 4653 uploads less than BASE
Flag BASE (338f89b) HEAD (4acb2d8)
3.9 297 0
ubuntu-latest 492 1
optional-deps 552 0
commsagent-slack 35 0
commsagent-discord 36 0
3.10 370 0
commsagent-telegram 35 0
3.13 330 0
core-without-llm 56 1
macos-latest 408 0
3.11 204 1
browser-use 16 0
3.12 141 0
windows-latest 442 0
jupyter-executor 35 0
graph-rag-falkor-db 24 0
twilio 36 0
retrievechat-pgvector 35 0
retrievechat 55 0
retrievechat-mongodb 37 0
retrievechat-qdrant 53 0
interop-crewai 34 0
docs 23 0
interop-pydantic-ai 36 0
websockets 36 0
crawl4ai 39 0
interop 39 0
interop-langchain 36 0
ollama 55 0
gpt-assistant-agent 11 0
teachable 13 0
agent-eval 4 0
gemini 57 0
llama-index-agent 11 0
websurfer 60 0
bedrock 54 0
cerebras 56 0
long-context 12 0
retrievechat-couchbase 9 0
anthropic 61 0
lmm 13 0
mistral 54 0
cohere 58 0
groq 55 0
together 55 0
swarm 55 0
integration 12 0
core-llm 9 0
neo4j 2 0
captainagent 1 0
autobuild 1 0
deepseek 1 0
openai 1 0
gemini-realtime 1 0
falkordb 2 0
openai-realtime 1 0
Files with missing lines Coverage Δ
autogen/llm_config.py 86.36% <100.00%> (-1.19%) ⬇️
autogen/oai/client.py 53.16% <100.00%> (-24.39%) ⬇️

... and 68 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

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]: CaptainAgent fails because AgentBuilder looks for OAI_CONFIG_LIST when using LLMConfig
2 participants