Skip to content

Conversation

@wellenzheng
Copy link
Collaborator

Description

Thank you for opening a Pull Request!
Before submitting your PR, there are a few things you can do to make sure it goes smoothly:

Fixes #<issue_number_goes_here>

zhengweijun added 9 commits July 23, 2025 11:29
Eliminate 'quality' and 'with_audio' parameters from various video generation methods in examples to streamline the code and improve clarity.
Change 'image_url' to 'image_urls' in multiple video generation examples for consistency and clarity.
Introduce a new function 'ofZhipu' in basic_usage.py to demonstrate the Zhipu chat completion feature. Update the main execution block to call this new function. Additionally, remove the unused 'switch_to_zhipu' parameter from the ZaiClient initialization for clarity.
Add ZhipuClient to the client module and refactor ZaiClient to inherit from a new BaseClient class. Update the basic_usage.py example to include a new function 'ofZai' for demonstrating ZaiClient functionality, while modifying 'ofZhipu' to utilize ZhipuClient. This enhances the clarity and structure of the client interactions.
Refactor README files to introduce ZhipuAIClient alongside ZaiClient, providing clear usage examples for both clients. Update basic_usage.py to replace ZhipuClient with ZhipuAIClient, ensuring consistency across the codebase. This enhances documentation and example clarity for users.
Refactor the naming of ZhipuAIClient to ZhipuAiClient across the codebase for consistency. Update the .gitignore file to include the .idea and .vscode directories. Remove the .vscode/settings.json file as it is no longer needed. Enhance README files to reflect the updated client naming conventions, ensuring clarity in usage examples.
# Conflicts:
#	examples/basic_usage.py
#	src/zai/_client.py
@wellenzheng wellenzheng self-assigned this Jul 23, 2025
@wellenzheng wellenzheng changed the title feat: zhipuclient feat: add zhipuclient Jul 23, 2025
@wellenzheng wellenzheng merged commit b4ff07c into main Jul 23, 2025
6 checks passed
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