-
Notifications
You must be signed in to change notification settings - Fork 57
Issue 38: Update to use Microsoft.Extensions.AI.Abstractions library#43
Open
quattro004 wants to merge 11 commits intodotnet:mainfrom
Open
Issue 38: Update to use Microsoft.Extensions.AI.Abstractions library#43quattro004 wants to merge 11 commits intodotnet:mainfrom
quattro004 wants to merge 11 commits intodotnet:mainfrom
Conversation
added 8 commits
March 17, 2025 13:06
…previous backend vs the update to use the Microsoft extensions AI libraries. Most tests are passing now.
… the repo shared config.
…epo shared config. See AddRepoSharedConfig
… dummy API key value if the configuration value isn't found so that the app will start up and not error out.
…responses, had removed too much code during the refactor to use Microsoft.Extensions.AI.
Author
|
@microsoft-github-policy-service agree |
Author
|
@microsoft-github-policy-service rerun |
added 3 commits
March 25, 2025 15:44
…ly. Had the wrong data in the response cache file that the tests were using.
Author
|
@dotnet-policy-service agree |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
I've updated the codebase to use the Microsoft.Extensions.AI.Abstractions library. I also noticed that since we are removing the SmartComponents.Inference.OpenAI project completely we might need a follow up task to update this documentation which mentions the project https://github.com/dotnet/smartcomponents/blob/main/docs/configure-openai-backend.md. Should I create this issue? Finally, I had some trouble getting all of the e2e tests to work, there's 10 that are failing and seem to be related to the same problem. I am having a hard time debugging them as it seems to change the state if it takes me too long in the debugger even in the existing test suite. Anyway, since I'm stuck on that I figured I would get this PR out there for review and we can proceed from there. Thanks for the opportunity to contribute.