Skip to content

[fix #15] ChatFunctionCall arguments field typed as Text#16

Open
Saizan wants to merge 1 commit into
agrafix:mainfrom
Saizan:issue15
Open

[fix #15] ChatFunctionCall arguments field typed as Text#16
Saizan wants to merge 1 commit into
agrafix:mainfrom
Saizan:issue15

Conversation

@Saizan

@Saizan Saizan commented Aug 21, 2023

Copy link
Copy Markdown

Here's a fix for issue 15, just changing the type of the field and the To/FromJSON instances.

@agrafix

agrafix commented Oct 25, 2023

Copy link
Copy Markdown
Owner

Thanks! What if we made this an Either of a parsed value or a text to make it more explicit how it could go "wrong" but keep the benefits of the parsed representation otherwise?

@nickhs

nickhs commented Mar 23, 2024

Copy link
Copy Markdown
Collaborator

hey @Saizan, let me know if you'd like to make those changes otherwise happy to do so! I'll also see if I can get a reliable hallucination and add a test

@Saizan

Saizan commented Mar 23, 2024

Copy link
Copy Markdown
Author

Feel free to go ahead!

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