search

ivanfioravanti / chatbot-ollama

Chatbot Ollama is an open source chat UI for Ollama.
Other
1.33k stars 217 forks source link

Default model check #7

Closed ryansereno closed 9 months ago

ryansereno commented 9 months ago

This one is more opinionated, so feel free to give feedback. Might not be exactly how you'd handle this.

But issue is, when I first ran this app, it wasn't clear to me why I was getting errors even though I had a model installed. Of course, it's because I neglected to set a .env default model variable or change the fallback model in ollama.ts. So here, I just added a toast alert to specify what exactly is going on; you may want to handle this in your own way, but this was my initial thought. This will probably also have implications when you get into adding/ deleting models?

When this error alert is combined with my first PR (adding useEffect to ModelSelector), I think it makes it a bit clearer to the user what is going on.

ivanfioravanti commented 9 months ago

I like this approach! We’ll think later about adding/removing models 💪