microsoft / semantic-kernel

Integrate cutting-edge LLM technology quickly and easily into your apps
https://aka.ms/semantic-kernel
MIT License
21.98k stars 3.27k forks source link

.Net: Investigate can we keep constructor for KernelFunction internal #3604

Closed markwallace-microsoft closed 11 months ago

markwallace-microsoft commented 12 months ago

I found not many but a few examples on GitHub of folks implementing ISKFunction:

How do we feel about these in the context of giving KernelFunction an internal ctor that prevents derivation?

markwallace-microsoft commented 11 months ago

We are going to start by keeping it internal as developers can create custom implementations from any arbitrary method and then they will get the pre/post function hooks support automatically.