-
Notifications
You must be signed in to change notification settings - Fork 5.6k
Merge csharp feature branch into main ai inference branch #29763
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
Closed
trangevi
wants to merge
59
commits into
Azure:dargilco/ai-model-inference
from
trangevi:trangevi/ai-model-inference
Closed
Changes from all commits
Commits
Show all changes
59 commits
Select commit
Hold shift + click to select a range
51fb73b
Update ImageEmbeddingsClient to change embed to embedImage
trangevi fd54bf9
Add typespec-csharp to the tspconfig
trangevi c53ca69
Merge remote-tracking branch 'upstream/dargilco/ai-model-inference' i…
trangevi 08c7c64
Move csharp client property renames
trangevi ab6c309
Missing semicolons
trangevi 4f8a7ea
Wrong namespace
trangevi be84fda
Incorrect namespaces
trangevi 4f56fb3
Aliases need to have things referenced differently, apparently
trangevi db6580b
Need to specify parameters?
trangevi a5b3fbc
Move alias renames to the individual file for now
trangevi 234a992
Merge branch 'dargilco/ai-model-inference' into trangevi/ai-model-inf…
trangevi 1c552f3
Change ChatCompletionsOptions to a model
trangevi 8e420b9
syntax
trangevi 420cdf6
Merge branch 'dargilco/ai-model-inference' into trangevi/ai-model-inf…
trangevi 8c122c5
Leave the model property
trangevi e6e24a1
Add additional properties for ChatCompletionsOptions
trangevi c33085f
3 dots, not 2
trangevi bc2ead4
Can't add docstring to generic spread property
trangevi cea8c46
Change unknown parameters to extra parameters
trangevi e5f72fe
Merge branch 'dargilco/ai-model-inference' into trangevi/ai-model-inf…
trangevi 69c429f
Remove unnecessary rename
trangevi 9353cd8
Removing api-key change for python, for now
trangevi 404726c
Merge branch 'main' into trangevi/ai-model-inference
trangevi 440f104
Merge branch 'dargilco/ai-model-inference' into trangevi/ai-model-inf…
trangevi acd2c9a
Make operator method internal
trangevi 7e90c46
Fix pass-through header
trangevi 6525a08
Re-enable protocol methods for csharp
trangevi c625ccb
Remove "spread" for ChatCompletionsOptions
trangevi b3ff7cf
Remove spread for additionalRequestHeaders
trangevi 3cad35a
needed commas
trangevi 4d8fe2c
another formatting attempt
trangevi 442e62e
okay, figured out how the formatting works
trangevi eba14da
Alias breaks the generator
trangevi 4a3dfd2
A model won't give us the interaction pattern that we want. Trying this.
trangevi 392243f
Forgot semicolon
trangevi 156d4e0
Merge branch 'dargilco/ai-model-inference' into trangevi/ai-model-inf…
trangevi 72ff5d0
Does swapping the order of parameters matter?
trangevi 063b051
Make extraParams optional?
trangevi dd47e68
Try and get rid of anonymous model
trangevi 1f62242
Try adding RequestHeadersTrait
trangevi 7254471
Maybe needs to be an object?
trangevi d752df6
Add using
trangevi 4039eb1
Revert to "good" state
trangevi 33fb66c
Merge branch 'dargilco/ai-model-inference' into trangevi/ai-model-inf…
trangevi d2d069e
Merge branch 'dargilco/ai-model-inference' into trangevi/ai-model-inf…
trangevi 7b1e6e1
Try moving extra parameters. Change Parameters back to unknown
trangevi 65239b3
Skip the alias
trangevi 323366c
Is it the header which is causing issues?
trangevi 1562993
Headers in the options object seems to not work
trangevi bd9d308
Remove commented operation
trangevi 7e3d4fd
Unnecessary using
trangevi 9b2fbf0
Merge branch 'dargilco/ai-model-inference' into trangevi/ai-model-inf…
trangevi 376a388
Add embeddings for csharp
trangevi b2d19d3
Merge branch 'dargilco/ai-model-inference' into trangevi/ai-model-inf…
trangevi 56e81a8
Client changes for C#
trangevi b1d22e9
Need the options objects to be models
trangevi fb263d7
Make the embedding methods public for now, for intial testing
trangevi 2ef0546
Remove image embedding for now until it can be tested
trangevi aa97e50
Add new streaming classes for C#
trangevi File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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
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
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
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
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
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
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
Oops, something went wrong.
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Shouldn't this be limited to C# only?