search

LMZimmer / Auto-PyTorch_refactor

Apache License 2.0
0 stars 2 forks source link

Fix documentation #30

Open ArlindKadra opened 3 years ago

ArlindKadra commented 3 years ago

https://github.com/LMZimmer/Auto-PyTorch_refactor/blob/0cee2749196e1b04a298b0e7f1357c7863326ffc/autoPyTorch/pipeline/tabular_classification.py#L76

It does not really fit the pipeline, it only applies the preprocessing step of the pipeline, it is confusing.

LMZimmer commented 3 years ago

@ravinkohli @franchuterivera Do I see it right that fit_transformer https://github.com/LMZimmer/Auto-PyTorch_refactor/blob/129cc7bc43550d1641378be1f7a1999b711a8316/autoPyTorch/pipeline/tabular_classification.py#L71 is actually unused? Then I'll just delete it

ravinkohli commented 3 years ago

I think it can be safely deleted