search

CODAIT / r4ml

Scalable R for Machine Learning
Apache License 2.0
42 stars 13 forks source link

preprocess omit.na bug #47

Closed bdwyer2 closed 7 years ago

bdwyer2 commented 7 years ago

This fixes issue #46 where the omit.na argument to r4ml.ml.preprocess doesn't always work as expected. I also added additional test cases to make sure we don't run into this issue again in the future.

R4ML-CI commented 7 years ago

Build triggered.

R4ML-CI commented 7 years ago

Build success. All unit tests passed.

R4ML-CI commented 7 years ago

Build triggered.

R4ML-CI commented 7 years ago

Build success. All unit tests passed.

bdwyer2 commented 7 years ago

@aloknsingh any more comments?