Skip to content
This repository has been archived by the owner on Apr 16, 2024. It is now read-only.

Disable off_prompt in local vector store client examples #199

Merged
merged 2 commits into from
Jan 17, 2024

Conversation

cjkindel
Copy link
Contributor

@cjkindel cjkindel commented Jan 11, 2024

Output artifacts from these tools will no longer go into TaskMemory. This is needed as there is no logic to retrieve the artifacts from TaskMemory in these examples. These only previously functioned through the underlying LLM, not the information present in the local vector store.

Example of failure in subtask logs:

INFO     Subtask 0947c8fa7180443a832c7efc2a6cd0bb
    Thought: The information about the "Attention Is All You Need" paper is stored in the memory. I will retrieve this information to provide the user with a detailed answer.                                                                  
    Action: {"name": "MemoryRetrieval", "path": "retrieve", "input": {"values": {"memory_name": "TaskMemory", "artifact_namespace": "ee06dca25dd64761a139aeabe9c07f0e"}}}
INFO     Subtask 0947c8fa7180443a832c7efc2a6cd0bb
    Response: tool not found 

📚 Documentation preview 📚: https://griptape--199.org.readthedocs.build/en/199/

SavagePencil
SavagePencil previously approved these changes Jan 11, 2024
@collindutter collindutter changed the base branch from dev to main January 12, 2024 00:54
@collindutter collindutter dismissed SavagePencil’s stale review January 12, 2024 00:54

The base branch was changed.

Output artifacts from these tools will no longer go into TaskMemory. This is needed as there is no logic to retrieve the artifacts from TaskMemory in these examples.
@cjkindel cjkindel requested a review from a team January 12, 2024 16:52
@collindutter collindutter merged commit 2220b28 into griptape-ai:main Jan 17, 2024
1 check passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants