LLM Provider updates - Githubissues

dimagi / open-chat-studio

A web based platform for building Chatbots backed by Large Language Models

BSD 3-Clause "New" or "Revised" License

16 stars 7 forks source link

LLM Provider updates #860

Closed snopoke closed 1 week ago

snopoke commented 1 week ago

Description

Show the list of models on the provider create / edit page
Allow the user to add / remove custom models from that page

User Impact

Default models are displayed to the user on the provider page
Users do not have to navigate to a separate page to view custom models

Demo

OCS custom model updates - 13 November 2024 - Watch Video

codecov-commenter commented 1 week ago

Codecov Report

Attention: Patch coverage is 69.76744% with 13 lines in your changes missing coverage. Please review.

Files with missing lines	Patch %	Lines
apps/service_providers/views.py	65.51%	10 Missing :warning:
apps/service_providers/forms.py	50.00%	2 Missing :warning:
apps/web/templatetags/default_tags.py	87.50%	1 Missing :warning:

Additional details and impacted files

:loudspeaker: Thoughts on this report? Let us know!

snopoke commented 1 week ago

LGTM! How often do users have more than one provider of the same type? I think I would find it confusing to add models for one provider but then have them show up elsewhere, although I do see why you moved this here. Also 👍🏽 to removing the edit view for these models.

Not many have more than one. Currently there are 3 teams that have 2 openai providers (not counting grok or others)