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

Feature/conversation memory #751

Merged
merged 7 commits into from
Apr 23, 2024
Merged

Feature/conversation memory #751

merged 7 commits into from
Apr 23, 2024

Conversation

collindutter
Copy link
Member

@collindutter collindutter commented Apr 17, 2024

  • Moves docs on Conversation Memory Drivers to dedicated Drivers section of docs.
  • Adds recipe for implementing "Sessions" with Amazon DynamoDb.

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

@collindutter collindutter requested review from SavagePencil and a team April 17, 2024 18:40
vachillo
vachillo previously approved these changes Apr 17, 2024
vachillo
vachillo previously approved these changes Apr 17, 2024
Copy link
Member

@SavagePencil SavagePencil left a comment

Choose a reason for hiding this comment

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

spell out the benefit

@@ -0,0 +1,92 @@
Griptape provides [Conversation Memory](../griptape-framework/structures/conversation-memory.md) as a means of persisting conversation context across multiple Structure runs.
By default, Griptape's Conversation Memory does not provide a Coversation Memory Driver which means the conversation history is lost after the Structure is finished running.
Copy link
Member

Choose a reason for hiding this comment

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

"If you provide it with a suitable a Driver, the memory of the previous conversation can be preserved between invocations (runs?) of a Structure, giving it additional context for how to respond."

Copy link
Member

@SavagePencil SavagePencil left a comment

Choose a reason for hiding this comment

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

mamma mia, the tiniest typo

@@ -0,0 +1,92 @@
Griptape provides [Conversation Memory](../griptape-framework/structures/conversation-memory.md) as a means of persisting conversation context across multiple Structure runs.
If you provide it with a suitable a Driver, the memory of the previous conversation can be preserved between run of a Structure, giving it additional context for how to respond.
Copy link
Member

Choose a reason for hiding this comment

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

"with a suitable a Driver" -> "with a suitable Driver"

(maybe were you playing Mario?)

Copy link
Member Author

Choose a reason for hiding this comment

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

It's-a-fixed

@collindutter collindutter merged commit f16f421 into dev Apr 23, 2024
10 checks passed
@collindutter collindutter deleted the feature/conversation-memory branch April 23, 2024 02:19
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.

None yet

3 participants