Skip to content

Conversation

@ktpedre
Copy link
Collaborator

@ktpedre ktpedre commented Nov 22, 2025

No description provided.

@garland3 garland3 requested a review from Copilot November 23, 2025 00:16
Copilot finished reviewing on behalf of garland3 November 23, 2025 00:18
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR adds support for Cerebras AI as a new LLM provider to the application's LiteLLM integration. Cerebras is integrated following the same pattern as existing providers (OpenAI, Anthropic, Google), with changes to the model configuration, provider detection logic, API key handling, and environment variable documentation.

Key Changes:

  • Added Cerebras provider detection and model name mapping to LiteLLMCaller
  • Configured a new Cerebras model in the LLM configuration
  • Added CEREBRAS_API_KEY to environment variable examples

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.

File Description
config/overrides/llmconfig.yml Adds cerebras-gpt-oss-120b model configuration with Cerebras API endpoint and credentials
backend/modules/llm/litellm_caller.py Implements Cerebras provider support with URL detection, model name mapping, API key handling, and api_base exclusion logic
.env.example Adds CEREBRAS_API_KEY environment variable placeholder for developer reference

compliance_level: "External"
compliance_level: "External"
cerebras-gpt-oss-120b:
model_url: "https://api.cerebras.ai/v1"
Copy link

Copilot AI Nov 23, 2025

Choose a reason for hiding this comment

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

The model_url for Cerebras appears incomplete compared to other providers. OpenAI uses the full endpoint path https://api.openai.com/v1/chat/completions, but Cerebras only specifies https://api.cerebras.ai/v1.

If LiteLLM's Cerebras provider requires the full endpoint path (like /v1/chat/completions), this configuration may not work correctly. Please verify that https://api.cerebras.ai/v1 is the correct format for LiteLLM's Cerebras integration, or update it to include the complete endpoint path if needed.

Suggested change
model_url: "https://api.cerebras.ai/v1"
model_url: "https://api.cerebras.ai/v1/chat/completions"

Copilot uses AI. Check for mistakes.
compliance_level: "External"
cerebras-gpt-oss-120b:
model_url: "https://api.cerebras.ai/v1"
model_name: "gpt-oss-120b"
Copy link

Copilot AI Nov 23, 2025

Choose a reason for hiding this comment

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

The model name gpt-oss-120b appears to be incorrect for a Cerebras model. This same model name is used for OpenRouter configurations (see line 14 in this file), which suggests it might be a copy-paste error.

Cerebras models typically have names like llama3.1-8b or llama3.1-70b. Please verify and update the model name to match an actual Cerebras model identifier that will be recognized by LiteLLM's Cerebras provider.

Suggested change
model_name: "gpt-oss-120b"
model_name: "llama3.1-70b"

Copilot uses AI. Check for mistakes.
@garland3 garland3 merged commit 09cf354 into sandialabs:main Nov 23, 2025
13 of 15 checks passed
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.

2 participants