Skip to content
This repository was archived by the owner on Jul 27, 2026. It is now read-only.

Update generate_configs.py - #132

Open
mzamini92 wants to merge 1 commit into
IBM:mainfrom
mzamini92:patch-1
Open

Update generate_configs.py#132
mzamini92 wants to merge 1 commit into
IBM:mainfrom
mzamini92:patch-1

Conversation

@mzamini92

@mzamini92 mzamini92 commented Jul 20, 2023

Copy link
Copy Markdown

some improvements on generate_configs file.
Instead of directly comparing the dataset names in get_data_handler_config, we can use constants to make the code more readable and maintainable. Rather than passing strings for model types like "sklearn" we used enums to represent different model types. Improved the serialization technique.
Signed-off-by: Mohamad Zamini mohamad.zamini@gmail.com

some improvements on generate_configs file. 
Instead of directly comparing the dataset names in get_data_handler_config, we can use constants to make the code more readable and maintainable.
Rather than passing strings for model types like "sklearn" we used enums to represent different model types. 
Improved the serialization technique.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant