gagb
gagb
> LGTM except that documentation is missing. I am fine merging this PR. We could add documentation in a separate PR. Ty, working on that issue
> Hi @antoan, > > Thanks for the note. Currently, there is only support for core autogen agent classes - UserProxy, Assistant (GroupChat support currently in development and on the...
This maybe limited by the context length of the model you are using or any parameters set to change max response length?
> Would it make sense to have the devcontainer compatible with AutoGen Studio? Currently it is a pain to run AutoGen Studio from the default devcontainer of this project. Best...
> > Would it make sense to have the devcontainer compatible with AutoGen Studio? Currently it is a pain to run AutoGen Studio from the default devcontainer of this project....
> Love to see the TUI in the sample apps. > > One natural question is if it will support the same "skills" format at AutoGenStudio -- it would be...
Thank you so much for comprehensive review @willmcgugan! This is so helpful and instructive for me! :) I really appreciate you taking time to review. I will update the app...
@DementedWeasel1971 you need to use different llm_config for group chat manager and gpt assistant. The former does not need assistant id and will complain. And the latter can use one....
@jackgerrits @ekzhu we should consider this PR for generalizing the issue that we also identified for executors + two agents.