Streaming Client Result investigation - assistants #46
+3,238
−2,586
Conversation
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
* Upload the nuget package as a release asset * Remove workflow_dispatch trigger from release * Use `published` type for release event triggering * Add contents: write permission to release workflow
* Added `ModifyMessage` and `ModifyMessageAsync` protocol and convenience methods to `AssistantClient`. * Fixed typo: `GetAssistantFileAssociation` --> `GetAssistantFileAssociations` * Fixed typo: `GetAssistantFileAssociationAsync` --> `GetAssistantFileAssociationsAsync` * Added missing usages of `ConfigureAwait(false)` in `AssistantClient`.
* Add convenience methods for image edits and variations * investigation * WIP * rework to depend on local implementation of ClientModel * go back to using BCL type * WIP * WIP * file client * update for filename * Add model implementation prototype * backup WIP * unlink demo client * refactor * move content creation routine to model * null * revert * cleanup * revert * Add clean generated client using System.ClientModel 1.0.0-beta.2 * delete files not used for diff * move to public clientmodel package * Create transcription * Create translation * tidy * tidy * Add Stream overloads for audio service methods * file endpoints * nits * nits * image edits * image variations * nits * stream overloads for file upload * stream overloads for image operations * nits * update for Content-Length * rework file name APIs * rework file name APIs * Update from @joseharriaga Co-authored-by: Jose Arriaga Maldonado <[email protected]> * Update from @joseharriaga Co-authored-by: Jose Arriaga Maldonado <[email protected]> * Update from @joseharriaga Co-authored-by: Jose Arriaga Maldonado <[email protected]> * Update from @joseharriaga Co-authored-by: Jose Arriaga Maldonado <[email protected]> * Update from @joseharriaga Co-authored-by: Jose Arriaga Maldonado <[email protected]> * Update from @joseharriaga Co-authored-by: Jose Arriaga Maldonado <[email protected]> --------- Co-authored-by: Jose Arriaga Maldonado <[email protected]> Co-authored-by: Jose Arriaga Maldonado <[email protected]>
…ent (#17) * polyfill ExperimentalAttribute and use it for AssistantClient * polyfill update from source, parallel #if directive * PR feedback
* basics for moving ImageSize to a non-enum type * update named static readonly sizes per Jose's proposal
…r APIView feedback (#41) This PR updates audio, file, and image service methods that use the multipart/form-data content type on requests. It includes the following: - Removes BinaryData overloads of these service methods, in favor of Stream overload -- this is so users don't inadvertently load large binary content into memory by calling the wrong overload with a large file. - Adds FileStream overloads for a bit of convenience when a file is opened from disk -- so users don't have to pass the file name in two places. - Updates some output APIs that were returning BinaryData that could possibly be large files to return Streams instead.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Poking at it to learn