-
Notifications
You must be signed in to change notification settings - Fork 381
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.
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
Add description field to kernelInfo #3399
Conversation
KernelInfo.Description = """ | ||
This Kernel can compile and execute C# code and display the results. | ||
The language is C# Scripting, a dialect of C# that is used for interactive programming. | ||
"""; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is this displayed anywhere in the UI? Will we need to localize this description for users who are on other languages?
Should we add a description for the composite kernel created by the dotnet-interactive tool? |
The description are not displayed in any UI and I added already the description to the composite kernel in the extension and on js and html kernels too
|
Ah I was referring to the composite kernel that is created in this method. Not super important though - we can always add it later. |
That composite kernel (like the one in the dotnet tool) doesn't do anything special besides routing, there is no need for custom description. |
This Pr adds support for description text on
KernelInfo
.The description property is forwarded to
ProxyKernels
too.Removed unused code with mermaid extension.