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

Doc: Document difference between agents #722

Merged
merged 40 commits into from
Apr 5, 2024
Merged

Conversation

JayQuimby
Copy link
Contributor

Issue #628 - Document difference between different agents (and maybe expose descriptions in UI)

  • Updated Documentation in all Agenthub/... files to docstrings
  • Small prompt change in codeact_agent for consistency
  • Bug fix in Monologue: LLM needed to be imported
  • Renamed docs/examples to docs/documentation

Feel free to delete docs/documentation/Agents.md if you feel it is unnecessary. I thought it would be nice just to compare them to each other, but up to you.

@JayQuimby
Copy link
Contributor Author

@enyst Thank you so much!

docs/documentation/Agents.md Outdated Show resolved Hide resolved
docs/documentation/Agents.md Outdated Show resolved Hide resolved
docs/documentation/Agents.md Outdated Show resolved Hide resolved
docs/documentation/Agents.md Outdated Show resolved Hide resolved
docs/documentation/Agents.md Outdated Show resolved Hide resolved
docs/documentation/Agents.md Outdated Show resolved Hide resolved
docs/documentation/Agents.md Outdated Show resolved Hide resolved
docs/documentation/Agents.md Outdated Show resolved Hide resolved
@enyst
Copy link
Collaborator

enyst commented Apr 4, 2024

Thank you, this is awesome!

@rbren
Copy link
Collaborator

rbren commented Apr 4, 2024

Thanks for doing this!

There's one bit of confusion that I think would be helpful to clear up: the current agents aren't really specialists in any way. They're meant to be standalone agents--they all do all the things. Here's how I'd describe each one:

  • Codeact: a minimalist agent
  • Monologue: an agent with long-term and short-term memory management
  • Planner: an agent that utilizes the Plan mechanism to plot a long-term course of action

@JayQuimby
Copy link
Contributor Author

the current agents aren't really specialists in any way. They're meant to be standalone agents

This makes a lot of sense now that you said it. I can go rewrite the confusing parts. Thanks for that clarification.

@JayQuimby
Copy link
Contributor Author

@rbren I have been looking at Monologue Agent, and it looks like there is no utilization of the LongTermMemory. Is this intentional? Specifically, I can't find any instances of MonologueAgent.search_for() which would use the long term memory. Should I make an issue for this?

@rbren
Copy link
Collaborator

rbren commented Apr 5, 2024

Yeah agree--the agent never really asks to search long-term memory. In the early days it would, but I haven't seen it in a while.

Feel free to open an issue for it, but I'm planning on doing an agent sprint soon anyways

@rbren rbren merged commit d612894 into All-Hands-AI:main Apr 5, 2024
LeeABarron pushed a commit to LeeABarron/OpenDevin that referenced this pull request Apr 6, 2024
* doc: Guide for using local LLM with Ollama

* forgot to delete print statement

* typos

* Updated guide - new working method

* Move to docs folder

* Fixed front end overwrite local model name

* Update llm.py

* Delete docs/examples/images/example.png

deleted example.png

* Documentation of agent differences

* rename examples to documentation

* Docstrings for all agents

* typo fix

* typo fixes

* Typo fixes

* more typo fixes

* typo fix

* typo fixes

* typos fixed

* Typo fixes

* top 10 list

* typo fix

* typo fix

* typos to the moon

* typos fixed

* typo fix

* typo fix

* anotha one

* The rest of the typos

* Corrected agent descriptions

* Agents markdown updated

---------

Co-authored-by: Robert Brennan <accounts@rbren.io>
lehcode pushed a commit to lehcode/oppendevin that referenced this pull request Apr 10, 2024
* doc: Guide for using local LLM with Ollama

* forgot to delete print statement

* typos

* Updated guide - new working method

* Move to docs folder

* Fixed front end overwrite local model name

* Update llm.py

* Delete docs/examples/images/example.png

deleted example.png

* Documentation of agent differences

* rename examples to documentation

* Docstrings for all agents

* typo fix

* typo fixes

* Typo fixes

* more typo fixes

* typo fix

* typo fixes

* typos fixed

* Typo fixes

* top 10 list

* typo fix

* typo fix

* typos to the moon

* typos fixed

* typo fix

* typo fix

* anotha one

* The rest of the typos

* Corrected agent descriptions

* Agents markdown updated

---------

Co-authored-by: Robert Brennan <accounts@rbren.io>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants