Python: Improve handling for kernel plugin from file. #10286
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Motivation and Context
This PR addresses an issue in the
KernelPlugin.from_directory()
method where classes without any@kernel_function
decorated methods are still being initialized during plugin loading.The fix makes sure that only classes with at least one
@kernel_function
decorated method are instantiated and included as plugins. Classes without@kernel_function
methods are now skipped entirely.Description
This PR:
from_python_file
to inspect each class for@kernel_function
decorated methods before instantiation.add_plugin
methodplugin_name
to make sure it is indeed a string, per the type hint.Contribution Checklist