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.

Sign up for GitHub

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

Jump to bottom

Python: FunctionCallContent doesn't have deserialize arguments #6259

Closed
matthewbolanos opened this issue May 14, 2024 · 2 comments
Closed

Python: FunctionCallContent doesn't have deserialize arguments #6259

matthewbolanos opened this issue May 14, 2024 · 2 comments
Labels
python Pull requests for the Python Semantic Kernel triage

Comments

@matthewbolanos
Copy link
Member

matthewbolanos commented May 14, 2024 •
edited

In .NET the arguments for FunctionCallContent is of type KernelArguments
@markwallace-microsoft markwallace-microsoft added python Pull requests for the Python Semantic Kernel triage labels May 14, 2024
@eavanvalkenburg
Copy link
Member

@matthewbolanos it has a method called to_kernel_arguments which does this, we don't want to parse the arguments too soon for streaming scenario's, in that case you get a partial json in the mean term, so we call this function only when the stream is done.

@matthewbolanos
Copy link
Member Author

I can live with this. Closing

@matthewbolanos matthewbolanos closed this as not planned Won't fix, can't repro, duplicate, stale May 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
python Pull requests for the Python Semantic Kernel triage
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@matthewbolanos @eavanvalkenburg @markwallace-microsoft