-
Notifications
You must be signed in to change notification settings - Fork 15
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
Learn flag #85
Learn flag #85
Conversation
If im getting this right, the learn flag lets you use the Structure agent, but without it you can only use OpenAIfxns?
|
I concur with Jorge's comment. Since it's related to learning, can we also add confirmation message with Yes/No option whenever mdagent chooses to use CreateNewTool? |
If I understand this, you want the system to ask the user every single time it goes to create a new tool? I don't think we want that |
ignore what I said then haha. I think I just became paranoid about using extra LLMs for a moment lol |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!, make sure to pull main first. I merged a PR earlier
I added a learn flag to mdagent. This is so that if you want to force mdagent to only use existing/learned skills, rather than learn new skills, you can. I also added the flag to streamlit & added a unit test for initializing mdagent with the flag.