Skip to content
Merged
Changes from 1 commit
Commits
Show all changes
100 commits
Select commit Hold shift + click to select a range
51fb73b
Update ImageEmbeddingsClient to change embed to embedImage
trangevi May 31, 2024
fd54bf9
Add typespec-csharp to the tspconfig
trangevi May 31, 2024
c53ca69
Merge remote-tracking branch 'upstream/dargilco/ai-model-inference' i…
trangevi Jun 4, 2024
08c7c64
Move csharp client property renames
trangevi Jun 5, 2024
ab6c309
Missing semicolons
trangevi Jun 5, 2024
4f8a7ea
Wrong namespace
trangevi Jun 5, 2024
be84fda
Incorrect namespaces
trangevi Jun 5, 2024
4f56fb3
Aliases need to have things referenced differently, apparently
trangevi Jun 5, 2024
db6580b
Need to specify parameters?
trangevi Jun 5, 2024
a5b3fbc
Move alias renames to the individual file for now
trangevi Jun 5, 2024
234a992
Merge branch 'dargilco/ai-model-inference' into trangevi/ai-model-inf…
trangevi Jun 12, 2024
1c552f3
Change ChatCompletionsOptions to a model
trangevi Jun 20, 2024
8e420b9
syntax
trangevi Jun 20, 2024
420cdf6
Merge branch 'dargilco/ai-model-inference' into trangevi/ai-model-inf…
trangevi Jun 21, 2024
8c122c5
Leave the model property
trangevi Jun 21, 2024
e6e24a1
Add additional properties for ChatCompletionsOptions
trangevi Jun 27, 2024
c33085f
3 dots, not 2
trangevi Jun 27, 2024
bc2ead4
Can't add docstring to generic spread property
trangevi Jun 27, 2024
cea8c46
Change unknown parameters to extra parameters
trangevi Jul 2, 2024
e5f72fe
Merge branch 'dargilco/ai-model-inference' into trangevi/ai-model-inf…
trangevi Jul 9, 2024
69c429f
Remove unnecessary rename
trangevi Jul 9, 2024
9353cd8
Removing api-key change for python, for now
trangevi Jul 11, 2024
404726c
Merge branch 'main' into trangevi/ai-model-inference
trangevi Jul 12, 2024
440f104
Merge branch 'dargilco/ai-model-inference' into trangevi/ai-model-inf…
trangevi Jul 23, 2024
acd2c9a
Make operator method internal
trangevi Jul 23, 2024
7e90c46
Fix pass-through header
trangevi Jul 25, 2024
6525a08
Re-enable protocol methods for csharp
trangevi Jul 26, 2024
c625ccb
Remove "spread" for ChatCompletionsOptions
trangevi Jul 26, 2024
b3ff7cf
Remove spread for additionalRequestHeaders
trangevi Jul 26, 2024
3cad35a
needed commas
trangevi Jul 26, 2024
4d8fe2c
another formatting attempt
trangevi Jul 26, 2024
442e62e
okay, figured out how the formatting works
trangevi Jul 26, 2024
eba14da
Alias breaks the generator
trangevi Jul 26, 2024
4a3dfd2
A model won't give us the interaction pattern that we want. Trying this.
trangevi Jul 26, 2024
392243f
Forgot semicolon
trangevi Jul 26, 2024
156d4e0
Merge branch 'dargilco/ai-model-inference' into trangevi/ai-model-inf…
trangevi Jul 29, 2024
72ff5d0
Does swapping the order of parameters matter?
trangevi Jul 29, 2024
063b051
Make extraParams optional?
trangevi Jul 29, 2024
dd47e68
Try and get rid of anonymous model
trangevi Jul 29, 2024
1f62242
Try adding RequestHeadersTrait
trangevi Jul 29, 2024
7254471
Maybe needs to be an object?
trangevi Jul 29, 2024
d752df6
Add using
trangevi Jul 29, 2024
4039eb1
Revert to "good" state
trangevi Jul 30, 2024
33fb66c
Merge branch 'dargilco/ai-model-inference' into trangevi/ai-model-inf…
trangevi Aug 5, 2024
d2d069e
Merge branch 'dargilco/ai-model-inference' into trangevi/ai-model-inf…
trangevi Aug 8, 2024
7b1e6e1
Try moving extra parameters. Change Parameters back to unknown
trangevi Aug 13, 2024
65239b3
Skip the alias
trangevi Aug 13, 2024
323366c
Is it the header which is causing issues?
trangevi Aug 13, 2024
1562993
Headers in the options object seems to not work
trangevi Aug 13, 2024
bd9d308
Remove commented operation
trangevi Aug 13, 2024
7e3d4fd
Unnecessary using
trangevi Aug 13, 2024
9b2fbf0
Merge branch 'dargilco/ai-model-inference' into trangevi/ai-model-inf…
trangevi Aug 23, 2024
376a388
Add embeddings for csharp
trangevi Aug 23, 2024
b2d19d3
Merge branch 'dargilco/ai-model-inference' into trangevi/ai-model-inf…
trangevi Aug 23, 2024
56e81a8
Client changes for C#
trangevi Aug 26, 2024
b1d22e9
Need the options objects to be models
trangevi Aug 26, 2024
fb263d7
Make the embedding methods public for now, for intial testing
trangevi Aug 28, 2024
2ef0546
Remove image embedding for now until it can be tested
trangevi Aug 29, 2024
aa97e50
Add new streaming classes for C#
trangevi Sep 27, 2024
96219f2
Merge branch 'dargilco/ai-model-inference' into trangevi/ai-model-inf…
trangevi Dec 10, 2024
331fcb3
Revert to Darren's branch to test
trangevi Dec 12, 2024
3e5944b
Explicitly set the completions options object to be an input
trangevi Dec 12, 2024
8592ffc
Other clients with spread parameters
trangevi Dec 12, 2024
844519f
No image embedding client yet, so leave this out for now
trangevi Dec 12, 2024
19827a6
Operations need to be internal for all languages
trangevi Dec 12, 2024
92ad99d
Comment
trangevi Dec 12, 2024
1859072
Add support for structured outputs
trangevi Dec 26, 2024
6d95236
We already had the property, just need to add one more option
trangevi Dec 26, 2024
009f004
Remove unnecessary inner class
trangevi Dec 26, 2024
c903862
Rename
trangevi Dec 26, 2024
f56435a
extra semicolon
trangevi Dec 26, 2024
e6bbdb3
Merge branch 'dargilco/ai-model-inference' into trangevi/ai-model-inf…
trangevi Dec 26, 2024
e69663c
Fix naming
trangevi Dec 27, 2024
b4f7079
Merge branch 'dargilco/ai-model-inference' into trangevi/ai-model-inf…
trangevi Dec 27, 2024
b4103c8
What if it's just unknown?
trangevi Dec 27, 2024
adb606c
unknown to record unknown
trangevi Dec 27, 2024
91a4877
Merge branch 'dargilco/ai-model-inference' into trangevi/ai-model-inf…
trangevi Dec 27, 2024
f09abe7
Merge branch 'dargilco/ai-model-inference' into trangevi/ai-model-inf…
trangevi Dec 27, 2024
2d7c2fc
Leave as unknown for now
trangevi Jan 3, 2025
3a39cfd
Add image embeddings client back to csharp
trangevi Jan 3, 2025
73fd4a3
Generation is fixed, switch back to record
trangevi Jan 6, 2025
42614ca
Make ImageEmbeddingsOptions explicitly an input
trangevi Jan 6, 2025
52907a9
Merge branch 'dargilco/ai-model-inference' into trangevi/ai-model-inf…
trangevi Jan 27, 2025
17b9692
First attempt. Make the main content item a parent of the two relevan…
trangevi Feb 3, 2025
340368f
Do I need this explicitly public?
trangevi Feb 3, 2025
aeb89b0
What about usage?
trangevi Feb 3, 2025
9dc6aeb
More inputs
trangevi Feb 4, 2025
476633f
Remove middle layer
trangevi Feb 4, 2025
9bba0ae
Missed some
trangevi Feb 4, 2025
3fad304
TSP formatting and swagger changes
trangevi Feb 5, 2025
fac8435
Another format change
trangevi Feb 5, 2025
3c99992
Merge pull request #6 from trangevi/trangevi/unified-audio-input
trangevi Feb 5, 2025
3271545
Merge branch 'dargilco/ai-model-inference' into trangevi/ai-model-inf…
trangevi Feb 6, 2025
f171fc0
Update with some changes to better reflect the desired merge
trangevi Feb 6, 2025
3d8564e
Mark options object as internal to address transitive access issue
trangevi Feb 10, 2025
41bbb07
review comments
trangevi Feb 11, 2025
eee1fa4
Merge branch 'dargilco/ai-model-inference' into trangevi/ai-model-inf…
trangevi Feb 14, 2025
5d2cdb4
Add renaming for python classes
trangevi Feb 14, 2025
091e8f7
Merge branch 'dargilco/ai-model-inference' into trangevi/ai-model-inf…
trangevi Feb 20, 2025
ab9c5a3
Update after npx command
trangevi Mar 14, 2025
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Mark options object as internal to address transitive access issue
Signed-off-by: trangevi <[email protected]>
  • Loading branch information
trangevi committed Feb 10, 2025
commit 3d8564e5bc0799e2c420edf70b83318d0ef7db02
1 change: 1 addition & 0 deletions specification/ai/ModelClient/client.tsp
Original file line number Diff line number Diff line change
Expand Up @@ -101,6 +101,7 @@ namespace Customizations; // The actual name here doesn't matter and is here for
// In Python we hand-write the 4 input message classes, so we make them internal here.
// The base class ChatRequestMessage has to have the same access as the derived classes,
// so we make it internal as well. However the Python code will make it public again without changes.
@@access(AI.Model.ChatCompletionsOptions, Access.internal, "python");
@@access(AI.Model.ChatRequestMessage, Access.internal, "python");
@@access(AI.Model.ChatRequestAssistantMessage, Access.internal, "python");
@@access(AI.Model.ChatRequestSystemMessage, Access.internal, "python");
Expand Down
Loading