-
Notifications
You must be signed in to change notification settings - Fork 3k
Add OVHcloud AI Endpoints block and add it also in Hugging Face block #2427
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
base: main
Are you sure you want to change the base?
Add OVHcloud AI Endpoints block and add it also in Hugging Face block #2427
Conversation
|
@eliasto is attempting to deploy a commit to the Sim Team on Vercel. A member of the Team first needs to authorize it. |
Greptile SummaryAdded OVHcloud AI Endpoints integration with chat and embeddings capabilities. Implementation follows codebase patterns and includes comprehensive documentation in English and French. Key Changes:
Critical Issue:
Confidence Score: 3/5
Important Files Changed
Sequence DiagramsequenceDiagram
participant User
participant SimUI as Sim UI/Block
participant OVHBlock as OVHcloud Block
participant ToolRegistry as Tool Registry
participant OVHChat as ovhcloud_chat Tool
participant OVHEmbed as ovhcloud_embeddings Tool
participant API as OVHcloud API Endpoints
User->>SimUI: Select OVHcloud AI Endpoints block
SimUI->>OVHBlock: Load block configuration
OVHBlock->>SimUI: Display operation dropdown (Chat/Embeddings)
alt Chat Operation
User->>SimUI: Select Chat, enter prompt & model
SimUI->>OVHBlock: params.operation = 'ovhcloud_chat'
OVHBlock->>ToolRegistry: Get ovhcloud_chat tool
ToolRegistry->>OVHChat: Execute with params
OVHChat->>API: POST /v1/chat/completions
API-->>OVHChat: Return completion response
OVHChat-->>SimUI: Return {content, model, usage}
else Embeddings Operation
User->>SimUI: Select Embeddings, enter input text & model
SimUI->>OVHBlock: params.operation = 'ovhcloud_embeddings'
OVHBlock->>ToolRegistry: Get ovhcloud_embeddings tool
ToolRegistry->>OVHEmbed: Execute with params
OVHEmbed->>API: POST /v1/embeddings
API-->>OVHEmbed: Return embeddings response
OVHEmbed-->>SimUI: Return {embedding, model, usage}
end
SimUI->>User: Display results
|
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.
Additional Comments (2)
-
apps/sim/tools/ovhcloud/embeddings.ts, line 10-22 (link)syntax: missing
inputparameter definition in params object - defined inOVHcloudEmbeddingsParamsinterface and used at line 35, but not declared here -
apps/sim/tools/ovhcloud/embeddings.ts, line 59 (link)syntax: output type mismatch - should be
arraynotstringsinceembeddingisnumber[]
20 files reviewed, 2 comments
4519cd1 to
f028a0b
Compare
Summary
Hi team,
This PR adds a new block for OVHcloud AI Endpoints, the inference provider of OVHcloud, the leading Cloud Provider in Europe.
It also adds documentation, and edit the Hugging Face block to add OVHcloud AI Endpoints.
Type of Change
Testing
A workflow has been created to test if the two block are functioning properly.
Checklist
Screenshots/Videos