Skip to content

[Fix Log] Fix log of import error of local vllm server#331

Merged
SunnyHaze merged 2 commits intoOpenDCAI:mainfrom
HeRunming:main
Oct 23, 2025
Merged

[Fix Log] Fix log of import error of local vllm server#331
SunnyHaze merged 2 commits intoOpenDCAI:mainfrom
HeRunming:main

Conversation

@HeRunming
Copy link
Contributor

  1. 鉴于现在已经移除了使用aisuite的api server,因此从requirements里面移除aisuite
  2. vllm server中,from vllm import LLM失败的处理被简单的认为是没有装vllm,但实际使用时发现安装错误的transformers版本也会导致在导包的时候就报错,因此分开处理,以避免为debug带来麻烦以及为用户带来困惑

Copy link
Collaborator

@SunnyHaze SunnyHaze left a comment

Choose a reason for hiding this comment

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

English plz~

Copy link
Collaborator

@SunnyHaze SunnyHaze left a comment

Choose a reason for hiding this comment

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

English~

Copy link
Collaborator

@SunnyHaze SunnyHaze left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

@SunnyHaze SunnyHaze merged commit 481de1c into OpenDCAI:main Oct 23, 2025
9 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.

2 participants

Comments