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

CopilotChat: Add Azure Cog Search for Semantic Memory w/ IOptions fix #866

Merged
merged 2 commits into from May 8, 2023

Conversation

adrianwyatt
Copy link
Contributor

Motivation and Context

Previous change introduced bug with configuration.

Description

Changes ISemanticTextMemory from Singleton to Scoped.

@adrianwyatt adrianwyatt self-assigned this May 8, 2023
@adrianwyatt adrianwyatt enabled auto-merge (squash) May 8, 2023 23:35
@adrianwyatt adrianwyatt merged commit 67a4854 into microsoft:main May 8, 2023
10 checks passed
@adrianwyatt adrianwyatt deleted the re-acs branch May 10, 2023 17:41
dehoward pushed a commit to lemillermicrosoft/semantic-kernel that referenced this pull request Jun 1, 2023
…microsoft#866)

### Motivation and Context
Previous change introduced bug with configuration.

### Description
Changes ISemanticTextMemory from Singleton to Scoped.
golden-aries pushed a commit to golden-aries/semantic-kernel that referenced this pull request Oct 10, 2023
…microsoft#866)

### Motivation and Context
Previous change introduced bug with configuration.

### Description
Changes ISemanticTextMemory from Singleton to Scoped.
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

2 participants