first shim wrapper around litellm#37
Conversation
|
Can we please target this to a different branch than |
requirements.txt
Outdated
| rich | ||
| browserbase No newline at end of file | ||
| browserbase | ||
| litellm No newline at end of file |
There was a problem hiding this comment.
I would probably spec out the lib versions here - litellm moves very fast from my experience and its easy to introduce breaking changes.
stagehand/llm/client.py
Outdated
| try: | ||
| # Use litellm's completion function | ||
| if self.async_mode: | ||
| response = litellm.acompletion(**filtered_params) |
There was a problem hiding this comment.
you would need to await this in the async mode, right? this call is otherwise blocking the event loop i think
There was a problem hiding this comment.
yea good catch, gonna remove acompletion until we find a use case for it
filip-michalsky
left a comment
There was a problem hiding this comment.
LGTM -
great start, I assume the next step is to wire up the first stagehand primitive here... maybe observe as Ani said?
please merge to a different branch than main though
It's time to start doing a full rewrite. LiteLLM will help a lot
here is a sample test: