-
Notifications
You must be signed in to change notification settings - Fork 3k
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
[AutoRelease] t2-weightsandbiases-2025-03-31-21903(can only be merged by SDK owner) #40285
[AutoRelease] t2-weightsandbiases-2025-03-31-21903(can only be merged by SDK owner) #40285
Conversation
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.
Pull Request Overview
This pull request adds the initial version of the Azure WeightsAndBiases Management Client, including both synchronous and asynchronous implementations and associated configuration, patch, and version files.
- Introduces the new mgmt client along with separate async (aio) modules.
- Includes auto-generated code with patch hooks and configuration updates.
- Provides documentation updates and a CHANGELOG for release history.
Reviewed Changes
Copilot reviewed 47 out of 51 changed files in this pull request and generated no comments.
Show a summary per file
File | Description |
---|---|
azure/mgmt/weightsandbiases/models/init.py | New auto-generated models module with patch extension. |
azure/mgmt/weightsandbiases/aio/operations/_patch.py | Async operations patch module following the customization pattern. |
azure/mgmt/weightsandbiases/aio/operations/init.py | Async operations initializer with patch integration. |
azure/mgmt/weightsandbiases/aio/_patch.py | Async patch module for additional customizations. |
azure/mgmt/weightsandbiases/aio/_configuration.py | Async client configuration setup including policies and authentication. |
azure/mgmt/weightsandbiases/aio/_client.py | Async client implementation with request sending logic and context manager support. |
azure/mgmt/weightsandbiases/aio/init.py | Async package initializer, exposing the async client and patch hooks. |
azure/mgmt/weightsandbiases/_version.py | Sets the client version to 1.0.0b1. |
azure/mgmt/weightsandbiases/_patch.py | Sync patch module for custom modifications. |
azure/mgmt/weightsandbiases/_configuration.py | Sync client configuration with pipeline policies and authentication. |
azure/mgmt/weightsandbiases/_client.py | Sync client implementation with request sending logic and context manager support. |
azure/mgmt/weightsandbiases/init.py | Module initializer exposing the client and version. |
azure/mgmt/init.py | Package initializer to extend the namespace. |
azure/init.py | Package initializer to extend the namespace. |
README.md | Documentation updates for setup, authentication, and usage examples. |
CHANGELOG.md | Release history documenting the initial version release. |
Files not reviewed (4)
- sdk/weightsandbiases/azure-mgmt-weightsandbiases/LICENSE: Language not supported
- sdk/weightsandbiases/azure-mgmt-weightsandbiases/MANIFEST.in: Language not supported
- sdk/weightsandbiases/azure-mgmt-weightsandbiases/_meta.json: Language not supported
- sdk/weightsandbiases/azure-mgmt-weightsandbiases/apiview-properties.json: Language not supported
Comments suppressed due to low confidence (2)
sdk/weightsandbiases/azure-mgmt-weightsandbiases/azure/mgmt/weightsandbiases/aio/_client.py:106
- [nitpick] Consider using a more descriptive literal for the first argument in self._serialize.url. Instead of "self._config.base_url", using a simpler identifier such as "base_url" could improve readability.
"endpoint": self._serialize.url("self._config.base_url", self._config.base_url, "str", skip_quote=True),
sdk/weightsandbiases/azure-mgmt-weightsandbiases/azure/mgmt/weightsandbiases/_client.py:54
- [nitpick] Consider adding an inline comment to clarify the purpose of the placeholder "{endpoint}" and explain how it is used later with format_url, aiding future maintainers.
_endpoint = "{endpoint}"
/azp run prepare-pipelines |
Azure Pipelines successfully started running 1 pipeline(s). |
https://github.com/Azure/sdk-release-request/issues/5930
Recording test success
https://dev.azure.com/azure-sdk/internal/_build?definitionId=7570