You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
github-actionsbot
changed the title
Python: missing more advanced json schema serializer
.Net: Python: missing more advanced json schema serializer
May 16, 2024
github-actionsbot
changed the title
.Net: Python: missing more advanced json schema serializer
Python: missing more advanced json schema serializer
May 16, 2024
… Memory Connectors. (#6335)
### Motivation and Context
The Python code base could handle some primitive types for the schema
for tool call objects and kernel parameter metadata. However, it
couldn't properly handle the more complex JSON schemas for tool call
objects.
<!-- Thank you for your contribution to the semantic-kernel repo!
Please help reviewers and future users, providing the following
information:
1. Why is this change required?
2. What problem does it solve?
3. What scenario does it contribute to?
4. If it fixes an open issue, please link to the issue here.
-->
### Description
This PR introduces:
- JSON schema handling for KernelParameterMetadata and tool call
objects.
- Updates to the tool call utils method to properly recurse on the
KernelParameterMetadata's stucture.
- Adds unit tests for this code.
- Add experimental_class/experimental_functions to various parts of the
code base like Memory Connectors and OpenAPI plugin.
- Fixes#6310
- Fixes#6280
<!-- Describe your changes, the overall approach, the underlying design.
These notes will help understanding how your code works. Thanks! -->
### Contribution Checklist
<!-- Before submitting this PR, please make sure: -->
- [X] The code builds clean without any errors or warnings
- [X] The PR follows the [SK Contribution
Guidelines](https://github.com/microsoft/semantic-kernel/blob/main/CONTRIBUTING.md)
and the [pre-submission formatting
script](https://github.com/microsoft/semantic-kernel/blob/main/CONTRIBUTING.md#development-scripts)
raises no violations
- [X] All unit tests pass, and I have added new tests where possible
- [X] I didn't break anyone 😄
Describe the bug
A clear and concise description of what the bug is.
Python can handle some schema types, but for more advanced schemas and handling there is no KernelJsonSchema.
In dotnet's KernelParameterMetadata, there exists:
KernelParameterMetadata L103
semantic-kernel/dotnet/src/SemanticKernel.Abstractions/Functions/KernelParameterMetadata.cs at main · microsoft/semantic-kernel (github.com)
The text was updated successfully, but these errors were encountered: