Skip to content

refactor(plugins): rename configuration_model_default to example_configuration#940

Merged
AlessandroPomponio merged 2 commits into
mainfrom
ap_650_rename_default_operation_configuration
May 18, 2026
Merged

refactor(plugins): rename configuration_model_default to example_configuration#940
AlessandroPomponio merged 2 commits into
mainfrom
ap_650_rename_default_operation_configuration

Conversation

@AlessandroPomponio
Copy link
Copy Markdown
Member

Summary

This pull request refactors parameter naming across operator decorators and configuration models to improve clarity. The parameter configuration_model_default has been renamed to example_configuration throughout the codebase to better reflect its purpose as an example for templating rather than a default value. Similarly, class methods that generate these example configurations have been renamed from default_parameters() and defaultOperationParameters() to the more descriptive example_configuration().

Resolves #650

Files Changed

📄 orchestrator/modules/operators/collections.py

Renamed the configuration_model_default parameter to example_configuration in three operator decorator functions (characterize_operation, modify_operation, and export_operation). Updated corresponding docstrings to clarify that this parameter provides an "example parameter model instance for templating" rather than a "default parameter model instance". The internal usage of this parameter was also updated to use the new name consistently.

📄 plugins/operators/anomalous_series/anomalous_series/operator.py

Renamed the default_parameters() class method to example_configuration() in the DetectAnomalousSeries configuration model. Updated the decorator call to use the new example_configuration parameter name instead of configuration_model_default.

📄 plugins/operators/profile_space/profile_space/operator.py

Updated the @characterize_operation decorator to use the new example_configuration parameter name instead of configuration_model_default for the ProfileParameters configuration.

📄 plugins/operators/ray_tune/ado_ray_tune/rifferla.py

Renamed the defaultOperationParameters() class method to example_configuration() in the RifferlaParameters configuration model. Updated the decorator call to use the new example_configuration parameter name.

📄 plugins/operators/trim/src/trim/operator.py

Updated the @characterize_operation decorator to use the new example_configuration parameter name and call the renamed example_configuration() method instead of defaultOperationParameters().

📄 plugins/operators/trim/src/trim/trim_pydantic.py

Renamed the defaultOperationParameters() class method to example_configuration() in the TrimParameters configuration model to align with the new naming convention.

…iguration

Signed-off-by: Alessandro Pomponio <alessandro.pomponio1@ibm.com>
Signed-off-by: Alessandro Pomponio <alessandro.pomponio1@ibm.com>
@AlessandroPomponio AlessandroPomponio added this pull request to the merge queue May 18, 2026
Merged via the queue into main with commit c2d571f May 18, 2026
17 checks passed
@AlessandroPomponio AlessandroPomponio deleted the ap_650_rename_default_operation_configuration branch May 18, 2026 09:23
@michael-johnston
Copy link
Copy Markdown
Member

@VassilisVassiliadis FYI

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

refactor(operators): rename defaultOperationParameters to example_values (or similar)

2 participants