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

added support for different types in ResponseSchema class #5789

Merged
merged 3 commits into from
Jun 7, 2023

Conversation

MassimilianoBiancucci
Copy link
Contributor

@MassimilianoBiancucci MassimilianoBiancucci commented Jun 6, 2023

I added support for specifing different types with ResponseSchema objects:

before

extracted_info = ResponseSchema(name="extracted_info", description="List of extracted information")
generate the following doc: json\n{\n\t\"extracted_info\": string // List of extracted information}
This brings GPT to create a JSON with only one string in the specified field even if you requested a List in the description.

now

extracted_info = ResponseSchema(name="extracted_info", type="List[string]", description="List of extracted information")
generate the following doc: json\n{\n\t\"extracted_info\": List[string] // List of extracted information}
This way the model responds better to the prompt generating an array of strings.

Tag maintainers/contributors who might be interested:
Agents / Tools / Toolkits
@vowelparrot

Don't know who can be interested, I suppose this is a tool, so I tagged you vowelparrot,
anyway, it's a minor change, and shouldn't impact any other part of the framework.

@vowelparrot
Copy link
Contributor

Thanks!

@hwchase17 hwchase17 merged commit 92b87c2 into langchain-ai:master Jun 7, 2023
12 checks passed
Undertone0809 pushed a commit to Undertone0809/langchain that referenced this pull request Jun 19, 2023
…ai#5789)

I added support for specifing different types with ResponseSchema
objects:

## before
`
extracted_info = ResponseSchema(name="extracted_info", description="List
of extracted information")
`
generate the following doc: ```json\n{\n\t\"extracted_info\": string //
List of extracted information}```
This brings GPT to create a JSON with only one string in the specified
field even if you requested a List in the description.

## now
`extracted_info = ResponseSchema(name="extracted_info",
type="List[string]", description="List of extracted information")
`
generate the following doc: ```json\n{\n\t\"extracted_info\":
List[string] // List of extracted information}```
This way the model responds better to the prompt generating an array of
strings.

Tag maintainers/contributors who might be interested:
  Agents / Tools / Toolkits
  @vowelparrot

Don't know who can be interested, I suppose this is a tool, so I tagged
you vowelparrot,
anyway, it's a minor change, and shouldn't impact any other part of the
framework.
This was referenced Jun 25, 2023
kacperlukawski pushed a commit to kacperlukawski/langchain that referenced this pull request Jun 29, 2023
…ai#5789)

I added support for specifing different types with ResponseSchema
objects:

## before
`
extracted_info = ResponseSchema(name="extracted_info", description="List
of extracted information")
`
generate the following doc: ```json\n{\n\t\"extracted_info\": string //
List of extracted information}```
This brings GPT to create a JSON with only one string in the specified
field even if you requested a List in the description.

## now
`extracted_info = ResponseSchema(name="extracted_info",
type="List[string]", description="List of extracted information")
`
generate the following doc: ```json\n{\n\t\"extracted_info\":
List[string] // List of extracted information}```
This way the model responds better to the prompt generating an array of
strings.

Tag maintainers/contributors who might be interested:
  Agents / Tools / Toolkits
  @vowelparrot

Don't know who can be interested, I suppose this is a tool, so I tagged
you vowelparrot,
anyway, it's a minor change, and shouldn't impact any other part of the
framework.
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.

None yet

3 participants