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

update the semantic kernel #44

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

update the semantic kernel #44

wants to merge 2 commits into from

Conversation

john0isaac
Copy link
Owner

Purpose

This is a breaking change, all of the mongo db connection variables has been changed.
In this PR:

  • Update the semantic-kernel, and openai packages.
  • Use urlquote to parse the mongo connection string.
  • Use HNSW index as it's now available in the semantic kernel.
  • Use env var for index name as it's no longer static.
  • Add variable for the mongodb sku and the app service sku to allow for ease of modification.

Does this introduce a breaking change?

[x] Yes
[ ] No

Pull Request Type

What kind of change does this Pull Request introduce?

[ ] Bugfix
[x] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Documentation content changes
[ ] Other... Please describe:

@john0isaac john0isaac added the enhancement New feature or request label Dec 22, 2024
@john0isaac john0isaac self-assigned this Dec 22, 2024
@john0isaac
Copy link
Owner Author

Awaiting merge and release of new version of semantic-kernel
microsoft/semantic-kernel#10027

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant