Skip to content

Commit 13f40b3

Browse files
authored
Revert "bugfix of openai_api_server for fastchat.serve.vllm_worker" (lm-sys#2400)
1 parent b921f16 commit 13f40b3

File tree

1 file changed

+14
-40
lines changed

1 file changed

+14
-40
lines changed

fastchat/serve/openai_api_server.py

Lines changed: 14 additions & 40 deletions
Original file line numberDiff line numberDiff line change
@@ -399,28 +399,15 @@ async def create_chat_completion(request: ChatCompletionRequest):
399399
for i, content in enumerate(all_tasks):
400400
if content["error_code"] != 0:
401401
return create_error_response(content["error_code"], content["text"])
402-
if isinstance(content["text"], list):
403-
for t in content["text"]:
404-
choices.append(
405-
ChatCompletionResponseChoice(
406-
index=i,
407-
message=ChatMessage(role="assistant", content=t),
408-
finish_reason=content.get("finish_reason", "stop"),
409-
)
410-
)
411-
else:
412-
choices.append(
413-
ChatCompletionResponseChoice(
414-
index=i,
415-
message=ChatMessage(role="assistant", content=content["text"]),
416-
finish_reason=content.get("finish_reason", "stop"),
417-
)
402+
choices.append(
403+
ChatCompletionResponseChoice(
404+
index=i,
405+
message=ChatMessage(role="assistant", content=content["text"]),
406+
finish_reason=content.get("finish_reason", "stop"),
418407
)
408+
)
419409
if "usage" in content:
420-
if isinstance(content["usage"], list):
421-
task_usage = UsageInfo.parse_obj(content["usage"][0])
422-
else:
423-
task_usage = UsageInfo.parse_obj(content["usage"])
410+
task_usage = UsageInfo.parse_obj(content["usage"])
424411
for usage_key, usage_value in task_usage.dict().items():
425412
setattr(usage, usage_key, getattr(usage, usage_key) + usage_value)
426413

@@ -788,27 +775,14 @@ async def create_chat_completion(request: APIChatCompletionRequest):
788775
for i, content in enumerate(all_tasks):
789776
if content["error_code"] != 0:
790777
return create_error_response(content["error_code"], content["text"])
791-
if isinstance(content["text"], list):
792-
for t in content["text"]:
793-
choices.append(
794-
ChatCompletionResponseChoice(
795-
index=i,
796-
message=ChatMessage(role="assistant", content=t),
797-
finish_reason=content.get("finish_reason", "stop"),
798-
)
799-
)
800-
else:
801-
choices.append(
802-
ChatCompletionResponseChoice(
803-
index=i,
804-
message=ChatMessage(role="assistant", content=content["text"]),
805-
finish_reason=content.get("finish_reason", "stop"),
806-
)
778+
choices.append(
779+
ChatCompletionResponseChoice(
780+
index=i,
781+
message=ChatMessage(role="assistant", content=content["text"]),
782+
finish_reason=content.get("finish_reason", "stop"),
807783
)
808-
if isinstance(content["usage"], list):
809-
task_usage = UsageInfo.parse_obj(content["usage"][0])
810-
else:
811-
task_usage = UsageInfo.parse_obj(content["usage"])
784+
)
785+
task_usage = UsageInfo.parse_obj(content["usage"])
812786
for usage_key, usage_value in task_usage.dict().items():
813787
setattr(usage, usage_key, getattr(usage, usage_key) + usage_value)
814788

0 commit comments

Comments
 (0)