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

[AutoRelease] t2-weightsandbiases-2025-03-31-21903(can only be merged by SDK owner) #40285

Merged
merged 2 commits into from
Apr 1, 2025

Conversation

azure-sdk
Copy link
Collaborator

@azure-sdk azure-sdk commented Mar 31, 2025

@azure-sdk
Copy link
Collaborator Author

@Copilot Copilot bot review requested due to automatic review settings March 31, 2025 07:44
Copy link
Contributor

@Copilot Copilot AI left a 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}"

@ChenxiJiang333
Copy link
Member

/azp run prepare-pipelines

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@msyyc msyyc merged commit a31440d into Azure:main Apr 1, 2025
22 checks passed
@azure-sdk azure-sdk deleted the t2-weightsandbiases-2025-03-31-21903 branch April 1, 2025 16:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants