Skip to content

[Bug] Concat OpenaiSDK reasoning content#2041

Merged
MaiziXiao merged 4 commits intoopen-compass:mainfrom
MaiziXiao:openai_reasnoning_content_concat
Apr 25, 2025
Merged

[Bug] Concat OpenaiSDK reasoning content#2041
MaiziXiao merged 4 commits intoopen-compass:mainfrom
MaiziXiao:openai_reasnoning_content_concat

Conversation

@MaiziXiao
Copy link
Copy Markdown
Contributor

  1. Concat OpenaiSDK reasoning content

# Concat Reasoning Content and tags to content
if (hasattr(responses.choices[0].message, 'reasoning_content')
and responses.choices[0].message.reasoning_content):
return (responses.choices[0].message.reasoning_content +
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

加个verbose的提示吧,我们是 没有拼在开头的,以及self.think_tag之后应该有/n

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Done. Check 过 QWQ 的输出,输出是 + + \n\n + content

Copy link
Copy Markdown
Collaborator

@tonysy tonysy left a comment

Choose a reason for hiding this comment

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

LGTM

@MaiziXiao MaiziXiao merged commit e8bc8c1 into open-compass:main Apr 25, 2025
8 checks passed
stephen-nju pushed a commit to stephen-nju/opencompass that referenced this pull request May 14, 2025
* [Bug] Concat OpenaiSDK reasoning content

* [Bug] Concat OpenaiSDK reasoning content

* update

* update
zyc140345 pushed a commit to zyc140345/opencompass that referenced this pull request Oct 23, 2025
* [Bug] Concat OpenaiSDK reasoning content

* [Bug] Concat OpenaiSDK reasoning content

* update

* update
iamkaia pushed a commit to iamkaia/opencompass that referenced this pull request Feb 4, 2026
* [Bug] Concat OpenaiSDK reasoning content

* [Bug] Concat OpenaiSDK reasoning content

* update

* update
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