Skip to content

Conversation

@DRMetaplectic
Copy link
Contributor

No description provided.

Comment on lines +254 to +267
class IntrospectiveAgentWorker(Workflow):
pass


class ToolInteractiveReflectionAgentWorker(Workflow):
pass


class LATSAgentWorker(Workflow):
pass


class CoAAgentPack(Workflow):
pass
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I propose that we just get rid of these 'legacy' agents. I don't think it's worth re-implementing them in the new framework.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

should I delete them now before ugrading? I will do that as a PR first

Comment on lines +44 to +45
class ReActAgent(Workflow):
"""Implement of RAG using a ReAcT Agent"""
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

https://developers.llamaindex.ai/python/examples/agent/react_agent/#run-some-queries

Add a comment to the docstring about how this is different from the builtin react agent

# Backwards compatibility which used chat
loop = asyncio.get_event_loop()
if loop.is_running():
raise RuntimeError()
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Add a message to the error

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.

2 participants