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

add tongyi model wrapper #46

Merged
merged 5 commits into from
Mar 4, 2024
Merged

add tongyi model wrapper #46

merged 5 commits into from
Mar 4, 2024

Conversation

qbc2016
Copy link
Collaborator

@qbc2016 qbc2016 commented Mar 4, 2024

As the title says.

Copy link
Collaborator

@rayrayraykk rayrayraykk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I suggest to use dashscope instead of post api:

response = dashscope.Generation.call(
            model=self.model_name,
            messages=messages,
            result_format="message",  # set the result to be "message" format.
        )

And see the inline comments.

src/agentscope/models/post_model.py Outdated Show resolved Hide resolved
src/agentscope/models/post_model.py Outdated Show resolved Hide resolved
src/agentscope/models/post_model.py Outdated Show resolved Hide resolved
@qbc2016 qbc2016 changed the title add post api wrapper for tongyi add tongyi model wrapper Mar 4, 2024
@pan-x-c pan-x-c merged commit 996bc94 into modelscope:main Mar 4, 2024
3 checks passed
@qbc2016 qbc2016 deleted the dev_tongyi branch August 15, 2024 09:02
qbc2016 added a commit to qbc2016/AgentScope that referenced this pull request Oct 29, 2024
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