Skip to content

Conversation

@neha-nupoor
Copy link
Contributor

@neha-nupoor neha-nupoor commented Jun 2, 2025

Description

  • Added common providers to the UI

Screenshots/Results
Screenshot 2025-06-01 at 7 36 32 PM

ollama working
Screenshot 2025-06-01 at 9 15 29 PM

  • Add a screenshot of the changes or code responses

Reference Links

  • Links to JIRA issue, links, Slack discussions, etc.

Checklist

This PR includes the following (tick all that apply):

  • Tests added/updated for new/changed functionality
  • Bug Fix (explain the bug in the description)
  • Refactoring or optimizations (no functional changes)
  • Documentation updated (README, docstrings, etc.)
  • Build or deployment related changes
  • Dependency updates (requirements.txt/pyproject.toml)
  • Setup change (update README/setup instructions if required)
  • Code style checks passed (PEP8, flake8, black, etc.)
  • Type hints added/updated (if applicable)
  • Pre-commit hooks run (if configured)

Copy link
Contributor

@kaushik-himself kaushik-himself left a comment

Choose a reason for hiding this comment

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

LGTM

try:
# Import litellm here to avoid dependency issues
from litellm import acompletion # Using acompletion instead of completion for async
from litellm import acompletion, _turn_on_debug # Using acompletion instead of completion for async
Copy link
Contributor

Choose a reason for hiding this comment

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

revert

{"name": "Azure", "value": "azure", "provider_name": "azure", "azure_api_key": "", "azure_api_base": "", "azure_api_version": "", "default_model": "gpt-4o"},
{"name": "vLLM Ollama", "value": "vllm_ollama", "provider_name": "hosted_vllm", "hosted_vllm_api_base": "", "default_model": "llama3.2:3b"}
{"name": "vLLM Ollama", "value": "vllm_ollama", "provider_name": "hosted_vllm", "hosted_vllm_api_base": "", "default_model": "llama3.2:3b"},
{"name": "Ollama", "value": "ollama", "provider_name": "ollama", "ollama_api_base": "http://localhost:11434", "default_model": "llama3.2:1b"},
Copy link
Contributor

Choose a reason for hiding this comment

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

Will run these and check

@kaushik-himself kaushik-himself merged commit 588d3d7 into main Jun 3, 2025
3 checks passed
@kaushik-himself kaushik-himself deleted the feat/add_ollama branch June 3, 2025 00:16
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.

4 participants