Skip to content

Fix LLMConfig for 5 notebooks (7)#2789

Open
0xfandom wants to merge 1 commit into
ag2ai:mainfrom
0xfandom:fix/issue-1394
Open

Fix LLMConfig for 5 notebooks (7)#2789
0xfandom wants to merge 1 commit into
ag2ai:mainfrom
0xfandom:fix/issue-1394

Conversation

@0xfandom
Copy link
Copy Markdown

@0xfandom 0xfandom commented May 8, 2026

Why are these changes needed?

Continues the migration from dict-based llm_config / config_list_from_json to the typed LLMConfig API across the example notebooks. Sequence (7) following #2782.

Notebooks updated:

  • notebook/agentchat_human_feedback.ipynb
  • notebook/agentchat_planning.ipynb
  • notebook/agentchat_stream.ipynb
  • notebook/agentchat_teachability.ipynb
  • notebook/agentchat_websockets.ipynb

Each one now uses autogen.LLMConfig(...) or autogen.LLMConfig.from_json(...).where(...) and passes the resulting object directly to agents instead of a dict containing config_list. Where filter dicts were used, they are now expressed via .where(model=...).

Related issue number

Refs #1394

Checks

AI assistance

  • I understand the changes in this PR and can explain them in my own words.
  • I have verified that the PR description accurately reflects the actual diff.

@CLAassistant
Copy link
Copy Markdown

CLAassistant commented May 8, 2026

CLA assistant check
All committers have signed the CLA.

@github-actions github-actions Bot added the documentation Improvements or additions to documentation label May 8, 2026
@codecov
Copy link
Copy Markdown

codecov Bot commented May 11, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
see 57 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

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants