Skip to content

Commit

Permalink
Add _type to json functions output parser (#11381)
Browse files Browse the repository at this point in the history
<!-- Thank you for contributing to LangChain!

Replace this entire comment with:
  - **Description:** a description of the change, 
  - **Issue:** the issue # it fixes (if applicable),
  - **Dependencies:** any dependencies required for this change,
- **Tag maintainer:** for a quicker response, tag the relevant
maintainer (see below),
- **Twitter handle:** we announce bigger features on Twitter. If your PR
gets announced, and you'd like a mention, we'll gladly shout you out!

Please make sure your PR is passing linting and testing before
submitting. Run `make format`, `make lint` and `make test` to check this
locally.

See contribution guidelines for more information on how to write/run
tests, lint, etc:

https://github.com/langchain-ai/langchain/blob/master/.github/CONTRIBUTING.md

If you're adding a new integration, please include:
1. a test for the integration, preferably unit tests that do not rely on
network access,
2. an example notebook showing its use. It lives in `docs/extras`
directory.

If no one reviews your PR within a few days, please @-mention one of
@baskaryan, @eyurtsev, @hwchase17.
 -->
  • Loading branch information
nfcampos committed Oct 4, 2023
1 parent 106608b commit 0d80226
Showing 1 changed file with 5 additions and 1 deletion.
6 changes: 5 additions & 1 deletion libs/langchain/langchain/output_parsers/openai_functions.py
Original file line number Diff line number Diff line change
Expand Up @@ -54,6 +54,10 @@ class JsonOutputFunctionsParser(BaseCumulativeTransformOutputParser[Any]):
args_only: bool = True
"""Whether to only return the arguments to the function call."""

@property
def _type(self) -> str:
return "json_functions"

def _diff(self, prev: Optional[Any], next: Any) -> Any:
return jsonpatch.make_patch(prev, next).patch

Expand Down Expand Up @@ -127,7 +131,7 @@ class JsonKeyOutputFunctionsParser(JsonOutputFunctionsParser):

def parse_result(self, result: List[Generation], *, partial: bool = False) -> Any:
res = super().parse_result(result)
return res[self.key_name]
return res.get(self.key_name) if partial else res[self.key_name]


class PydanticOutputFunctionsParser(OutputFunctionsParser):
Expand Down

0 comments on commit 0d80226

Please sign in to comment.