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.

Sign up for GitHub

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

Jump to bottom

Azure AI Projects Python SDK #38338

Open
wants to merge 133 commits into
base: main
Choose a base branch
from
Open

Azure AI Projects Python SDK #38338

wants to merge 133 commits into from
+46,347 −0

Conversation

dargilco
Copy link
Member

@dargilco dargilco commented Nov 5, 2024

No description provided.

dargilco and others added 30 commits September 25, 2024 11:28
@dargilco
First version of client with endpoint operations
e119c2d
@dargilco
Make Endpoints sample work
5edce71
@dargilco
Create EndpointProperties class. Implement client.endpoints.get_default
7f17e49
@dargilco
Support AzureAIClient.from_connection_string()
f41ae25
@dargilco
Make default input argument for .endpoints.get
ee2aad6
@dargilco
Re-emit from latest TypeSpec. First attempt to support 3 different en…
1619243 
…dpoints
@dargilco
Fix construction of the three endpoints
5b9ecad
@dargilco
Re-emit from TypeSpec. Rename host_name to endpoint
8d33253
@dargilco
Bug fix
4640714
@dargilco
Add methods ai_client.inference.get_chat_completions_client() and ai_…
9fdbccb 
…client.inference.get_azure_openai_client()
@dargilco
Merge remote-tracking branch 'origin/main' into dargilco/azure-ai-client
a719078
@singankit
Adding sample and changing client name (#37724)
b1a9215
@dargilco
Add .inference.get_embeddings_client() method
6b53fc6
@jhakulin
Jhakulin/azure ai client agents (#37748)
3043ab1 
* generated agents

* merge latest

* update

* add patches

* update

* new version generated

* fixes

* fix merge error

* update

* use assistant_id for now

* update
@dargilco
Merge remote-tracking branch 'origin/main' into dargilco/azure-ai-client
dfbcb3c
@dargilco
Create seperate inference samples
f237671
@dargilco
Move SASTokenCredential into .models
367f88b
@jhakulin
Add more agents samples (#37777)
7bcf705 
* add more samples

* adjust sleep

* review updates
@singankit
Update _patch.py (#37760)
ae5a0af
@singankit
Online Evaluation changes (#37794)
b540375
@dargilco
Patch constructor for async AzureAIClient (#37804)
72cadc3
@dargilco
Merge remote-tracking branch 'origin/main' into dargilco/azure-ai-client
8805675
@singankit
Users/singankit/evaluation add data mapping (#37824)
8935a4d 
* Update sample_evaluations.py

* Update sample_evaluations.py
@dargilco
Fix .inference.get_azure_openai_client() for the async AzureAIClient (#…
c4d3540 
…37828)
@dargilco
Merge remote-tracking branch 'origin/main' into dargilco/azure-ai-client
a0d1232
@howieleung
migrate async, poll functions, and samples from assistant to agent (#…
37405e0 
…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
@glharper
[AI Client] adapt Sophia's assistant tests to client repo (#37819)
b2fa08e 
* [AI Client] adapt Sophia's assistant tests to client repo

* get tests running and passing with pytest

* recordings pass and pushed

* review feedback
@howieleung
Took out streaming from create_run, create_and_process_run, and submi…
1eea304 
…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
@dargilco
Miscellaneous updates (#37852)
e204002
@jhakulin
Jhakulin/azure ai client pylance (#37887)
d9f5c3a 
* 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
nick863 and others added 2 commits November 8, 2024 14:55
@nick863
Fix typing in samples (#38354)
c6edff5 
* Fix samples

* Re enable type checks and remove extra code

* Remove non needed code

* Remove non needed changes

* Remove non needed changes

* Fixes

* Return non optional client

* Remove extra line

* Fix file names

* Fixes

* Fix async file

* Fix mypy issues and suppress the ones related to typespec

* Fix cSpell

* Re generate code and fix unit tests

* Delete sdk/ai/azure-ai-projects/samples/config.json

* Fix typo

* Fix unit test
@dargilco
Run "black" (again)
131554f
johanste
johanste reviewed Nov 9, 2024
View reviewed changes
.gitignore Show resolved Hide resolved
@howieleung
Copy link
Member

Why I am not sure I can't resolve two comments from Krista, so I type here:

Krista: Not blocking preview, but it would be nice to include a timeout param so a user could stop waiting if they wanted
Howie: https://github.com/orgs/ai-platform-ml-platform/projects/7/views/1?pane=issue&itemId=86793660

Krista: Maybe i'm missing it, but I don't see the update. Was it only done with sync?
I think you refer to your previous comment related forcing to use keyword. I just added here
#38426

kdestin and others added 15 commits November 11, 2024 12:57
@kdestin
[ai] fix(pylint): Resolve W1309, W0107, W0707, R1705, W0401…
a204c17 
…, `R1714`, `W0611`, `W0404`, `W0201`, `W1201` and `W1203` (#38447)

* style: Run isort

* fix(pylint): Resolve W1309(f-string-without-interpolation)

         From sdk/ai/azure-ai-projects, run:

             ruff check --select=F541 azure --fix

* fix(pylint): Resolve W0107(unnecessary-pass)

    From sdk/ai/azure-ai-projects, run

        ruff check --select=PIE790 --fix azure

* fix(pylint): Resolve W0707(raise-missing-from)

* fix(pylint): Resolve R1705(no-else-return)

    From azure/ai/azure-ai-projects, run

        ruff check --select=RET505 --fix azure

* fix(pylint): Resolve W0401(wildcard-import)

* fix(pylint): Resolve R1714(consider-using-in)

* fix(pylint): Resolve W0611(unused-import)

* fix(pylint): Resolve W0404(reimported)

* fix(pylint): Resolve W0201(attribute-defined-outside-init)

* fix(pylint): Resolve W1201(logging-not-lazy) and W1203(logging-fstring-interpolation)
@howieleung
Add readme for get_message, save_file, get_file_content, tracing, Bin…
4ae547e 
…g grounding, enterprise (#38426)

* git Add readme for get_message, save_file, get_file_content, tracing

* Readme for enterprise search, code interpreter, attachment

* More documentation for tools

* Fixed create_thread documentation

* resolved comments

* Resolved comments

* Resolved comments from Krista
@nick863
More fixes to mypy (#38446)
7a1de64 
* Fix unit tests and some type issues.

* Fixes

* Fix mypy
@M-Hietala
agents telemetry pyright fixes (#38457)
d920d44 
Co-authored-by: Marko Hietala <[email protected]>
@nick863
Fix pyright for agents. (#38464)
7a66f7e 
* Fix as if python version is 3.8

* Fixed

* Add custom config

* Move line, ignoring import
@glharper
fix pip install instruction (#38480)
a629bfa
@kdestin
[ai] fix(pylint): Resolve C0301, C4739, C4740, C4741, C4742…
80f7311 
…, `C4743` for `azure-ai-projects` (#38459)

* fix(pylint): Resolve C0301(line-too-long)

* fix(pylint): Resolve C4743(docstring-should-be-keyword)

* fix(pylint): Resolve C4739(docstring-missing-param)

* fix(pylint): Resolve C4740(docstring-missing-type)

* fix(pylint): Resolve C4741(docstring-missing-return)

* fix(pylint): Resolve C4742(docstring-missing-rtype)

* style: Run black
@dargilco
Enable test recordings for tests\inference, tests\telemetry and tests…
78d345d 
…\connection (#38475)
@dargilco
Update cspell.json
fc77caa
@howieleung
Removed enterprise and add installation for tracing (#38483)
90e27d4 
* Removed enterprise and add installation for tracing

* clean up

* resolved comments
@dargilco
Fix mypy errors (#38487)
a06c0cc
@dargilco
Merge remote-tracking branch 'origin/main' into feature/azure-ai-proj…
6573482 
…ects
@dargilco
Run black --config ../../../eng/black-pyproject.toml
4cd7be7
@M-Hietala
agents tracing pylint fixes (#38496)
fb046e2 
Co-authored-by: Marko Hietala <[email protected]>
@dargilco
Remove premature adding of asserts.json file
723c59b
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@howieleung howieleung howieleung left review comments

@kristapratico kristapratico kristapratico left review comments

@johanste johanste johanste left review comments

@kashifkhan kashifkhan kashifkhan left review comments

@luigiw luigiw Awaiting requested review from luigiw luigiw is a code owner

@needuv needuv Awaiting requested review from needuv needuv is a code owner

@achauhan-scc achauhan-scc Awaiting requested review from achauhan-scc achauhan-scc is a code owner

@kingernupur kingernupur Awaiting requested review from kingernupur kingernupur is a code owner

@paulshealy1 paulshealy1 Awaiting requested review from paulshealy1 paulshealy1 is a code owner

@singankit singankit Awaiting requested review from singankit singankit is a code owner

@scbedd scbedd Awaiting requested review from scbedd scbedd is a code owner

@weshaggard weshaggard Awaiting requested review from weshaggard weshaggard is a code owner

@benbp benbp Awaiting requested review from benbp benbp is a code owner

At least 1 approving review is required to merge this pull request.

Assignees

@dargilco dargilco

Labels
AI
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.