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

[Provider] xAI #757

Merged
merged 7 commits into from
Nov 26, 2024
Merged

[Provider] xAI #757

merged 7 commits into from
Nov 26, 2024

Conversation

Akshay-66
Copy link
Contributor

Title:

  • Added xAI to Provider

Description:

  • Done with chat completion part in xai
  • Integrate xAI provider configuration

Related Issues:

added the provider xai for portkey
Added xAI configuration to Providers list
@VisargD
Copy link
Collaborator

VisargD commented Nov 21, 2024

Hi @Akshay-66 ! Thanks for the PR. We will review this today.

src/providers/index.ts Outdated Show resolved Hide resolved
src/providers/xAI/index.ts Outdated Show resolved Hide resolved
src/providers/xAI/chatComplete.ts Outdated Show resolved Hide resolved
src/providers/x-ai/index.ts Outdated Show resolved Hide resolved
- added complete bool
@b4s36t4
Copy link
Contributor

b4s36t4 commented Nov 25, 2024

@Akshay-66 Please resolve conflicts.

- Embedding Model Name is "v1" it seems (But didn't work on test).
Copy link
Contributor

@b4s36t4 b4s36t4 left a comment

Choose a reason for hiding this comment

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

LGTM!

@VisargD VisargD merged commit 8dde9dc into Portkey-AI:main Nov 26, 2024
1 check passed
@Akshay-66 Akshay-66 deleted the provider branch November 27, 2024 15:36
@Akshay-66
Copy link
Contributor Author

Thank you @narengogi and @vrushankportkey for guiding me in every step possible and thanks to @VisargD and @b4s36t4 for reviews and allowing me to do everything 😄

@vrushankportkey
Copy link
Collaborator

Thank you @Akshay-66 for the contribution!

@narengogi narengogi self-assigned this Dec 20, 2024
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.

5 participants