@randombet

Discussion / Follow-ups

I’m keeping this as an open discussion on whether we should maintain examples for both single and multiple models, or simplify to lists only.

Advantages of keeping both examples:

• Clarity → avoids confusion about whether a single string is valid.
• Flexibility → supports minimal cases (single) and batch use cases (multiple).
• Symmetry → consistent with tag filtering behavior.

Future work:
This raises a follow-up task: updating all other notebooks to ensure filtering examples follow the same consistent style.

@aakash232 Did you verify the documented use cases for config_list is working? How did you do that?

@aakash232 Did you verify the documented use cases for config_list is working? How did you do that?

@randombet
I did the local notebook setup and ran (+ logged the outputs for) the cells using both ways.
Currently, FOR A SINGLE PARAMETER, both ways are working "pass as a string" and "pass one element via list".

Based on the discussions (in github), I went ahead and supported the proposal + usage of "pass as a string" pattern.

P.S. Another solid check would be to do this in all notebooks. (Which points to the creation of a new issue/PR, as stated)

@aakash232 Please follow https://docs.ag2.ai/latest/docs/contributor-guide/pre-commit/ and fix the error.

Codecov Report

✅ All modified and coverable lines are covered by tests.

... and 41 files with indirect coverage changes