Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add Moonshot model client #596

Closed
wants to merge 1 commit into from
Closed

Conversation

mxsl-gr
Copy link
Contributor

@mxsl-gr mxsl-gr commented Apr 17, 2024

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
Copy link
Contributor Author

mxsl-gr commented Apr 23, 2024

Add autoconfigure unit test and squash commits.

@mxsl-gr mxsl-gr mentioned this pull request Apr 24, 2024
@tzolov tzolov added this to the 1.0.0-M1 milestone Apr 26, 2024
@mxsl-gr mxsl-gr mentioned this pull request May 22, 2024
@tzolov tzolov modified the milestones: 1.0.0-M1, 1.0.0-M2 May 28, 2024
@mxsl-gr
Copy link
Contributor Author

mxsl-gr commented Jun 3, 2024

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
Copy link
Contributor Author

mxsl-gr commented Jun 20, 2024

@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
Copy link
Contributor Author

mxsl-gr commented Jun 20, 2024

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 #751 and DeepSeek #702

@markpollack
Copy link
Member

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

@mxsl-gr
Copy link
Contributor Author

mxsl-gr commented Jun 21, 2024

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.

@markpollack
Copy link
Member

Merged in d5b8123 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!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants