search

kearnz / autoimpute

Python package for Imputation Methods
MIT License
237 stars 19 forks source link

Strategy parameter not included in documentation #61

Closed nirtiac closed 3 years ago

nirtiac commented 3 years ago

The strategy parameter is not documented in key functions such as MultipleImputer. This sadly has been making it difficult for me to figure out how (and if) I can use this package for mixed-type imputation including with categorical features one-hot encoded.

kearnz commented 3 years ago

Hi @nirtiac,

Thanks for bringing this to my attention. The parameters should be autogenerated, and I had not noticed that the strategy parameter is missing. Let me look into this - should have a fix by the end of the week. Will leave the issue open until then.

Joe

kearnz commented 3 years ago

Hi @nirtiac,

I added the strategy param to the docstrings for each imputer. I also added the BaseImputer to the docs, which contains all the strategies. Still fidgeting with the docstring formatting for autogenerating docs, so bear with me there, but hopefully the content is at least helpful.

For the cleanest view on the strategies, you may want to look at the BaseImputer itself, followed by the folder that contains each strategy.

Closing for now, but feel free to re open if any other issues

nirtiac commented 3 years ago

Thank you @kearnz!! Much appreciated!