sotopia-lab / sotopia

Sotopia: an Open-ended Social Learning Environment (ICLR 2024 spotlight)
https://docs.sotopia.world
MIT License
148 stars 19 forks source link

[FEAT]: Deprecating LLM Names #43

Closed ProKil closed 4 months ago

ProKil commented 5 months ago

Description

As we are moving towards LiteLLM as the unified backend for all LLM calls, and use runtime checking for verifying llm names. Let's remove LLM_Name type.

Additional Information

No response

XuhuiZhou commented 5 months ago

We have some special agent names that need to be especially careful: human redis

LiteLLM won't handle those situation

ProKil commented 5 months ago

The question is whether we want to keep a way to statically check the validity of the LLM's name? We can also create a router to handle model names not accepted by LiteLLM.

XuhuiZhou commented 4 months ago

We are keeping the LLM_Name