MLBazaar / MLBlocks

A library for composing end-to-end tunable machine learning pipelines.
https://mlbazaar.github.io/MLBlocks
MIT License
114 stars 35 forks source link

Isolate primitives from their hyperparameters dictionary #94

Closed csala closed 5 years ago

csala commented 5 years ago

The current implementation does not isolate the MLBlock hyperparameters dictionary from the underlying primitive, allowing this to modify its content and leading to unexpected behaviors and bugs.

This isolation should be enforced by using deepcopy instead of a simple copy when returning the hyperparameters in the get_hyperparameters method, and by always accessing the hyperparameters through this method instead of directly when passing them to the underlying primitive.