Skip to content

feat: Update orchestration specification#1646

Open
github-actions[bot] wants to merge 5 commits intomainfrom
spec-update/orchestration/rel-0.115.19
Open

feat: Update orchestration specification#1646
github-actions[bot] wants to merge 5 commits intomainfrom
spec-update/orchestration/rel-0.115.19

Conversation

@github-actions
Copy link
Copy Markdown
Contributor

Context

Update orchestration specification file based on rel-0.115.19.

This PR was created automatically by the spec-update workflow.
You can commit on top of this branch, but as long as this PR is open the action can't be re-run.

  • Compilation outcome: failure
  • Test run outcome: skipped

Before merging, make sure to update tests and release notes, if necessary.

Definition of Done

  • Unit / type tests cover new classes
  • Release notes / Changeset updated

Copy link
Copy Markdown
Member

@davidkna-sap davidkna-sap left a comment

Choose a reason for hiding this comment

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

@davidkna-sap davidkna-sap dismissed their stale review March 30, 2026 09:46

Discussed with Charles

@@ -22,7 +22,6 @@ export class OrchestrationEmbeddingResponse {
* @returns Array of embedding data objects containing both vectors, indices, and object types.
*/
getEmbeddings(): EmbeddingData[] {
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I considered adding an argument here to return a specific embedding for multi-embedding, but decided against it (e.g. getEmbeddings('float') only support for multi-embeddings). I have an implementation locally and could push it if necessary.

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.

3 participants