Evan Mattson

Results 147 comments of Evan Mattson

Thanks for your response, @philippHorn. @TaoChenOSU what are your thoughts?

> I think a similar commit to [#10859](https://github.com/microsoft/semantic-kernel/pull/10859) is needed for python as well. Thanks, @riywo. The handling for Python was done in #10329.

Thanks for filing @q-stefanmuscalu. This is a known gap. I'll work on this.

This issue will be fixed by removing the `if param.is_required` condition here: https://github.com/microsoft/semantic-kernel/blob/0fda22e8f547e100e4dd9b8b1307142af42a251a/python/semantic_kernel/connectors/ai/function_calling_utils.py#L39C98-L39C115.

@slreznit closing this PR as there hasn't been any activity since the end of April. If you'd like to move forward, please re-open or create a new PR. Thanks.

This is looking great, @andrewldesousa. Thank you for working on it!