Skip to content

Conversation

@mxsl-gr
Copy link
Contributor

@mxsl-gr mxsl-gr commented Sep 9, 2024

this PR content:

  1. enhance the compatibility of the MiniMax chat model, add web search mode response in choice.message
  2. add web search mode for stream mode
  3. unit tests

related issue #1292

@markpollack
Copy link
Member

@mxsl-gr I'll merge as the code looks ok, but I get failing tests on MiniMaxChatOptionsTests

java.lang.AssertionError: 
Expecting actual:
  "根据目前可用的信息,关于2024年奥运会的具体金牌数量,包括美国的金牌总数,在提供的参考资料中并没有明确的记录。上述参考资料主要涉及的是残奥会的金牌榜和相关报道,而不是夏季奥运会的数据【1†source】【2†source】【3†source】【4†source】【5†source】【6†source】。

由于奥运会的赛事报道和数据更新是实时进行的,通常在奥运会闭幕后会有详细的奖牌统计公布。为了获取2024年夏季奥运会美国队的金牌总数,建议查询奥运会官方发布的最新统计数据或权威体育新闻报道。目前提供的信息无法直接回答美国在2024年夏季奥运会上获得的金牌总数。"
to contain:
  "40" 

	at org.springframework.ai.minimax.chat.MiniMaxChatOptionsTests.testWebSearch(MiniMaxChatOptionsTests.java:86)
	at java.base/java.lang.reflect.Method.invoke(Method.java:580)

and

java.lang.AssertionError: 
Expecting actual:
  "截至2024年9月10日,关于美国在2024年奥运会上获得的金牌总数的具体数据尚未公布。根据提供的参考资料,我们可以看到关于2024年奥运会的报道,但没有明确提及美国在奥运会上的金牌总数【1†source】【2†source】【3†source】【4†source】【5†source】【6†source】。

如果您需要了解美国在2024年奥运会的具体金牌数,建议关注官方奥运会网站或相关的体育新闻更新,因为该数据会随着比赛的进程而更新变化。"
to contain:
  "40" 

	at org.springframework.ai.minimax.chat.MiniMaxChatOptionsTests.testWebSearchStream(MiniMaxChatOptionsTests.java:124)
	at java.base/java.lang.reflect.Method.invoke(Method.java:580)

maybe I need to run in some other way as compared to how you are running them ?

Thanks once again!

@markpollack
Copy link
Member

merged in 38cbe638604f77bc8e471d95188ad2eb4d684e5

@markpollack markpollack closed this Sep 9, 2024
@markpollack markpollack added this to the 1.0.0-M3 milestone Sep 9, 2024
@mxsl-gr mxsl-gr deleted the feat/minimax-single-message branch September 10, 2024 00:10
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