Skip to content

CopilotChat: Replace IMemoryStore with ISemanticTextMemory in BotController#1140

Merged
hathind-ms merged 1 commit intomicrosoft:mainfrom
adrianwyatt:bot-imemorystore
May 22, 2023
Merged

CopilotChat: Replace IMemoryStore with ISemanticTextMemory in BotController#1140
hathind-ms merged 1 commit intomicrosoft:mainfrom
adrianwyatt:bot-imemorystore

Conversation

@adrianwyatt
Copy link
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 adrianwyatt changed the title Replace IMemoryStore with ISemanticTextMemory in BotController CopilotChat: Replace IMemoryStore with ISemanticTextMemory in BotController May 21, 2023
@adrianwyatt adrianwyatt self-assigned this May 21, 2023
@hathind-ms hathind-ms enabled auto-merge (squash) May 21, 2023 21:23
@hathind-ms hathind-ms merged commit 01ea7c6 into microsoft:main May 22, 2023
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