Fix #2960 by checking if the values are a list of lists.

microsoft / autogen

A programming framework for agentic AI 🤖

https://microsoft.github.io/autogen/

Creative Commons Attribution 4.0 International

31.45k stars 4.58k forks source link

Closed thinkall closed 3 months ago

thinkall commented 3 months ago

To fix #2960

Closes #2960

[ ] I've included any doc changes needed for https://microsoft.github.io/autogen/. See https://microsoft.github.io/autogen/docs/Contribute#documentation to build and test documentation locally.
[ ] I've added tests (if relevant) corresponding to the changes introduced in this PR.
[ ] I've made sure all auto checks have passed.

thinkall commented 3 months ago

i think this should handle the out of bounds error, looks good to me. thanks @thinkall

Thank you @Hk669 , I've updated code to reduce dictionary look up as you suggested.

Hk669 commented 3 months ago

Thank you @Hk669 , I've updated code to reduce dictionary look up as you suggested.

Thanks @thinkall , looks great.