Implement context-aware agent functionality. #1394
Merged
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
🦋 Changeset detectedLatest commit: 67d2300 The changes in this PR will be included in the next version bump. This PR includes changesets to release 24 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
@llamaindex/autotool
@llamaindex/cloud
@llamaindex/community
@llamaindex/core
@llamaindex/env
@llamaindex/experimental
llamaindex
@llamaindex/wasm-tools
@llamaindex/anthropic
@llamaindex/clip
@llamaindex/deepinfra
@llamaindex/groq
@llamaindex/huggingface
@llamaindex/ollama
@llamaindex/openai
@llamaindex/portkey-ai
@llamaindex/replicate
commit: |
himself65
reviewed
Oct 25, 2024
erik-balfe
force-pushed
the
feature/context-aware-agent-2
branch
from
3a2788f
to
23ae084
Compare
|
@himself65 About all the comments above. There’s a TypeScript error: I believe it’s currently impossible to implement this correctly with all the necessary types while extending base LLMAgent or AgentRunner classes. The simple approach that should work doesn’t function as intended in TypeScript, unfortunately. In latest version I have rewritten the code using a less general approach that works with OpenaiAgent and AnthropicAgent, along with some other workarounds, but without using 'any'. |
|
nice work |
- Add ContextAwareAgent mixin for retrieval capabilities for agents - Add contextAware openai agent and anthropic agent - Update agent guide about RAG with context aware agent example
erik-balfe
force-pushed
the
feature/context-aware-agent-2
branch
from
d57deef
to
67d2300
Compare
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.
Context-Aware Agent Implementation
This PR introduces context-awareness functionality for agents, inspired by the discussion in #1224.
Implementation:
withContextAwarenessmixin incontextAwareMixin.tsOpenAIContextAwareAgentandAnthropicContextAwareAgentMotivation:
The goal is to allow agents to access indexed data without explicitly calling retriever tools, reducing latency in real-time scenarios especially with not so fast LMs. This approach is analogous to the ContextChatEngine, which automatically incorporates retriever data into the chat context.
Approach:
The implementation uses a mixin pattern to add context retrieval and injection capabilities to existing agent classes. This method allows for flexibility and easy extension to different agent types.
Feedback on the implementation approach and any potential improvements is welcome.
Just recreated branch and PR (old one: #1235) because of conflicts in prev version. All modules were moved to core after that PR was been created. So its easier to create a new one.