spring-ai icon indicating copy to clipboard operation
spring-ai copied to clipboard

Published 20 hours ago • verified publisher icon spring-projects

Add Moonshot model client

Open mxsl-gr opened this issue 10 months ago • 1 comments

Hi, this PR is add Moonshot model client and has passed unit testing. I can provide my api_key if needed for testing

the PR content:

  • Moonshot AI model client
  • unit test
  • spring starter
  • chat client documents 

Moonshot is a popular LLM in China Mainland recently, the link: Moonshot

mxsl-gr avatar Apr 17 '24 07:04 mxsl-gr

Add autoconfigure unit test and squash commits.

mxsl-gr avatar Apr 23 '24 02:04 mxsl-gr

Hi @markpollack , sorry to bother you, I noticed there have been some recent refactorings to the code. Do these model client PRs also need to be adjusted accordingly?

mxsl-gr avatar Jun 03 '24 04:06 mxsl-gr

@tzolov hi, sorry to bother you, it seems Mark is on vacation... this PR has completed the refactoring, has been squashed and force pushed again, and also added support for function calls.

mxsl-gr avatar Jun 20 '24 11:06 mxsl-gr

I sent the API keys for Moonshot and others model provider's key to Mark via email before. I'm not sure if Mark has shared them with you. If you need them, please contact me and I can provide them again, including QianFan on https://github.com/spring-projects/spring-ai/pull/751 and DeepSeek https://github.com/spring-projects/spring-ai/pull/702

mxsl-gr avatar Jun 20 '24 11:06 mxsl-gr

Hello, I apologize but was pulled into other areas of work. I will be going through this today. Thanks again.

markpollack avatar Jun 21 '24 14:06 markpollack

Hello, I apologize but was pulled into other areas of work. I will be going through this today. Thanks again.

Wow, welcome back, I just checked the emails, and the API keys for Deepseek and Moonshot were sent before. I noticed that i haven't sent the API key for Qianfan, so I just sent you a new email with it. but the balance in it is not very much, but it should be enough for testing, If the balance is insufficient, please contact me.

mxsl-gr avatar Jun 21 '24 15:06 mxsl-gr

Merged in d5b8123e60411f5d4a2cd5ee9eaf31f92a72c88c made a few minor changes. Note that BeanOutputParser is deprecated, so the tests should use BeanOutputConverter and other similar OutputConverter classes.

The api keys you sent worked!

markpollack avatar Jun 21 '24 19:06 markpollack