Skip to content

docs(giskard-llm): OpenAI-compatible multi-model gateway example (DaoXE) - #2606

Open
seven7763 wants to merge 2 commits into
Giskard-AI:mainfrom
seven7763:docs/daoxe-openai-compatible-gateway
Open

docs(giskard-llm): OpenAI-compatible multi-model gateway example (DaoXE)#2606
seven7763 wants to merge 2 commits into
Giskard-AI:mainfrom
seven7763:docs/daoxe-openai-compatible-gateway

Conversation

@seven7763

Copy link
Copy Markdown

Summary

Adds a short example to libs/giskard-llm/README.md showing how to point LLMClient at an OpenAI-compatible multi-model gateway via provider=\"openai\" + custom base_url.

Example uses DaoXE (https://daoxe.com/v1).

  • Docs only
  • Same pattern as Azure Foundry OpenAI v1 (openai provider + base_url)

Notes

  • Account-scoped model IDs
  • Chat Completions path
  • DaoXE not available in mainland China
  • Contributor is affiliated with DaoXE

Test plan

  • README section renders
  • Optional: LLMClient.configure(\"daoxe\", provider=\"openai\", base_url=\"https://daoxe.com/v1\") with a real key

Show LLMClient.configure(provider=openai, base_url=https://daoxe.com/v1)
with account-scoped model IDs. Chat Completions path only.

Signed-off-by: seven7763 <246023385+seven7763@users.noreply.github.com>

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Code Review

This pull request updates the libs/giskard-llm/README.md file to include a new section on OpenAI-compatible multi-model gateways, providing a Python code example using DaoXE. The review feedback suggests a minor grammatical improvement to one of the bullet points in the notes section to enhance readability.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

Comment thread libs/giskard-llm/README.md Outdated
Notes:

- Model IDs are account-scoped; do not hardcode a static public catalog.
- Chat Completions path only on this snippet.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

medium

The phrase 'Chat Completions path only on this snippet' is grammatically incomplete. Rephrasing it to be a complete sentence improves readability and clarity.

Suggested change
- Chat Completions path only on this snippet.
- This snippet only demonstrates the Chat Completions path.

@seven7763

Copy link
Copy Markdown
Author

Addressed the Gemini wording note in 75ab957: rewrote the note to a complete sentence — “This snippet only demonstrates the Chat Completions path.”

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

Development

Successfully merging this pull request may close these issues.

1 participant