OpenHands icon indicating copy to clipboard operation
OpenHands copied to clipboard

fix: add the community-maintained model list

Open iFurySt opened this issue 1 year ago • 4 comments

#591 #649 #259 multi issues discuss this problem, i just merged the model_list and model_cost.keys()(community-maintained) to solve this problem for now.

image image

iFurySt avatar Apr 03 '24 14:04 iFurySt

I was pondering switching to model_cost.keys() as well - the point I got hung up on is, does it make sense for OpenDevin to list vision models?

foragerr avatar Apr 03 '24 14:04 foragerr

yes, but the community-maintained models are not only vision models, sth like

  "groq/mixtral-8x7b-32768",
  "groq/llama2-70b-4096",
  "groq/gemma-7b-it",

also be used by someone.

iFurySt avatar Apr 03 '24 14:04 iFurySt

It seems like the underlying issue was fixed upstream in LiteLLM. Do we still need this?

rbren avatar Apr 04 '24 03:04 rbren

Do we still need this?

Maybe.

There's still a lot of discrepancies between what model_cost.keys() returns and model_list returns. I can see some value in using a superset. But users should probably be warned that some entries on the super long list (~400 entries) are potentially unusable.

Common models: 206
Models in model_cost, but not model_list: 102
Models in model_list, but not model_cost: 45

I posted a question for the litellm folks for their opinion/guidance.

foragerr avatar Apr 04 '24 14:04 foragerr

👍 let's keep an eye on LiteLLM's response

rbren avatar Apr 05 '24 03:04 rbren