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

Dashscope service causing :ValueError: too many values to unpack (expected 11) #1367

Closed
dahaipeng opened this issue Jun 27, 2024 · 2 comments

Comments

@dahaipeng
Copy link

) = _get_protocol_params(kwargs)

When using DashScope service, in this line, the_get_protocol_paramsmethod returns 13 values but the unpack logic assumes that 11 values are returned, causing the ValueError: too many values to unpack (expected 11).
A proper way that works for me is adding another two values in the unpacking logic:

(
        api_protocol,
        ws_stream_mode,
        is_binary_input,
        http_method,
        stream,
        async_request,
        query,
        headers,
        request_timeout,
        form,
        resources,
        base_address,
        flattened_output
    ) = _get_protocol_params(kwargs)

The version of dashscope package is 1.19.3

@shenchucheng
Copy link
Collaborator

Thanks for your report. Could you please submit a pull request with your proposed changes?

xiaoxiaoimg pushed a commit to xiaoxiaoimg/MetaGPT that referenced this issue Jul 16, 2024
xiaoxiaoimg pushed a commit to xiaoxiaoimg/MetaGPT that referenced this issue Jul 16, 2024
xiaoxiaoimg pushed a commit to xiaoxiaoimg/MetaGPT that referenced this issue Jul 16, 2024
xiaoxiaoimg pushed a commit to xiaoxiaoimg/MetaGPT that referenced this issue Aug 22, 2024
xiaoxiaoimg pushed a commit to xiaoxiaoimg/MetaGPT that referenced this issue Sep 12, 2024
xiaoxiaoimg pushed a commit to xiaoxiaoimg/MetaGPT that referenced this issue Sep 23, 2024
xiaoxiaoimg pushed a commit to xiaoxiaoimg/MetaGPT that referenced this issue Sep 24, 2024
xiaoxiaoimg pushed a commit to xiaoxiaoimg/MetaGPT that referenced this issue Sep 24, 2024
@better629
Copy link
Collaborator

better629 commented Oct 10, 2024

@dahaipeng
I reproduce the problem with dashscope~=1.19.3, and it works with dashscope==1.14.1. You can download the version if needs. And I will submit a PR about the problem.

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

No branches or pull requests

4 participants