search

Neuraxio / Neuraxle

The world's cleanest AutoML library ✨ - Do hyperparameter tuning with the right pipeline abstractions to write clean deep learning production pipelines. Let your pipeline steps have hyperparameter spaces. Design steps in your pipeline like components. Compatible with Scikit-Learn, TensorFlow, and most other libraries, frameworks and MLOps environments.
https://www.neuraxle.org/
Apache License 2.0
608 stars 62 forks source link

Bug: Cleanup all of the scipy distributions that are already available within the regular distribution module #520

Closed guillaume-chevalier closed 2 years ago

guillaume-chevalier commented 3 years ago

Describe the bug Some distributions are duplicated in the distributions modules with scipy implementation. This complexifies the maintenance and addition of new features. For instance:

To Reproduce try to auto import distributions and you will find two versions.

Expected behavior This was never asked for to be in Neuraxle and was part of #380. The goal of #380 was to be able to do: 

hyperparams_space = HyperparamsSpace({
    'some_hp_dist_neuraxle' = RandInt(1, 2),
    'some_hp_dist_scipy' = scipy.randint(1, 2)
}]

But instead the scipy distributions were implemented like Neuraxle distributions in addition to building the wrapper of the scipy distributions.

Suggested Fix Delete all of the distributions in Neuraxle that are already in Neuraxle format to remove duplicates.