Skip to content

CopilotChat: Replace IMemoryStore with ISemanticTextMemory in BotController - #1140

Merged
Harleen Thind (hathind-ms) merged 1 commit into
microsoft:mainfrom
adrianwyatt:bot-imemorystore
May 22, 2023
Merged

Harleen Thind (hathind-ms) merged 1 commit into
microsoft:mainfrom
adrianwyatt:bot-imemorystore

Conversation

@adrianwyatt

Copy link
Copy Markdown
Contributor

Motivation and Context

BotController required IMemoryStore which is not available when using Azure Cognitive Search

Description

  • Swapped IMemoryStore for ISemanticTextMemory in BotController to support high-level memory solutions, like Az Cog Search.
  • Added cancellation token support to bot controller

@adrianwyatt Adrian Bonar (adrianwyatt) changed the title Replace IMemoryStore with ISemanticTextMemory in BotController CopilotChat: Replace IMemoryStore with ISemanticTextMemory in BotController May 21, 2023
@hathind-ms
Harleen Thind (hathind-ms) enabled auto-merge (squash) May 21, 2023 21:23
@hathind-ms
Harleen Thind (hathind-ms) merged commit 01ea7c6 into microsoft:main May 22, 2023
Shawn Callegari (shawncal) pushed a commit to shawncal/semantic-kernel that referenced this pull request Jul 6, 2023
…roller (microsoft#1140)

### Motivation and Context
BotController required IMemoryStore which is not available when using
Azure Cognitive Search

### Description
- Swapped IMemoryStore for ISemanticTextMemory in BotController to
support high-level memory solutions, like Az Cog Search.
- Added cancellation token support to bot controller
golden-aries pushed a commit to golden-aries/semantic-kernel that referenced this pull request Oct 10, 2023
…roller (microsoft#1140)

### Motivation and Context
BotController required IMemoryStore which is not available when using
Azure Cognitive Search

### Description
- Swapped IMemoryStore for ISemanticTextMemory in BotController to
support high-level memory solutions, like Az Cog Search.
- Added cancellation token support to bot controller
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