Update sample_evaluations.py #38530
Open
Update sample_evaluations.py #38530
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
…client.inference.get_azure_openai_client()
* generated agents * merge latest * update * add patches * update * new version generated * fixes * fix merge error * update * use assistant_id for now * update
* add more samples * adjust sleep * review updates
* Update sample_evaluations.py * Update sample_evaluations.py
…37831) * migrate async, poll functions, and samples from assistant to agent * migrate async, poll functions, and samples from assistant to agent * resolved comments * resolve comments * using print instead of logging * moved ai_client to main function * Resolved comment * Fixed samples
* [AI Client] adapt Sophia's assistant tests to client repo * get tests running and passing with pytest * recordings pass and pushed * review feedback
…t_tool_outputs_to_run and declare separate functions for streaming (#37851) * Took out streaming from create_run, create_and_process_run, and submit_tool_outputs_to_run and declare separate functions for streaming * Took out create_and_process_stream and update samples accordingly * fixed samples
* Fixes to Pylance error * fix sample * update * use file_upload_and_poll * updates to async samples * remove comment as we are using poll methods * more fixes
Co-authored-by: Marko Hietala <[email protected]>
* fix(pylint): Resolve R1705(no-else-return)
* fix(pylint): Resolve C0207(use-maxsplit-arg)
* fix(pylint): Resolve C0412(ungrouped-imports)
* fix(pylint): Resolve W1401(anomalous-backslash-in-string)
* fix(pylint): Resolve W0105(pointless-string-statement)
* fix(pylint): Ignore W0221(arguments-differ)
pylint has an open issue tracking false positives for
arguments-differ with overloads in subclasses
* fix(pylint): Ignore W0718(broad-exception-caught)
* fix(pylint): Ignore E0401(import-error) and E0611(no-name-in-module)
* fix(pylint): Ignore C4748(client-accepts-api-verison-keyword)
Currently, neither client classes supports overriding the api version
* fix(pylint): Ignore W0212(protected-access)
* fix(pylint): resolve W0707(raise-missing-from)
* fix(pylint): Ignore E1102(not-callable)
pylint appears to fail to correctly infer that
settings.tracing_implementation actually is callable
* fix(pylint): Ignore W0231(super-init-not-called)
* fix(pylint): Ignore W0613(unused-argument)
* fix(pylint): Ignore W0236(invalid-overriden-method)
* fix(pylint): Ignore R0914(too-many-locals)
* fix(pylint): Ignore R0902(too-many-instance-attributes)
* fix(pylint): Ignore R0915(too-many-statements)
* fix(pylint): Ignore R0911(too-many-return-statements)
* Ignore most of pylint errors * Add generated paths to pylint.rc * Use backslash for consistency in pylintrc files --------- Co-authored-by: Darren Cohen <[email protected]>
singankit
requested review from
luigiw,
needuv,
achauhan-scc,
kingernupur and
paulshealy1
as code owners
dargilco
approved these changes
Nov 14, 2024
|
API change check APIView has identified API level changes in this PR and created following API reviews. |
kashifkhan
requested review from
l0lawrence,
scbedd,
weshaggard and
benbp
as code owners
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.
Description
Please add an informative description that covers that changes made by the pull request and link all relevant issues.
If an SDK is being regenerated based on a new swagger spec, a link to the pull request containing these swagger spec changes has been included above.
All SDK Contribution checklist:
General Guidelines and Best Practices
Testing Guidelines