search

awslabs / datawig

Imputation of missing values in tables.
Apache License 2.0
475 stars 69 forks source link

Update simple_imputer.py #130

Closed angeliney closed 4 years ago

angeliney commented 4 years ago

SimpleImputer.complete should delete the correct directories.

Issue #, if available: 129

Description of changes: Changing directory to delete within SimpleImputer.complete

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

tdhd commented 4 years ago

Rebased on master with pylint changes.

tdhd commented 4 years ago

@angeliney Can you add a unit test for this?

tdhd commented 4 years ago

Thanks for your PR!

I am closing this as I just merged https://github.com/awslabs/datawig/pull/135.