Skip to content
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

Fixes to Assistant, VectorStore client methods #70

Merged
merged 2 commits into from
Jul 23, 2024

Conversation

trrwilson
Copy link
Collaborator

  • Fixes GetAssistantAsync does not return a useful value #62 -- the remaining convenience methods in AssistantClient and VectorStoreClient have optional CancellationTokens added as method parameters, including the typed instance convenience overloads
  • Closes GetRunStepsAsync does not yet take a CancellationToken parameter #61 -- adds missing [EditorBrowsable(EditorBrowsableState.Never)] attributes to AssistantClient protocol methods, improving strongly typed convenience method discoverability
  • Addresses a yet-unreported problem with the new max_num_results property for file_search not being serialized/deserialized

@trrwilson trrwilson force-pushed the user/travisw/2024-06-17-assistant-fixes branch from e632c23 to 8c994fa Compare July 22, 2024 21:33
@trrwilson trrwilson merged commit d77539c into main Jul 23, 2024
1 check passed
@trrwilson trrwilson deleted the user/travisw/2024-06-17-assistant-fixes branch July 23, 2024 15:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants