Skip to content

Comments

Tools support#36

Merged
tonybaloney merged 9 commits intotonybaloney:mainfrom
cmbrose:cmbrose/tools
May 29, 2025
Merged

Tools support#36
tonybaloney merged 9 commits intotonybaloney:mainfrom
cmbrose:cmbrose/tools

Conversation

@cmbrose
Copy link
Contributor

@cmbrose cmbrose commented May 28, 2025

Adding tools support, as usual piggybacking off the openai examples

$ llm -m github/gpt-4o --functions 'def multiply(x: int, y: int) -> int:
    """Multiply two numbers."""
    return x * y
' 'what is 34234 * 213345' --td

Tool call: multiply({'x': 34234, 'y': 213345})
  7303652730

34234 multiplied by 213345 equals 7,303,652,730.

@tonybaloney
Copy link
Owner

Needs rebasing. Please update https://github.com/tonybaloney/llm-github-models/blob/main/pyproject.toml#L13 to llm>=0.26 at the same time as this feature won't work in older versions.

@cmbrose cmbrose reopened this May 28, 2025
@cmbrose cmbrose marked this pull request as ready for review May 28, 2025 16:56
@cmbrose cmbrose changed the title [WIP] Tools support Tools support May 28, 2025
("o3", True, False, True, ["text", "image"], ["text"]),
("o3-mini", False, True, False, ["text"], ["text"]),
("o4-mini", True, False, True, ["text", "image"], ["text"]),
("AI21-Jamba-1.5-Large", True, False, False, False, ["text"], ["text"]),
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These tuples are getting out of control 😆

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It would be great if the GitHub Models API had an endpoint that returned all of these details!

@tonybaloney
Copy link
Owner

Fantastic! thank you

@tonybaloney tonybaloney merged commit ed9b7d2 into tonybaloney:main May 29, 2025
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants