-
Notifications
You must be signed in to change notification settings - Fork 237
Update long-running operation APIs to use LRO subclient pattern #129
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
annelo-msft
wants to merge
154
commits into
openai:main
from
annelo-msft:oai-lro-updateenumerator-pattern
Closed
Changes from all commits
Commits
Show all changes
154 commits
Select commit
Hold shift + click to select a range
f4ab277
Add project reference to SCM; move to new paging types
annelo-msft 38dd09d
wire return values and stub out implementations
annelo-msft 17c4078
update page collection to use new page token contract
annelo-msft 9cf0447
Update tests and samples
annelo-msft 7679061
fix page token serialization logic
annelo-msft 6acd48b
add async page collections
annelo-msft 6327af6
Add rehydration overload
annelo-msft e0c14eb
Add async rehydration overload
annelo-msft 2d46435
backup WIP on generalized implementation
annelo-msft bae625a
backup WIP on generalized implementation
annelo-msft 03e17b6
remove Assistant-specific page collection implementations in favor of…
annelo-msft 1f150ea
remove Assistant-specific page collection implementations in favor of…
annelo-msft 6ba8dbd
Backup of generalized protocol paging idea
annelo-msft 7c1f74d
remove Assistant-specific page result enumerator implementations in f…
annelo-msft bc2004e
take BinaryData instead of ClientToken, which cannot be constructed a…
annelo-msft 6138636
rework rehydration methods to use new ClientToken pattern
annelo-msft bae34b9
add test for rehydration method
annelo-msft 5283bc5
add a second test for rehydration method
annelo-msft e0305ea
SCM renames
annelo-msft 64bea2b
updates from mini pr review
annelo-msft ea8db96
rework RequestOptions
annelo-msft bcfed80
bug fix
annelo-msft e365566
GetPageCore
annelo-msft 1ed51af
backup of initial thinking on operation-specific page token idea
annelo-msft 2ad6dc1
extend operation-specific page token idea to protocol methods
annelo-msft 5b236a6
rename ClientToken to ContinuationToken
annelo-msft 444bee0
nits: tidy
annelo-msft 0060d5f
add test to illustrate convenience/protocol interop
annelo-msft 0aca908
updates based on feedback from Krzysztof
annelo-msft c4a8a63
add protocol implementations of all Assistant-related paged methods
annelo-msft 4a9d6af
add convenience implementations of Assistant-related paged methods
annelo-msft f91969f
move to CurrentPageToken instead of FirstPageToken
annelo-msft 90eed11
starting idea around client as enumerator; backing up
annelo-msft 313753e
more backup
annelo-msft a8e17e3
tidy up before taking inventory
annelo-msft c5261d1
backup prior to attempt to converge around single internal collection…
annelo-msft 9c05fc2
more refining
annelo-msft 209b71d
implement interface explicitly
annelo-msft b0d5f04
more tidy
annelo-msft c5b568d
Make it work for async
annelo-msft e07fb0b
restructure to look more like mini-client
annelo-msft f3c5e96
simplify
annelo-msft 8b762e2
more nits and simplify
annelo-msft 7cd2e92
nits
annelo-msft 67d70fa
renames and a little rework
annelo-msft daab108
move page tokens
annelo-msft 818b6bd
Implement remaining Assistants endpoings
annelo-msft 34264b5
Implement vector store paginated endpoints
annelo-msft 74e8b34
follow-ups to convenience and tests for vector stores
annelo-msft 827cf2b
bug fix
annelo-msft 891ccf5
temp bug fix
annelo-msft b15bd18
Proof of concept of PageEnumerator pattern for MessageCollection
annelo-msft 34175dc
Refactor to PageEnumerator pattern
annelo-msft 4a175d8
bug fix
annelo-msft 8656f12
nit
annelo-msft 4d2bf33
updates from SCM polish work
annelo-msft 73faa95
Implement new LRO pattern on top of page collection implementatino
annelo-msft fc5d7ed
add missed files
annelo-msft 529b399
add async overloads and rework poller a bit
annelo-msft a1422d1
fix some bugs
annelo-msft d338ec8
back up WIP
annelo-msft 80dbcfa
more backup WIP
annelo-msft 1ea8036
back up WIP
annelo-msft b2b3556
more backup of WIP - idea of protocol methods returning OperationResu…
annelo-msft 415fff5
re-impl of polling LRO
annelo-msft c7d0870
nits
annelo-msft 9ef8fc8
backup of streaming LRO WIP
annelo-msft e8886a3
intial e2e of async streaming ThreadRunLRO
annelo-msft 279b3e7
backup while testing
annelo-msft 4eeffaf
backup
annelo-msft e860fee
protocol operation implementation and polling test succeeds
annelo-msft aa94594
Streaming protocol implementation and test; one case open to solve still
annelo-msft 46ffefc
handle IsCompleted for protocol/streaming by throwing, since we can't…
annelo-msft b1602b2
add test for streaming cancellation
annelo-msft 05254c9
Initially implement StreamingThreadRunOperation.WaitAsync; with test …
annelo-msft 4d5b0ef
add GetUpdatesStreaming method to StreamingThreadRunOperation
annelo-msft 4482369
add SubmitToolOutputsToRunStreaming convenience method
annelo-msft 5c384f3
add basic test for streaming SubmitToolOutput method; fix to Required…
annelo-msft 67b0bdb
backup idea of ContinuableEnumerator
annelo-msft bd1722b
backup successful implementation of stream replacement ida
annelo-msft 10ff848
add functional implementation of WaitAsync for streaming LRO
annelo-msft 465b893
rewrite in terms of UpdateStatus
annelo-msft 8f35fa6
move initialization to constructor
annelo-msft d8a6829
updates from SCM changes and some more comments and TODOs
annelo-msft f068567
Merge remote-tracking branch 'upstream/main' into oai-pageenumerator-…
annelo-msft 4008584
move to SCM beta.5 package
annelo-msft b3d4fd7
Merge remote-tracking branch 'upstream/main' into oai-streamlroinherits
annelo-msft 2e97ef7
move over update for protocol-convenience cast
annelo-msft 1ee0486
some fixes
annelo-msft 4ce3888
updates from SCM code
annelo-msft 0622853
fix tests
annelo-msft e492b7c
remove internal protocol method implementations that have moved to in…
annelo-msft f811de1
nits
annelo-msft e7e3de7
add refdocs for paginated endpoint service methods
annelo-msft 88319b8
add refdocs for options types
annelo-msft 7118d13
add refdocs for collection rehydration overloads
annelo-msft e275801
add test and fix bugs
annelo-msft 8d66ade
add a test for vector store
annelo-msft e1bdc4e
Merge remote-tracking branch 'upstream/main' into oai-streamlroinherits
annelo-msft 8c77af5
Merge branch 'oai-streamlroinherits' of https://github.com/annelo-msf…
annelo-msft cfae419
updates from SCM changes
annelo-msft e99b2ba
Merge remote-tracking branch 'upstream/main' into oai-streamlroinherits
annelo-msft ae6eb9d
bug fix
annelo-msft d0417ae
back up WIP in adding convenience overloads to ThreadRunOperation min…
annelo-msft 1666794
backup prior to running tests
annelo-msft 422b594
add ReturnWhen parameter to protocol methods
annelo-msft 1944863
bug fixes
annelo-msft 6c637c7
Merge remote-tracking branch 'upstream/main' into oai-pageenumerator-…
annelo-msft 0813f06
Merge branch 'oai-pageenumerator-pattern' into oai-streamlroinherits
annelo-msft aacf13b
more updates from paging PR branch
annelo-msft 408bfef
add operation rehydration methods and tests
annelo-msft 9a920f8
nits: backup before enumerable experiment
annelo-msft 5373a8c
experiment with adding GetUpdates API to polling LRO type to enable e…
annelo-msft ea73b60
Merge remote-tracking branch 'upstream/main' into oai-streamlroinherits
annelo-msft 8d9eadb
name RunId to Id; add GetRun methods back to client
annelo-msft f231512
Begin experiment around LRO using generated update enumerator
annelo-msft f98a66f
implement generated enumerator pattern for protocol LRO
annelo-msft ac99e31
rename ThreadRunOperation and related to RunOperation
annelo-msft d462a11
implement generated enumerator pattern for convenience polling LRO
annelo-msft d714a9e
Back up functional implementation of streaming LRO GetUpdates while f…
annelo-msft 7041424
Make streaming convenience submit tools/Wait test pass
annelo-msft cf7a745
back up idea showing enumerable layering idea won't work
annelo-msft 8bf827f
illustrate polling with custom polling interval
annelo-msft 252d9fe
backup stub test for fine-tuning
annelo-msft 42f5f0c
add initial (untested) fine tuning LRO implementation
annelo-msft f0103e9
add initial implementation of protocol VectorStoreFileBatchOperation
annelo-msft 54be070
initial checkin of convenience layer for VectorStoreFileBatchOperation
annelo-msft ef1f94c
update tests to use LRO type
annelo-msft a3dfbb7
add back tests
annelo-msft 2f42e22
add examples back
annelo-msft 843fb97
make examples green
annelo-msft 1c03389
Merge remote-tracking branch 'upstream/main' into oai-lro-updateenume…
annelo-msft 08fee67
Add implementations where throwing NotImplementedException
annelo-msft dccc488
have streaming LRO override GetUpdates on polling type
annelo-msft b5adec2
updates
annelo-msft 2b8f481
cleanup in progress
annelo-msft d6bee73
more cleanup and rename rehydration methods to ContinueRun
annelo-msft 363a17a
Merge remote-tracking branch 'upstream/main' into oai-lro-updateenume…
annelo-msft e2605df
post-merge
annelo-msft f2745f6
revert file with bad merge
annelo-msft 47f68d6
public API updates from looking at the APIView
annelo-msft 18d389a
Merge remote-tracking branch 'upstream/main' into oai-lro-updateenume…
annelo-msft 042b904
updates from merge
annelo-msft c17fa59
more post-merge updates
annelo-msft c7daa98
add EBN to RunOperation protocol methods
annelo-msft facf668
fix bug in polling implementation; rework meta enumerator to not crea…
annelo-msft 3bdb145
Look at what WaitReturnReason enum implementation might look like
annelo-msft 032441e
back out WaitReturnReason change
annelo-msft 974e19a
initial add of BatchOperation for Batch protocol methods
annelo-msft 36a98d9
update with final SCM 1.1.0-beta.6 APIs
annelo-msft e374436
Add CancellationTokens back to RunOperation for consistency with SCM …
annelo-msft 978a5e2
make protocol methods for LROs take ReturnWhen parameter
annelo-msft fcd664d
nits
annelo-msft f0fb0ff
Merge remote-tracking branch 'upstream/main' into oai-lro-updateenume…
annelo-msft 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 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 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 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 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 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 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.
Should be able to update this to SCM 1.1.0-beta.6 when it ships next week.