-
Notifications
You must be signed in to change notification settings - Fork 604
provider: added support for Hyperbolic AI #1108
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
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks good, just need to remove the boilerplate
src/providers/hyperbolic/index.ts
Outdated
} from './imageGenerate'; | ||
|
||
const HyperbolicConfig: ProviderConfigs = { | ||
chatComplete: HyperbolicChatCompleteConfig, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I can see in the documentation that the chat completions endpoint is openai compliant, you can use the transformers for openai-base and avoid the boilerplate
checkout the implementation for any of groq, x-ai, nebius etc.
the open-ai-base package has the implementation
example PR: https://github.com/Portkey-AI/gateway/pull/801/files
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for your review! I'm working on the requested changes and will update the pull request soon.
Important PR Review SkippedPR review skipped as per the configuration setting. Run a manually review by commenting /matter review 💡Tips to use Matter AICommand List
|
@narengogi, I have made the necessary changes. Just had one question about the documentation, parameters frequency_penalty and presence_penalty have different value ranges from the OpenAI docs. Also, I have added extra params ( top_k, min_p, repetition_penalty ) which were not present in openai listed params. Please take a look and let me know if anything else is needed. |
Important PR Review SkippedPR review skipped as per the configuration setting. Run a manually review by commenting /matter review 💡Tips to use Matter AICommand List
|
Important PR Review SkippedPR review skipped as per the configuration setting. Run a manually review by commenting /matter review 💡Tips to use Matter AICommand List
|
Description
Added Hyperbolic provider support, including API configuration and response transforms for chat completion and image generation.
Motivation
This adds support for the Hyperbolic provider as requested in issue #672
Type of Change
Related Issues
Fixes #672