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

Move Function Arguments to String #422

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Conversation

fkesheh
Copy link

@fkesheh fkesheh commented Nov 24, 2023

Current functions implementation doesn't work with GPT-4-Turbo, forming bad JSON strings due to JsonNode usage when concatenating chunks (#338)

Trying to execute ""
Trying to execute "{""
Trying to execute "{"location"
Trying to execute "{"location:"
Trying to execute "{"location:Bar"
Trying to execute "{"location:Barcelona"
Trying to execute "{"location:Barcelona,"
Trying to execute "{"location:Barcelona,unit"
Trying to execute "{"location:Barcelona,unit:"
Trying to execute "{"location:Barcelona,unit:CE"
Trying to execute "{"location:Barcelona,unit:CELS"
Trying to execute "{"location:Barcelona,unit:CELSI"
Trying to execute "{"location:Barcelona,unit:CELSIUS"
Trying to execute "{"location:Barcelona,unit:CELSIUS"}"

! com.fasterxml.jackson.core.JsonParseException: Unexpected character ('}' (code 125)): was expecting a colon to separate field name and value
! at [Source: (String)"{"location:Barcelona,unit:CELSIUS"}"; line: 1, column: 36]

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.

1 participant