-
Notifications
You must be signed in to change notification settings - Fork 134
Inference gateway integration #22
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
rektdeckard
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice! Have not run the agent, but looks good
| # To add tools, use the @function_tool decorator. | ||
| # Here's an example that adds a simple weather tool. | ||
| # You also have to add `from livekit.agents.llm import function_tool, RunContext` to the top of this file | ||
| # @function_tool | ||
| # async def lookup_weather(self, context: RunContext, location: str): | ||
| # """Use this tool to look up current weather information in the given location. | ||
| # | ||
| # If the location is not supported by the weather service, the tool will indicate this. You must tell the user the location's weather is unavailable. | ||
| # | ||
| # Args: | ||
| # location: The location to look up weather information for (e.g. city name) | ||
| # """ | ||
| # | ||
| # logger.info(f"Looking up weather for {location}") | ||
| # | ||
| # return "sunny with a temperature of 70 degrees." |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Might be extra clear here to link to tool call docs page and to state that the return value is passed to LLM node?
| ```bash | ||
| lk app env -w .env.local | ||
| lk cloud auth | ||
| lk app env -w -d .env.local |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just my preference, but I like to add the long form so it's more obvious what each does:
lk app env --write --destination .env.local
longcw
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm!
134cea0 to
69505ad
Compare
Uh oh!
There was an error while loading. Please reload this page.