Skip to content

♻️ All models should use nexent.core.model instead of smolagent #1971#2027

Merged
Phinease merged 2 commits intodevelopfrom
pyh/fix_llm_call_develop
Dec 8, 2025
Merged

♻️ All models should use nexent.core.model instead of smolagent #1971#2027
Phinease merged 2 commits intodevelopfrom
pyh/fix_llm_call_develop

Conversation

@YehongPan
Copy link
Copy Markdown
Contributor

♻️ All models should use nexent.core.model instead of smolagent #1971
[Specification Details]
1.Change the original llm_call in the code from OpenAIServerModel (smolagents) to OpenAIModel (nexent)
[Test Result]
Knowledgebase auto summary:
image
image

Agent import and generate:
image
image
image

Normal Chat:
image

[Specification Details] 1.Change the original llm_call in the code from OpenAIServerModel (smolagents) to OpenAIModel (nexent) 2.fix test cases.
@YehongPan YehongPan requested a review from Phinease as a code owner December 5, 2025 07:30
@codecov
Copy link
Copy Markdown

codecov bot commented Dec 5, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@Phinease
Copy link
Copy Markdown
Member

Phinease commented Dec 7, 2025

是否覆盖验证了思考与非思考模型?包括一些多模态模型当做语言模型调用的情况?

@YehongPan
Copy link
Copy Markdown
Contributor Author

是否覆盖验证了思考与非思考模型?包括一些多模态模型当做语言模型调用的情况?

双瑞哥,这个我周一再详细验证一下哇,确实没有考虑到多模态模型。

@YehongPan
Copy link
Copy Markdown
Contributor Author

是否覆盖验证了思考与非思考模型?包括一些多模态模型当做语言模型调用的情况?

已做验证,VL,VL-Thinking都能支持。
如图:
img_v3_02sp_5d8d11ac-b0ac-4931-b798-dc979bfffabg
img_v3_02sp_58770a97-f975-40ca-af22-27300d332b8g
img_v3_02sp_a940fecc-daa0-4363-8b48-181f59d4790g
img_v3_02sp_7ecd0fa4-6714-48b5-8e25-962e9fb85ecg
img_v3_02sp_18b609ed-ac1f-4163-9f16-8c2b8fe7bc9g
img_v3_02sp_c6cb8055-b162-42ac-b27d-5cac05fc836g
img_v3_02sp_1da7f9b1-a057-4a00-ac57-605ae19fe7fg

@Phinease Phinease merged commit 3aad19d into develop Dec 8, 2025
6 checks passed
@YehongPan YehongPan deleted the pyh/fix_llm_call_develop branch January 4, 2026 01:45
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.

2 participants