-
Notifications
You must be signed in to change notification settings - Fork 3k
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
Refactor monologue and SWE agent to use the messages in state history #1863
Conversation
remove now unused method
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.
Mostly LGTM, but I had one naming suggestion.
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.
Thanks!
Definitely. Not blocking, but it's quite likely there'll be some conflicts. I'll take care of that. |
…OpenDevin#1863) * Refactor monologue to use the messages in state history * add messages, clean up * fix monologue * update integration tests * move private method * update SWE agent to use the history from State * integration tests for SWE agent * rename monologue to initial_thoughts, since that is what it is
This PR updates monologue agent to the current architecture. It's a follow-up to #1847.
Necessary for the #1771 to start working with the right components.
@rbren
I assume we need to take a look at any agent who currently has some sort of own history or copies, for the session management.Done.Please note that this PR is cutting off monologue from memory condenser, since it doesn't maintain its own history of events anymore, thus can't directly replace it. This can be reworked and re-connected after 1810 and 1771.