Skip to content
This repository has been archived by the owner on Oct 10, 2024. It is now read-only.

Add tool call id in TS and example for v3 tokenization #76

Merged
merged 4 commits into from
May 23, 2024

Conversation

fuegoio
Copy link
Contributor

@fuegoio fuegoio commented May 23, 2024

The following models have now support for multiple tool calls and therefore need a tool call id in the tool message to be able to identify which tool call is linked to which tool response:

  • mistral-small-2402
  • open-mixtral-8x22b
  • open-mistral-7b

@fuegoio fuegoio force-pushed the fuego/add-tool-call-id branch from 133146d to 47fcf76 Compare May 23, 2024 08:06
@fuegoio fuegoio force-pushed the fuego/add-tool-call-id branch from 47fcf76 to 38fda09 Compare May 23, 2024 08:10
Copy link
Contributor

@hrjn hrjn left a comment

Choose a reason for hiding this comment

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

A few small typos but lgtm otherwise 🙂

examples/function_calling.js Outdated Show resolved Hide resolved
examples/function_calling.js Outdated Show resolved Hide resolved
fuegoio and others added 2 commits May 23, 2024 17:13
@fuegoio fuegoio merged commit 7f1c9ad into main May 23, 2024
4 checks passed
@fuegoio fuegoio deleted the fuego/add-tool-call-id branch May 23, 2024 15:19
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants