Refactor monologue and SWE agent to use the messages in state history #1863
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
remove now unused method
enyst
changed the title
neubig
reviewed
May 22, 2024
|
Definitely. Not blocking, but it's quite likely there'll be some conflicts. I'll take care of that. |
super-dainiu
pushed a commit
to super-dainiu/OpenDevin
that referenced
this pull request
May 23, 2024
…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
4 tasks
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.