Skip to content
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

Update for latest autogen and memgpt #1

Closed
nickbit opened this issue Dec 3, 2023 · 5 comments
Closed

Update for latest autogen and memgpt #1

nickbit opened this issue Dec 3, 2023 · 5 comments
Assignees
Labels
bug Something isn't working

Comments

@nickbit
Copy link

nickbit commented Dec 3, 2023

Hi Tyler,
You may need to update the files to comply with the latest changes of autogen and memgpt.
For example, create_autogen_memgpt_agent has a changed signature and does not accept a persona argument anymore.
Thank you for your great work.

@CynicalWilson
Copy link

yeah @tylerprogramming , looks like persona_description isn;t workinjg anymore :-(
@nickbit I've been tryingt, unsioccessfully, to find a proper workaround for this... would you maybe know how to get past the
File "X:\tyler_ai\autogen_memgpt_lmstudio\app.py", line 66, in <module> coder = create_autogen_memgpt_agent( ^^^^^^^^^^^^^^^^^^^^^^^^^^^^ TypeError: create_autogen_memgpt_agent() got an unexpected keyword argument 'persona_description'
issue?

@tylerprogramming
Copy link
Owner

@nickbit @CynicalWilson okay thank you for letting me know, I will check this out tomorrow and look at their repo to see what changed. 👍

@tylerprogramming
Copy link
Owner

Just quickly looking at it (not at home) it looks like they have a system_message as a parameter now, and that gets converted to a persona inside the function call. And it’s persona now not persona_desc. Yeah they definitely changed that, I think to make it look more like AutoGen. I’ll have an update when I get on my computer tomorrow

@tylerprogramming
Copy link
Owner

tylerprogramming commented Dec 4, 2023

@nickbit
@CynicalWilson

I added an issue on memgpt: cpacker/MemGPT#568

I updated code locally to work-ish. For just GPT and not local, and also just AutoGen and not MemGPT agents, it works. However as soon as I set it True to use a MemGPT agent I get a 401 for the openai API. After debugging through it I see that it doesn't take the api_key for the memgpt configuration. I see where it happens after debugging, and I've tried to hard-set it a few ways but couldn't get it to work. I opened an issue ticket with a link above. I will get back to this again as soon as I can. I will update code and push it, but just know it may not work just yet. Just want to see what they respond with or if somebody else faced the same issue.

@tylerprogramming tylerprogramming self-assigned this Dec 4, 2023
@tylerprogramming tylerprogramming added the bug Something isn't working label Dec 4, 2023
@tylerprogramming
Copy link
Owner

Going to mark this as closed now. I updated the code, though there are a lot of changes. Will re-factor when I create another project using it and mess around with it. Thank you

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants