Skip to content
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

json parser + minor function call param change #17

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

adivik2000
Copy link
Contributor

@adivik2000 adivik2000 commented Sep 12, 2023

This PR has 3 changes related to function execution with chat.

  1. Added an extra parser from https://github.com/Stevenic/alphawave-py. Some of the function arguments returned by openai weren't being parsed by json.loads(). Hence added this.
  2. Changed the function_call param from True to "auto" as per https://platform.openai.com/docs/api-reference/chat/create#function_call
  3. Commented out argument validator for now as it was failing with (AttributeError: 'function' object has no attribute 'validate'). Anywhere I can read about this, that is not a pydantic decorator?

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.

1 participant